Modify

Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#10684 closed defect (fixed)

Fix icon scaling

Reported by: stoecker Owned by: team
Priority: normal Milestone: 16.01
Component: Core Version:
Keywords: icon scale Cc: Klumbumbus, skyper

Description

Currently we do hard scaling of many icons to 24 or 16 pixel (preferences tabs?).

This should be a settings value, so that a user can choose "big icons" for large screen resolutions.

Attachments (2)

squareicons.png (62.4 KB ) - added by Klumbumbus 9 years ago.
upscaling.png (15.1 KB ) - added by Klumbumbus 8 years ago.

Download all attachments as: .zip

Change History (53)

comment:1 by Don-vip, 9 years ago

If you think to people with Retina screens that won't change anything. The OSX Aqua look and feel force the icon size and we can't do anything about it except switching from icon tabs to text tabs.

comment:2 by stoecker, 9 years ago

Where does it force the icon size?

  • toolbar
  • menu
  • mapview
  • preferences
  • table entries
  • ...

We have many places were we use scaled images. And not all of them are proper anymore with 4k displays.

Thought I will start major rework only if I have a 4k screen :-)

comment:3 by Don-vip, 9 years ago

at least preference tabs for sure:

One other thing to keep in mind about JTabbedPane objects in OS X is that they have a standard size. If you put an image in a tab, the image is scaled to fit the tab instead of the tab to the image. This standard size applies to several other Swing components as well.

I don't know what "other Swing components" are also impacted.

comment:4 by Klumbumbus, 9 years ago

Cc: Klumbumbus added

comment:5 by stoecker, 9 years ago

See also #10688 comments.

comment:6 by stoecker, 9 years ago

In 7687/josm:

see #10684, see #10688 fix icon scaling a bit

comment:7 by stoecker, 9 years ago

In 7691/josm:

see #10684 - large icons also for imagery

comment:8 by stoecker, 9 years ago

In 7693/josm:

see #10684 - no double loading of images, cleanup action icons - menu icon size defaults now to 16x16 (previously most time 24x24) with some errors - maybe 24x24 should be default?

by Klumbumbus, 9 years ago

Attachment: squareicons.png added

comment:9 by Klumbumbus, 9 years ago

icons are stretched now to square size. See for example shop=beauty ("Schönheitssalon" in the screenshot)


Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-11-02 17:58:46
Last Changed Author: Don-vip
Revision: 7695
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Relative URL: ^/trunk
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-11-02 18:43:27 +0100 (Sun, 02 Nov 2014)
Last Changed Rev: 7695

Identification: JOSM/1.5 (7695 de) Windows 7 32-Bit
Memory Usage: 326 MB / 742 MB (101 MB allocated, but free)
Java version: 1.7.0_71, Oracle Corporation, Java HotSpot(TM) Client VM
VM arguments: [-Djava.security.manager, -Djava.security.policy=file:C:\Program Files\Java\jre7\lib\security\javaws.policy, -DtrustProxy=true, -Djnlpx.home=<java.home>\bin, -Djnlpx.origFilenameArg=C:\Program Files\josm-latest.jnlp, -Djnlpx.remove=true, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.heapsize=256m,768m, -Djnlpx.splashport=54219, -Djnlpx.jvm=<java.home>\bin\javaw.exe, -Djnlpx.vmargs=LURqYXZhLnV0aWwuQXJyYXlzLnVzZUxlZ2FjeU1lcmdlU29ydD10cnVlAA==]
Dataset consistency test: No problems found

comment:10 by Klumbumbus, 9 years ago

BTW why is milestone 14.12 and not 14.11?

comment:11 by stoecker, 9 years ago

In 7696/josm:

Fix scaling, see #10684

comment:12 by stoecker, 9 years ago

One place used setMaxSize(), the other used setSize(). Seems my choice was the wrong one. Fixed.

BTW why is milestone 14.12 and not 14.11?

Because it wont be finished in 14.11 :-)

in reply to:  12 ; comment:13 by Don-vip, 9 years ago

Replying to stoecker:

Because it wont be finished in 14.11 :-)

is there so much work? We still have 4 entire weeks, it's the longest development period of this year as I pushed October release sooner :)

in reply to:  13 ; comment:14 by stoecker, 9 years ago

Replying to Don-vip:

is there so much work? We still have 4 entire weeks, it's the longest development period of this year as I pushed October release sooner :)

  • Fix overlay handling (see comment in one of the last checkins)
  • Find and replace all hardcoded resizings
  • At least ImageSizes.LIST type for layer and other lists is missing as 16x16, probably more types
  • Add ImageProvider passthrough for styles and probably other places (or maybe ImageResource - I'm not sure yet if my new function for JosmAction should use ImageResource instead).
  • test and check plugins for incompatibilities.

And all this together with the fact that I have probably no more time as of tomorrow. If this was my work then probably 2 days only. :-)

in reply to:  14 comment:15 by Don-vip, 9 years ago

Replying to stoecker:

  • test and check plugins for incompatibilities.

I have updated some of them in [o30806:30811].

comment:16 by Don-vip, 9 years ago

In 7731/josm:

see #10684 - fix javadoc

comment:17 by Don-vip, 9 years ago

In 7746/josm:

see #10684 - fix deprecation warning

comment:18 by Don-vip, 9 years ago

Milestone: 14.1215.01

comment:19 by Don-vip, 9 years ago

Milestone: 15.0115.02

move tickets that have not been treated this month to next milestone

comment:20 by stoecker, 9 years ago

In 8095/josm:

see #10684 - add proper overlay interface for ImageProvider()

comment:21 by stoecker, 9 years ago

In 8097/josm:

see #10684, see #10688 - fix image scaling for mappaint

comment:22 by stoecker, 9 years ago

Ticket #10688 has been marked as a duplicate of this ticket.

comment:23 by skyper, 9 years ago

Cc: skyper added
Keywords: icon scale added

comment:24 by stoecker, 9 years ago

Milestone: 15.0215.03

Still a lot to do.

comment:25 by stoecker, 9 years ago

Milestone: 15.0315.04

Move one month.

comment:26 by stoecker, 9 years ago

Milestone: 15.0415.05

comment:27 by stoecker, 9 years ago

In 8323/josm:

see #10684 - remove remaining overlay() calls

comment:28 by Don-vip, 9 years ago

Milestone: 15.0515.06

comment:29 by Don-vip, 9 years ago

Milestone: 15.0615.07

skip milestone 15.06

comment:30 by Don-vip, 9 years ago

Milestone: 15.0715.08

Milestone renamed

comment:31 by Don-vip, 9 years ago

Milestone: 15.08

comment:32 by stoecker, 8 years ago

In 9251/josm:

see #10684 - extend SVG with small dimension when a larger image is requested

comment:33 by Don-vip, 8 years ago

Can you please enable Checkstyle in your IDE? Since r9235 it should work without any particular configuration.

comment:34 by stoecker, 8 years ago

I think in principle this ticket is done. Remaining task would be to search the core and plugin code and fix all places, where hardcoded image sizes are used.

Can you please enable Checkstyle in your IDE? Since r9235 it should work without any particular configuration.

That's impossible. I use no IDE :-)

in reply to:  34 ; comment:35 by Don-vip, 8 years ago

Replying to stoecker:

That's impossible. I use no IDE :-)

not an excuse ;)

comment:36 by Don-vip, 8 years ago

Milestone: 16.01

in reply to:  35 comment:37 by stoecker, 8 years ago

Milestone: 16.01

Replying to Don-vip:

Replying to stoecker:

That's impossible. I use no IDE :-)

not an excuse ;)

Baah. Vi. That's only the last resort when nothing else is available. That's my nerd check: Someone who can use vi is a nerd. Sadly I fall in that group.

comment:38 by Don-vip, 8 years ago

In 9252/josm:

see #10684 - update to checkstyle 6.14.1

comment:39 by Don-vip, 8 years ago

Resolution: fixed
Status: newclosed

comment:40 by stoecker, 8 years ago

In 9253/josm:

see #10684 - use squared toolbar buttons always, move side button image size definition to ImageProvider

comment:41 by Klumbumbus, 8 years ago

In 9275/josm:

see #10684 - use svg logo for scaling in toolbar (since r9251)

comment:42 by Klumbumbus, 8 years ago

was the milestone accidentally removed in comment:37?

Last edited 8 years ago by Klumbumbus (previous) (diff)

in reply to:  32 ; comment:43 by Klumbumbus, 8 years ago

Replying to stoecker:

In 9251/josm:

see #10684 - extend SVG with small dimension when a larger image is requested

While some icons do upscale in the toolbar, some don't. See screenshot


Food preset icon, Show ID mappaint style icon and Layer Checker mappaint style icon. (The other small icons are png)

Last edited 8 years ago by Klumbumbus (previous) (diff)

by Klumbumbus, 8 years ago

Attachment: upscaling.png added

in reply to:  42 comment:44 by stoecker, 8 years ago

Milestone: 16.01

Replying to Klumbumbus:

was the milestone accidentally removed in comment:37?

Yes. Parallel editing...

in reply to:  43 comment:45 by stoecker, 8 years ago

Replying to Klumbumbus:

Food preset icon, Show ID mappaint style icon and Layer Checker mappaint style icon. (The other small icons are png)

Are the small SVG toolbar entries all styles?

comment:46 by Klumbumbus, 8 years ago

The food Icon is from internal preset

comment:47 by stoecker, 8 years ago

Resolution: fixed
Status: closedreopened

Works with the uncleaned SVN icons, but not with cleaned ones in the jar. Strange.

comment:48 by Don-vip, 8 years ago

Maybe related: see problems encountered in #12201 and how [o31865] fixed them.

comment:49 by stoecker, 8 years ago

Resolution: fixed
Status: reopenedclosed

In 9281/josm:

fix #10684 - fix SVG upscaling

comment:50 by stoecker, 8 years ago

In 10356/josm:

see #9995, see #10684 - remove more hardcoded places of images

comment:51 by stoecker, 8 years ago

In 10357/josm:

see #9995, see #10684 - remove more hardcoded places of images

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.