Modify

Opened 12 years ago

Closed 3 years ago

Last modified 3 years ago

#7548 closed enhancement (fixed)

Re-organize the preference dialog

Reported by: simon04 Owned by: team
Priority: major Milestone: 20.11
Component: Core Version:
Keywords: Cc: Klumbumbus, openstreetmap.org-user-d1g

Description

To me, the current preference dialog is not very intuitive. Especially, Often, I click through several panes to find the desired one.

I would like to make the following improvement suggestions:

  • Instead of icons use (additionally) textual labels for the buttons on the left hand side. This allows IMO to find the desired one more quickly.
  • Reorganize all panes in categories and sub-categories as follows:
  • Mapping
    • OSM Data
    • GPS Points
    • Map Projection
    • Map Paint Styles
    • Tagging Presets
    • Imagery
  • Servers
    • Authentication
    • Proxy settings
  • UI/JOSM
    • Colors
    • Look and Feel
    • File backup
    • Toolbar
    • Keyboard Shortcuts
  • Advanced
    • Audio
    • Validator
    • Remote control
    • Advanced Preferences
  • Plugins
    • Install/update
    • Update policy
    • … (custom panes of plugins)

I'm happy to hear some opinions :-)

Attachments (25)

7548.png (33.5 KB ) - added by simon04 12 years ago.
7548_alpha.patch (48.2 KB ) - added by simon04 12 years ago.
7548_v02.patch (47.8 KB ) - added by simon04 12 years ago.
phpyWYECZ (44.9 KB ) - added by bastiK 12 years ago.
updated patch to r5506
7548_2014.patch (81.1 KB ) - added by simon04 10 years ago.
7548_2014_v2.patch (85.3 KB ) - added by simon04 10 years ago.
7548_plugins.patch (4.8 KB ) - added by simon04 10 years ago.
7548_2014_v3.patch (85.5 KB ) - added by simon04 10 years ago.
2020-10-11-102936_852x756_scrot.png (42.3 KB ) - added by simon04 3 years ago.
2020-10-11-112249_898x956_scrot.png (40.5 KB ) - added by simon04 3 years ago.
2020-10-11-114632_898x956_scrot.png (57.5 KB ) - added by simon04 3 years ago.
2020-10-11-123753_898x956_scrot.png (41.2 KB ) - added by simon04 3 years ago.
2020-10-11-124515_898x956_scrot.png (46.9 KB ) - added by simon04 3 years ago.
2020-10-11-192837_598x742_scrot.png (28.8 KB ) - added by simon04 3 years ago.
7548-init.patch (848 bytes ) - added by GerdP 3 years ago.
prefs-with-search.png (30.6 KB ) - added by GerdP 3 years ago.
display.png (41.2 KB ) - added by Don-vip 3 years ago.
Screenshot 2020-10-21 at 19.13.22.png (154.7 KB ) - added by Stereo 3 years ago.
The preferences panel also looks very very weird on macOS in Java 15.
7548-no-empty.patch (2.4 KB ) - added by GerdP 3 years ago.
@Klumbumbus Please try this patch. It solves the problem on both my PC and the laptop
Screenshot 2020-10-26 at 14.15.28.png (153.0 KB ) - added by Stereo 3 years ago.
17265 macos
Screenshot 2020-11-03 at 10.16.45.png (269.0 KB ) - added by Stereo 3 years ago.
17292 macOS
Captura de pantalla 2020-11-03 a las 11.35.54.png (96.2 KB ) - added by sanchi 3 years ago.
Screenshot 2020-11-03 at 16.28.10.png (266.5 KB ) - added by Stereo 3 years ago.
Metal 17292 macos
Screenshot 2020-11-03 at 18.00.08.png (336.7 KB ) - added by Stereo 3 years ago.
17291 Metal macOS
EmptyPrefs.png (44.0 KB ) - added by mdk 3 years ago.

Change History (193)

by simon04, 12 years ago

Attachment: 7548.png added

comment:1 by simon04, 12 years ago


by simon04, 12 years ago

Attachment: 7548_alpha.patch added

comment:2 by simon04, 12 years ago

Summary: Re-organize the preference dialog[alpha Patch] Re-organize the preference dialog
Type: defectenhancement

comment:3 by akks, 12 years ago

It looks really good, but not too easy :)

Other possible problem: are you sure it will fit on 800x600 width or 1024x768 with enlarged font, like in
http://josm.openstreetmap.de/ticket/7450#comment:40 ?

Maybe we need some shorter TODO list for non-experts?
Main reasons to change preferences for them are

  • specify OSM account, maybe with OpenID
  • specify proxy, if it exists
  • install needed plugins (recommended on forums, etc.)
  • configure imagery sources (choose specific for your region)
  • sometimes - remote control, beacuse online tools need it

On the other hand, we can not hide all other preferences or do reordering for non-experts only...

Last edited 12 years ago by akks (previous) (diff)

in reply to:  3 ; comment:4 by simon04, 12 years ago

Replying to akks:

It looks really good, but not too easy :)

Actually, the required changes have been the same for each preference tab (except for some).

Other possible problem: are you sure it will fit on 800x600 width or 1024x768 with enlarged font, like in
http://josm.openstreetmap.de/ticket/7450#comment:40 ?

Currently, (vertical and horizontal) scrollbars are shown on the right pane whenever necessary. However, this happens also in unexpected cases at the moment (e.g., instead of horizontally scrolling the tagging presets lists themselves, an outer scrollbar is shown).

Maybe we need some shorter TODO list for non-experts?
Main reasons to change preferences for them are

  • specify OSM account, maybe with OpenID
  • specify proxy, if it exists
  • install needed plugins (recommended on forums, etc.)
  • configure imagery sources (choose specific for your region)
  • sometimes - remote control, beacuse online tools need it

On the other hand, we can not hide all other preferences or do reordering for non-experts only...

This would suggest to have something like a quick setup wizard. I guess, the OSM account can be skipped as this is requested anyway when uploading changes.

Concerning plugins, custom styles, custom presets and imagery sources, it would be great to somehow have "bundled versions" which installs the things automatically.

in reply to:  4 comment:5 by akks, 12 years ago

Replying to simon04:

This would suggest to have something like a quick setup wizard. I guess, the OSM account can be skipped as this is requested anyway when uploading changes.

Yes, some quick setup wizard would be good. Maybe something with highlighting most important preferences tabs and showing tooltips on them could help too...

Concerning plugins, custom styles, custom presets and imagery sources, it would be great to somehow have "bundled versions" which installs the things automatically.

Currently this can be already done by custom configurations, (#4421)
For example, I have settings for drawing forests in Russia: http://josm.openstreetmap.de/attachment/ticket/4421/forestMapping.xml

Last edited 12 years ago by akks (previous) (diff)

by simon04, 12 years ago

Attachment: 7548_v02.patch added

comment:6 by simon04, 12 years ago

I resolved some merging conflicts due to code changes. In addition, the horizontal scrolling problem as described above has been resolved by not using the ScrollPane for the corresponding preferences.

comment:7 by bastiK, 12 years ago

Some comments:

  • "File backup" can be moved to the Advanced section.
  • Instead of JTabbedPane you could use JList or JTree on the left and CardLayout on the right: When there is not enough vertical space for all the entries on the left, the scrolling works better for a list. By using one JList for each category, you can add the large icons as labels in between, so they aren't clickable any more.
  • It would be nice to have new icons for "Mapping" and "User Interface".

comment:8 by simon04, 12 years ago

Thank you for your comments, bastiK!

  • Good point concerning "File backup"
  • Concerning icons, we might find something on http://thenounproject.com/noun/earth/#icon-No1238
  • What would be the advantage of the CardLayout?
  • The big icons in the current solution could display some help text for the corresponding settings (just as an idea).

in reply to:  8 comment:9 by bastiK, 12 years ago

Replying to simon04:

  • What would be the advantage of the CardLayout?
  • The big icons in the current solution could display some help text for the corresponding settings (just as an idea).

It is a little unconventional to have two kinds of tabs (icon and text). With CardLayout you can simulate a JTabbedPane, and use your own GUI to switch the content of the main panel.

comment:10 by simon04, 12 years ago

JOSM (is|might be) unconventional in other aspects as well. :-)

IMO, this sounds like (a lot?) effort for mostly aesthetic things. ATM, I'm not really up to those changes, are you? ;-)

comment:11 by stoecker, 12 years ago

What's the status of this? Something for a hidden option?

comment:12 by simon04, 12 years ago

This patch might need some adaption due to r5465. Having a hidden option might be difficult (additional work), as I had to restructure some preference panels to fit the new idea. We might want to add this change right at the beginning of a "free development" phase.

Dirk, what do you think?

comment:13 by bastiK, 12 years ago

Hidden option wouldn't do much good and is too much work to implement.

by bastiK, 12 years ago

Attachment: phpyWYECZ added

updated patch to r5506

by simon04, 10 years ago

Attachment: 7548_2014.patch added

comment:14 by simon04, 10 years ago

Summary: [alpha Patch] Re-organize the preference dialog[β Patch] Re-organize the preference dialog

Updated patch to current revision, quite some merging conflicts ;-). Also improved some messages, dialogs. attachment:7548_2014.patch

Should we give it a try? What do you think?

comment:15 by Don-vip, 10 years ago

I give it a look after I finish #9984 :)

comment:16 by Don-vip, 10 years ago

It's really nice I love it :) I vote in favour of this new layout. But I have encountered a strange bug when clicking in Connection Settings:

Revision: 7133
Is-Local-Build: true
Build-Date: 2014-05-15 23:48:37

Identification: JOSM/1.5 (7133 SVN en) Windows 8.1 64-Bit
Memory Usage: 79 MB / 2731 MB (37 MB allocated, but free)
Java version: 1.8.0_05, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
VM arguments: [-Xmx3G, -Dfile.encoding=UTF-8]
Program arguments: [--debug]

java.lang.NoClassDefFoundError: sun/awt/VerticalBagLayout
	at org.openstreetmap.josm.gui.preferences.server.ServerAccessPreference.buildContentPanel(ServerAccessPreference.java:42)
	at org.openstreetmap.josm.gui.preferences.server.ServerAccessPreference.addGui(ServerAccessPreference.java:56)
	at org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane.stateChanged(PreferenceTabbedPane.java:593)
	at javax.swing.DefaultSingleSelectionModel.fireStateChanged(DefaultSingleSelectionModel.java:132)
	at javax.swing.DefaultSingleSelectionModel.setSelectedIndex(DefaultSingleSelectionModel.java:67)
	at javax.swing.JTabbedPane.setSelectedIndexImpl(JTabbedPane.java:616)
	at javax.swing.JTabbedPane.setSelectedIndex(JTabbedPane.java:591)
	at javax.swing.plaf.basic.BasicTabbedPaneUI$Handler.mousePressed(BasicTabbedPaneUI.java:3645)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:279)
	at java.awt.Component.processMouseEvent(Component.java:6524)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3321)
	at java.awt.Component.processEvent(Component.java:6292)
	at java.awt.Container.processEvent(Container.java:2234)
	at java.awt.Component.dispatchEventImpl(Component.java:4883)
	at java.awt.Container.dispatchEventImpl(Container.java:2292)
	at java.awt.Component.dispatchEvent(Component.java:4705)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4898)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4530)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4462)
	at java.awt.Container.dispatchEventImpl(Container.java:2278)
	at java.awt.Window.dispatchEventImpl(Window.java:2739)
	at java.awt.Component.dispatchEvent(Component.java:4705)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:746)
	at java.awt.EventQueue.access$400(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:697)
	at java.awt.EventQueue$3.run(EventQueue.java:691)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.awt.EventQueue$4.run(EventQueue.java:719)
	at java.awt.EventQueue$4.run(EventQueue.java:717)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:716)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:109)
	at java.awt.WaitDispatchSupport$2.run(WaitDispatchSupport.java:184)
	at java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:229)
	at java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:227)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.WaitDispatchSupport.enter(WaitDispatchSupport.java:227)
	at java.awt.Dialog.show(Dialog.java:1084)
	at java.awt.Component.show(Component.java:1656)
	at java.awt.Component.setVisible(Component.java:1608)
	at java.awt.Window.setVisible(Window.java:1014)
	at java.awt.Dialog.setVisible(Dialog.java:1005)
	at org.openstreetmap.josm.gui.preferences.PreferenceDialog.setVisible(PreferenceDialog.java:126)
	at org.openstreetmap.josm.actions.PreferencesAction.run(PreferencesAction.java:112)
	at org.openstreetmap.josm.actions.PreferencesAction.actionPerformed(PreferencesAction.java:101)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2346)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
	at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:289)
	at java.awt.Component.processMouseEvent(Component.java:6527)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3321)
	at java.awt.Component.processEvent(Component.java:6292)
	at java.awt.Container.processEvent(Container.java:2234)
	at java.awt.Component.dispatchEventImpl(Component.java:4883)
	at java.awt.Container.dispatchEventImpl(Container.java:2292)
	at java.awt.Component.dispatchEvent(Component.java:4705)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4898)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4533)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4462)
	at java.awt.Container.dispatchEventImpl(Container.java:2278)
	at java.awt.Window.dispatchEventImpl(Window.java:2739)
	at java.awt.Component.dispatchEvent(Component.java:4705)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:746)
	at java.awt.EventQueue.access$400(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:697)
	at java.awt.EventQueue$3.run(EventQueue.java:691)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.awt.EventQueue$4.run(EventQueue.java:719)
	at java.awt.EventQueue$4.run(EventQueue.java:717)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:716)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)
Caused by: java.lang.ClassNotFoundException: sun.awt.VerticalBagLayout
	at java.net.URLClassLoader$1.run(URLClassLoader.java:372)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:361)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(URLClassLoader.java:360)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	... 87 more

comment:17 by Don-vip, 10 years ago

Also, the direct access from validator dialog to validator preferences (through the small icon left dock/undock icon) does not work anymore.

in reply to:  17 comment:18 by skyper, 10 years ago

Replying to Don-vip:

Also, the direct access from validator dialog to validator preferences (through the small icon left dock/undock icon) does not work anymore.

How about the mappaint toggle dialog. Does this link work ?

comment:19 by Don-vip, 10 years ago

Don't think so, it's the same code :)

by simon04, 10 years ago

Attachment: 7548_2014_v2.patch added

comment:20 by simon04, 10 years ago

Thank you for your feedback!

In attachment:7548_2014_v2.patch​, comment:16 and comment:17 have been resolved (comment:18 wasn't a problem).
(Concerning comment:16: In the old days I tried to come around GridBagLayout by trying other managers )

comment:21 by Don-vip, 10 years ago

Much better :) Still another problem with direct access:

  1. setup a nonexistent proxy and erase the MOTD in user directory
  2. restart JOSM, there's an error message with a "change proxy settings" button
  3. This button leads to connection tab, not the proxy one

comment:22 by Don-vip, 10 years ago

Also the 2 methods PreferencesAction.forPreferenceSubTab should be removed, no?

comment:23 by simon04, 10 years ago

Milestone: 14.06

Right, thank you – patch adapted. Some plugins need also to be adapted – patch attached.

Let's keep this for the subsequent stable release to have enough time for finding more bugs …

by simon04, 10 years ago

Attachment: 7548_plugins.patch added

by simon04, 10 years ago

Attachment: 7548_2014_v3.patch added

comment:24 by Don-vip, 10 years ago

Time to give it a try? :)

comment:25 by Don-vip, 10 years ago

Milestone: 14.0614.07

Move all tickets for which no work has been done yet to next milestone

comment:26 by Don-vip, 10 years ago

The patch may need an update after r7299.

comment:27 by Don-vip, 10 years ago

Milestone: 14.0714.08

Move some tickets to next milestone

comment:28 by Don-vip, 10 years ago

Milestone: 14.08

comment:29 by Klumbumbus, 10 years ago

Cc: Klumbumbus added

comment:30 by openstreetmap.org-user-d1g, 8 years ago

Yes, text labels are far better than only icons in tabs.

Compared to Firefox
In the Firefeox we have "file", "edit" "view" and so on.
If you open "about:preferences" there would be "General", "Search", "Content", "Applications", "Privacy", "Security", "Sync", "Advanced"
There no icons in the "main menu", icons in about:preferences is alongside to ordinary text labels.

Any reason why it wasn't merged? Nobody had time for it?

comment:31 by openstreetmap.org-user-d1g, 8 years ago

Cc: penstreetmap.org-user-d1g added

comment:32 by openstreetmap.org-user-d1g, 8 years ago

Cc: openstreetmap.org-user-d1g added; penstreetmap.org-user-d1g removed

comment:33 by openstreetmap.org-user-d1g, 8 years ago

Please don't use centrer aligned text in the horizontal tabs (on the left). Is is harder to find beginning of the each word.

Firefox/Chromium aligns text to the left.

Last edited 8 years ago by openstreetmap.org-user-d1g (previous) (diff)

comment:34 by skyper, 4 years ago

Was this patch forgotten or does it simply not work?

+1 for changing.

comment:35 by simon04, 4 years ago

Milestone: 20.10

It was ready, for forgotten twice after a stabilization phase. Now it needs a big overhaul due to various merge conflicts. I can try to update it again.

(Btw: I started working on this idea on the train to FOSSGIS 2012…)

in reply to:  35 comment:36 by skyper, 4 years ago

Replying to simon04:

It was ready, for forgotten twice after a stabilization phase. Now it needs a big overhaul due to various merge conflicts. I can try to update it again.

(Btw: I started working on this idea on the train to FOSSGIS 2012…)

Where is the FOSSGIS 2021? Guess to far away in time to wait for a release of the enhancement on the train to the conference.

comment:37 by simon04, 4 years ago

2021: Rapperswil, CH (≈4h)

in reply to:  37 comment:38 by skyper, 4 years ago

Replying to simon04:

2021: Rapperswil, CH (≈4h)

Oh, forgot about it. Think we talked about it already in March. See you there. Greetings from Vorder-Östrereich.

comment:39 by Klumbumbus, 4 years ago

I would prefer to have small icons for the tabs too, not just text. We already have icons for most of the entries and we can find/create the rest for sure.
Also when adding text to the tabs you should check if it still works with the macos LaF as the tabs are different there, see #15570 and #14399.

comment:40 by simon04, 3 years ago

Owner: changed from team to simon04
Status: newassigned

comment:41 by simon04, 3 years ago

In 17159/josm:

see #7548 - Re-organize the preference dialog (display preferences)

by simon04, 3 years ago

comment:42 by GerdP, 3 years ago

The tab "Display Settings" is now empty.

comment:43 by simon04, 3 years ago

In 17160/josm:

see #7548 - Re-organize the preference dialog (connection preferences)

by simon04, 3 years ago

comment:44 by simon04, 3 years ago

In 17161/josm:

see #7548 - Re-organize the preference dialog (map preferences)

by simon04, 3 years ago

comment:45 by simon04, 3 years ago

In 17162/josm:

see #7548 - Re-organize the preference dialog (connection preferences)

by simon04, 3 years ago

by simon04, 3 years ago

comment:47 by GerdP, 3 years ago

I don't use any of the mentioned plugins. Does this explain the tab is empty?

comment:48 by skyper, 3 years ago

As the list on the left can get quite long now, especially with some plugins installed, are there any plans to have an option to fold all tab entries but the one in the active section?

in reply to:  47 ; comment:49 by simon04, 3 years ago

Replying to GerdP:

I don't use any of the mentioned plugins. Does this explain the tab is empty?

Yes.

Replying to skyper:

are there any plans to have an option to fold all tab entries but the one in the active section?

No, as this would make the whole reorganisation pointless (all entries should be directly accessible).

The icons could be smaller, such as:


Last edited 3 years ago by simon04 (previous) (diff)

by simon04, 3 years ago

comment:50 by Don-vip, 3 years ago

Summary: [β Patch] Re-organize the preference dialogRe-organize the preference dialog

in reply to:  49 comment:51 by skyper, 3 years ago

Replying to simon04:

Replying to skyper:

are there any plans to have an option to fold all tab entries but the one in the active section?

No, as this would make the whole reorganisation pointless (all entries should be directly accessible).

The icons could be smaller, such as:

Better, maybe, even centered for the group.

comment:52 by Klumbumbus, 3 years ago

I would go with:

  • an own 24px icon for every line on the left
  • left aligned text
  • no empty tabs

comment:53 by simon04, 3 years ago

In 17178/josm:

see #7548 - Re-organize the preference dialog (icons)

in reply to:  52 ; comment:54 by simon04, 3 years ago

Replying to Klumbumbus:

  • left aligned text

Not so easy, see https://stackoverflow.com/questions/26308859/jtabbedpane-tab-placement-set-to-left-but-icons-are-not-aligned for a rather complex solution. I'll look into that in depth later…

comment:55 by simon04, 3 years ago

In 17179/josm:

see #7548 - Re-organize the preference dialog (fix unit tests)

by GerdP, 3 years ago

Attachment: 7548-init.patch added

comment:56 by GerdP, 3 years ago

@Simon: During my work on #19876 I noticed that the DisplayPreferences tab was always initialized. Reason is that the ChangeListener reacts (too early) when javax.swing.JTabbedPane.insertTab(String title, Icon icon, Component component, String tip, int index) is called first time and this code is executed:

        if (pages.size() == 1) {
            setSelectedIndex(0);
        }

With the small patch the listener is added later and thus DisplayPreferences is not (wrongly) added to list settingsInitialized.

in reply to:  56 comment:57 by simon04, 3 years ago

Replying to GerdP:

@Simon: During my work on #19876 I noticed that the DisplayPreferences tab was always initialized.

Thanks for noticing. Please go ahead and commit your patch! :-)

comment:58 by GerdP, 3 years ago

In 17201/josm:

see #7548: Re-organize the preference dialog
Add addChangeListener later (7548-init.patch)

comment:59 by GerdP, 3 years ago

Just to make sure: I still don't see the content of the DisplayPreferences tab.

in reply to:  59 comment:60 by skyper, 3 years ago

Replying to GerdP:

Just to make sure: I still don't see the content of the DisplayPreferences tab.

Is there any content to be displayed? Think this and Map Settings are leftovers and only serve as titles right now.

comment:61 by GerdP, 3 years ago

Ahh! Very confusing, at least for me.
Why not remove the empty tabs? I would alse move the "language" tab into the "look and feel" panel.

in reply to:  61 comment:62 by skyper, 3 years ago

Replying to GerdP:

Ahh! Very confusing, at least for me.
Why not remove the empty tabs? I would alse move the "language" tab into the "look and feel" panel.

+1

The OSM-Server and Proxy settings tabs need a vertical scroll bar, if the content is not fully displayed.

in reply to:  61 comment:63 by Klumbumbus, 3 years ago

Replying to GerdP:

Why not remove the empty tabs?

See comment:46 for the reason and a possible solution.

in reply to:  54 comment:64 by Klumbumbus, 3 years ago

Replying to simon04:

Replying to Klumbumbus:

  • left aligned text

Not so easy, see https://stackoverflow.com/questions/26308859/jtabbedpane-tab-placement-set-to-left-but-icons-are-not-aligned for a rather complex solution. I'll look into that in depth later…

tbh if we can't find a solution for the left alignment we should revert this reorganization because the centered text really looks ugly and doesn't help at all to find the wanted tab faster. Or we move away from these tabs and use something different?
Some brainstorming (i.e. some other apps with icons in preferences):

by GerdP, 3 years ago

Attachment: prefs-with-search.png added

comment:65 by GerdP, 3 years ago

tbh if we can't find a solution for the left alignment we should revert this reorganization because the centered text really looks ugly and doesn't help at all to find the wanted tab faster.

+1

Or we move away from these tabs and use something different?

I think all preference dialogs have the same problem for newbes. Situation is this: User doesn't like a default behaviour and wonders if there is an option to change that. As long as they don't know the internal structure of the application they can only guess where such an option could be hidden. At that point in time the icons don't help me at all. Typically I look at the texts only at that time and maybe I remember the icon to find that setting faster next time.

In fact I prefer to use the filter in the expert preferences. It's like a very simple Google search.
The System settings in MS Windows were changed often and - I think - never to the better. Their best idea was to add the search function which simply lists possible links. Since that exists I don't have to care about the organisation.

comment:66 by simon04, 3 years ago

In 17227/josm:

see #7548 - Re-organize the preference dialog (remove "settings" from tab names)

comment:67 by simon04, 3 years ago

In 17228/josm:

see #7548 - Re-organize the preference dialog (left-align tab labels)

comment:68 by simon04, 3 years ago

In 17229/josm:

see #7548 - Re-organize the preference dialog (hide empty tabs)

comment:69 by simon04, 3 years ago

In 17231/josm:

see #7548 - Re-organize the preference dialog (add various icons)

Source of the language icon: https://github.com/FortAwesome/Font-Awesome/blob/5.15.1/svgs/solid/language.svg (CC BY 4.0)

in reply to:  69 comment:70 by skyper, 3 years ago

Replying to simon04:

In 17231/josm:

see #7548 - Re-organize the preference dialog (add various icons)

Source of the language icon: https://github.com/FortAwesome/Font-Awesome/blob/5.15.1/svgs/solid/language.svg (CC BY 4.0)

Few icon paths seem to be wrong. Thought we do not use file extensions and a regression in #19964.

comment:71 by GerdP, 3 years ago

I tried r17237 , press F12 directly after start and the display for OSM Data is still empty. I have to click on another tab and then again on the first to get its content.

comment:72 by skyper, 3 years ago

Yeap, same for me.

Is the file extension needed?

comment:73 by Klumbumbus, 3 years ago

In 17245/josm:

  • see #7548 - Change/add icons, remove file extension
  • ignore landform=dune_system

in reply to:  67 comment:74 by Klumbumbus, 3 years ago

Replying to simon04:

In 17228/josm:

see #7548 - Re-organize the preference dialog (left-align tab labels)

Great.

comment:75 by Klumbumbus, 3 years ago

I think we should

  • completeley remove the "Map" tab (MapPreference.java) as it is never displayed
  • rename the "Display" tab (DisplayPreference.java) to "Plugins", change its description and move the tab down after "Imagery".
Last edited 3 years ago by Klumbumbus (previous) (diff)

in reply to:  75 ; comment:76 by Klumbumbus, 3 years ago

Replying to Klumbumbus:

  • rename the "Display" tab (DisplayPreference.java) to "Plugins"

Ah, no we already have a "Plugins" tab. So maybe "Plugins settings"? Or we change them to own tabs, not subtabs?

by Don-vip, 3 years ago

Attachment: display.png added

comment:77 by Don-vip, 3 years ago

I'm looking at #19968 and the colorscheme plugin preferences go into the "display" tab, this is weird:


in reply to:  77 comment:78 by Klumbumbus, 3 years ago

Replying to Don-vip:

I'm looking at #19968 and the colorscheme plugin preferences go into the "display" tab, this is weird:

"Colors" was previously a subtag of "Display" thats why "Color schemes" was a subtag of "Display" too. Now "Color schemes" makes more sense as a subtab of "Colors".

comment:79 by Don-vip, 3 years ago

@Simon ColorPreference.getTabPane() returns null so I can't add color schemes tab there. How do you want to address this?

by Stereo, 3 years ago

The preferences panel also looks very very weird on macOS in Java 15.

comment:80 by Stereo, 3 years ago

The preferences panel also looks very very weird on macOS in Java 15.

The preferences panel looking very very weird on macOS in Java 15.

Last edited 3 years ago by Stereo (previous) (diff)

comment:81 by Don-vip, 3 years ago

The Aqua L&F doesn't allow to get horizontal tabs. The time has probably come to move away from a tab-based panel to a list or tree-based one. We should also add a filter input text at the top allowing us to search for a preference by its name, like in many other applications.

comment:82 by GerdP, 3 years ago

In 17256/josm:

see #7548: Re-organize the preference dialog

  • Use setSelectedIndex(-1) to activate the filling of the first display

in reply to:  81 ; comment:83 by simon04, 3 years ago

Replying to Don-vip:

@Simon ColorPreference.getTabPane() returns null so I can't add color schemes tab there. How do you want to address this?

ColorPreference isn't instantiating a JTabbedPane in its constructor –

  • src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java

    diff --git a/src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java b/src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java
    index 45196fb3b..6ac830a4a 100644
    a b public PreferenceSetting createPreferenceSetting() {  
    8182
    8283    ColorPreference() {
    8384        super(/* ICON(preferences/) */ "color",
    84                 tr("Colors"), tr("Change colors used in program dialogs and in map paint styles."));
     85                tr("Colors"), tr("Change colors used in program dialogs and in map paint styles."), false, new JTabbedPane());
    8586    }
    8687
    8788    private ColorTableModel tableModel;

comment:84 by Klumbumbus, 3 years ago

There is a complie warning:

compile:
    [javac] Compiling 1633 source files to C:\Users\stefa\Documents\OSM\josm\core\build
    [javac] C:\Users\stefa\Documents\OSM\josm\core\src\org\openstreetmap\josm\gui\preferences\server\ProxyPreferencesPanel.java:182: warning: [UnusedVariable] The local variable 'gc' is never read.
    [javac]         GridBagConstraints gc = new GridBagConstraints();
    [javac]                            ^
    [javac]     (see https://errorprone.info/bugpattern/UnusedVariable)
    [javac]   Did you mean to remove this line or 'new GridBagConstraints();'?
    [javac] 1 warning

in reply to:  82 comment:85 by Klumbumbus, 3 years ago

Replying to GerdP:

In 17256/josm:

see #7548: Re-organize the preference dialog

  • Use setSelectedIndex(-1) to activate the filling of the first display

Now the Display tab is completely blank when opening the preferences.

comment:86 by GerdP, 3 years ago

In 17264/josm:

see #7548: Re-organize the preference dialog

  • revert r17256, doesn't help and maybe worsens the problem on some systems

comment:87 by GerdP, 3 years ago

Strange, I think it fixed the problem on my PC, but on my Laptop it is still there :(
So, I've revert it. On my Laptop this doesn't change anything.

comment:88 by GerdP, 3 years ago

On my machine it happens when F12 is pressed for the first time. If you chose another tab and then the first again the problem is gone (for that JOSM session). If you just chose another tab this is also opened again with that content.

in reply to:  88 comment:89 by Klumbumbus, 3 years ago

Replying to GerdP:

If you chose another tab and then the first again the problem is gone (for that JOSM session).

Yes that worked for me too in both versions, but is a bad workaround.

comment:90 by GerdP, 3 years ago

On my PC this work around doesn't work. Seems we have a race condition.

by GerdP, 3 years ago

Attachment: 7548-no-empty.patch added

@Klumbumbus Please try this patch. It solves the problem on both my PC and the laptop

comment:91 by Klumbumbus, 3 years ago

Yes that works fine for me.

comment:92 by GerdP, 3 years ago

In 17265/josm:

see #7548: Re-organize the preference dialog
Apply 7548-no-empty.patch to avoid empty top panel

  • make sure that a panel is selected when selectPreviouslySelectedPreferences() is called and nothing was previously selected
  • reverts r17097

in reply to:  83 comment:93 by Don-vip, 3 years ago

Replying to simon04:

Replying to Don-vip:

@Simon ColorPreference.getTabPane() returns null so I can't add color schemes tab there. How do you want to address this?

ColorPreference isn't instantiating a JTabbedPane in its constructor –

This isn't enough to make it being displayed. I think all preferences should define a tab pane. Those who currently have no tab pane should define their contents in the first tab named the same as preferences. Make the tab pane hidden if it contains only one tab, so that visually there is no difference between the current system and the new one, except this is expendable: all plugins can define new tabs everywhere, not only in the preferences where tabs are defined by JOSM core.

by Stereo, 3 years ago

17265 macos

comment:94 by Stereo, 3 years ago

17265 macos

It still looks unusable on macOS in 17265, Java 16. For some reason, opening the preferences gets me the Mapillary settings, too.

comment:95 by GerdP, 3 years ago

I think the Mapillary dialog is intended, although I think that we should show a core dialog if nothing special was selected. OSM data would be my preference.

comment:96 by Don-vip, 3 years ago

In 17271/josm:

see #7548 - remove unused variable

comment:97 by Klumbumbus, 3 years ago

In 17281/josm:

in reply to:  76 comment:98 by Klumbumbus, 3 years ago

Replying to Klumbumbus:

Replying to Klumbumbus:

  • rename the "Display" tab (DisplayPreference.java) to "Plugins"

Ah, no we already have a "Plugins" tab. So maybe "Plugins settings"? Or we change them to own tabs, not subtabs?

Or we move the subtabs of plugins from "Display" tab to "Plugins" tab. (Or to other tabs if it fits better like in the case of colorscheme, comment:78) This makes the "Display" tab completeley obsolete, which would be fine.

comment:99 by Stereo, 3 years ago

Let me know if I can help test things for macOS?

comment:100 by Klumbumbus, 3 years ago

I guess wee need to find a completely different solution than tabs as they don't really work on macOS.

comment:101 by Stereo, 3 years ago

For macOS, something like a JTree, or a JList with headers? A JTabbedPane with tabs at the top would be limited to one line.

See also #15570 for how the Preferences dialog was already broken in macOS before.

Last edited 3 years ago by Stereo (previous) (diff)

comment:102 by GerdP, 3 years ago

See ticket:20013#comment:9. Is this a blocker? I fear I can't help here but I have a few more patches ready.

comment:103 by Stereo, 3 years ago

Priority: normalblocker

Yeah, the current state makes the preferences pretty unusable on macOS. I'll mark it as blocker for now, just for clarity.

Last edited 3 years ago by Stereo (previous) (diff)

comment:104 by simon04, 3 years ago

Owner: changed from simon04 to team
Status: assignednew

What's the minimal change required to fix the macOS problem?

Suggestion for platform=macOS: Disable the tab labels and only display the icons.

in reply to:  104 comment:105 by Don-vip, 3 years ago

Replying to simon04:

Suggestion for platform=macOS: Disable the tab labels and only display the icons.

We can give it a try to see if it unblocks the situation.

comment:106 by simon04, 3 years ago

In 17292/josm:

see #7548 - Re-organize the preference dialog (disable tab titles for macOS)

comment:107 by GerdP, 3 years ago

I guess the Mapillary plugin still dominates the settings when installed (comment:94)?

by Stereo, 3 years ago

17292 macOS

comment:108 by Stereo, 3 years ago

Thank you! So in a way, it's back to how it was before.

The icons are very small, and don't explain what's going to happen if I click on them. It is hard to guide users (Click the icon with the two computers... no the other one).

The Display icon shows up twice.

The Mapillary plugin indeed still dominates the settings.

Horizontal tabs at the top couldn't work?

17292 macOS

comment:109 by sanchi, 3 years ago

Mapillary and Streetside dominate.

https://josm.openstreetmap.de/raw-attachment/ticket/7548/Captura%20de%20pantalla%202020-11-03%20a%20las%2011.35.54.png

in reply to:  106 comment:110 by Don-vip, 3 years ago

Replying to simon04:

In 17292/josm:

see #7548 - Re-organize the preference dialog (disable tab titles for macOS)

We should restrict it to Aqua, I guess this works fine with Metal or any other look-and-feel. @Stereo do you confirm?

by Stereo, 3 years ago

Metal 17292 macos

comment:111 by Stereo, 3 years ago

Metal 17292 macos

It still doesn't look very usable in Metal on macOS.

comment:112 by GerdP, 3 years ago

It should better with versions before r17292 (as they should have Text next to icons.

comment:113 by Klumbumbus, 3 years ago

@Stereo, Don-vip meant you should test with macos but with another LookAndFeel than Aqua with a version before 17292. If that works the plan is to disable the text not for macos in general (as r17292 did) but for the Aqua LookAndFeel only.

by Stereo, 3 years ago

17291 Metal macOS

comment:114 by Stereo, 3 years ago

Is this how you thought it would look in 17291?

17291 Metal macOS

comment:115 by GerdP, 3 years ago

Yes, looks similar in Windows.

comment:116 by Klumbumbus, 3 years ago

In 17293/josm:

see #7548 - Move "Look an Feel" combobox to the top

comment:117 by Klumbumbus, 3 years ago

Current summary (I hope I have not forgotten anything):

What should be done before next release:

  • ✓ (r17297) Change the text disable from macos to Aqua LaF only (comment:110)
  • ✓ (r17298) Make the "Look and Feel" tab a subtag of the "Display" tab. Without special plugins it is then the only content in the "Display" tab which is fine. This fixes also these problems:
    • removes twice used icon (comment:108)
    • plugins can keep adding subtabs in the "Display" tab and then it is not the first which is displayed when opening the preferences ("plugin dominates the settings", comment:108, comment:109)

What should be done, but is not a blocker for release:

  • ✓ (r17312) The "Proxy" tab has an ugly high space top
  • ✓ (r17313) Use more space on the right side of the tabs icons (between icon and text), it looks a bit glued together
  • ✓ (r17314) Overhall to allow plugins to add subtabs to every more tabs (comment:83, comment:93)
  • ✓ (r35643/osm) Move the colorscheme plugin subtab from "Display" tab to "Colors" tab (comment:78)
  • Consider if the plugins mapillary and streetside either move their subtabs from "Display" tab to "Plugins" tab (like the plugins wikipedia and pdfimport do) or use own tabs. Subtabs of "Display" tab doesn't fit so well anymore imho.
  • Switch to Jtree, JList or ... to improve the Aqua LaF workaround (comment:81, comment:101, comment:108)
  • Add a searchbox to filter the preferences to find things quicker (comment:81)
  • ✓ (r17315) Check about dead code after restructure, e.g. remove the "Map" tab (MapPreference.java) as it is never displayed (comment:75)
Last edited 3 years ago by Don-vip (previous) (diff)

comment:118 by Stereo, 3 years ago

Sounds ok to me!

comment:119 by GerdP, 3 years ago

reg. Aqua: Is this dialog the only place that looks ugly with Aqua? If not, wouldn't it be better to disallow Aqua for JOSM?

comment:120 by Stereo, 3 years ago

Disabling Aqua would get you screams from the macOS crowd, myself included :)

in reply to:  119 comment:121 by Don-vip, 3 years ago

Replying to GerdP:

wouldn't it be better to disallow Aqua for JOSM?

This would be crazy, this is absolutely not possible.

comment:122 by simon04, 3 years ago

In 17297/josm:

see #7548 - Re-organize the preference dialog (disable tab titles for com.apple.laf.AquaLookAndFeel)

comment:123 by simon04, 3 years ago

In 17298/josm:

see #7548 - Re-organize the preference dialog (Look and Feel as subtab in display preferences)

comment:124 by GerdP, 3 years ago

Looks okay now on Windows (with + without plugins)

comment:125 by GerdP, 3 years ago

I see this message in my log each time when I open the preferences and the "Display" preferences are displayed.

2020-11-06 08:45:13.522 WARNING: Could not load Mapillary brand image!

in reply to:  125 comment:126 by taylor.smock, 3 years ago

Replying to GerdP:

I see this message in my log each time when I open the preferences and the "Display" preferences are displayed.

2020-11-06 08:45:13.522 WARNING: Could not load Mapillary brand image!

This has already been fixed in the Mapillary plugin (see #19969). I haven't made a release yet that fixes it, but the fix is in the source tree.

comment:128 by skyper, 3 years ago

With complete new preferences, when enable/disable expert mode the selection is lost and no content displayed but a blue background.

in reply to:  128 comment:129 by Klumbumbus, 3 years ago

Replying to skyper:

when enable/disable expert mode ... no content displayed

Yes, that should be fixed before next release.

comment:130 by GerdP, 3 years ago

In 17305/josm:

see #7548: Re-organize the preference dialog

  • Correct handling of expert mode toggle (when enable/disable expert mode ... no content displayed)

(regression introduced by r17265)

comment:131 by GerdP, 3 years ago

Shit, still doesn't work when last selection was one that is only shown in expert mode and expert mode is toggled while dialog is not visible (View ->Expert Mode)

comment:132 by GerdP, 3 years ago

That one is caused by a memory leak. PreferenceTabbedPane still listens to the toggle change after dialog was closed. It is not destroyed properly.

comment:133 by GerdP, 3 years ago

In 17306/josm:

see #7548: Re-organize the preference dialog

  • let selectTabBy() always select a tab, either the wanted or the first if the wanted doesn't exist

TODO: fix memory leak caused by listeners

comment:134 by GerdP, 3 years ago

The memory leak isn't that easy to fix, but there is no need to handle it in this ticket.

comment:135 by simon04, 3 years ago

Priority: blockermajor

comment:136 by skyper, 3 years ago

Ok, the tab stays now, but the selection highlighting on the left is still lost.

Found two more issues which could be related to each other:

  1. The focus of the list on the left is off if you use the shortcuts from panels like validator or mappaint styles
  2. The list on the left needs a defined bottom to stop scrolling down. Opening preferences with the shortcut from validator panel offers me a few entries visible on top with lots of empty space below, where almost all entries could be display without having to scroll the list.
Last edited 3 years ago by skyper (previous) (diff)

comment:137 by GerdP, 3 years ago

Seems that I re-introduce an old error each time when I fix a new one :(
I did only test with a dialog that shows all entries. When the dialog is too small for that there is also still the empty page.

comment:138 by GerdP, 3 years ago

In 17307/josm:

see #7548: Re-organize the preference dialog

  • move code for expert toggle into expertChanged() and make sure that a tab is selected
  • always use SwingUtilities.invokeLater() when the PreferenceDialog should be opened with a special tab

comment:139 by GerdP, 3 years ago

Are there still plans to release a tested version 20.10 or can I commit changes for 20.11?

in reply to:  139 comment:140 by Don-vip, 3 years ago

Replying to GerdP:

Are there still plans to release a tested version 20.10 or can I commit changes for 20.11?

Yes I plan to fix the remaining points in comment:117 and release it today.

comment:141 by Don-vip, 3 years ago

"Display" is used without context for two very different things now:

    private DisplayPreference() {
        super(/* ICON(preferences/) */ "display", tr("Display"),
                tr("Various settings that influence the visual representation of the whole program."), false, new JTabbedPane());
    }
        <item name="Clock" icon="presets/service/clock.svg" type="node" preset_name_label="true">
            <link wiki="Tag:amenity=clock" />
            <key key="amenity" value="clock" />
            <optional>
                <combo key="display" text="Display" values="analog,digital,sundial,unorthodox" />
                <reference ref="support" />
                <combo key="faces" text="Faces" text_context="clock" values="1,2,3,4" />
                <!-- ... -->
            </optional>
        </item> <!-- Clock -->

comment:142 by Don-vip, 3 years ago

In 17311/josm:

see #7548 - distinguish i18n context of "Display" string

comment:143 by Don-vip, 3 years ago

In 17312/josm:

see #7548 - fix UI issues in proxy preferences panel

comment:144 by skyper, 3 years ago

The "OSM-Server" tab needs a scroll bar. With OAuth enabled, click on "Display Advanced OAuth Parameters" and context above and below turns unreachable.

comment:145 by Don-vip, 3 years ago

In 17313/josm:

see #7548 - make tab width dynamic + add 5px padding between icon and text

comment:146 by Don-vip, 3 years ago

No release for today, these UI fixes took me way longer than expected, I'll resume tomorrow.

comment:147 by Don-vip, 3 years ago

In 17314/josm:

see #7548 - make clear which preferences settings allow new tabs from plugins. Make color preferences extensible. Hide tab area if a single tab is defined.

comment:148 by Don-vip, 3 years ago

In 35643/osm:

see #19968 - see #7548 - move colorscheme preferences to Color preferences tab + code cleanup

comment:149 by Don-vip, 3 years ago

In 35645/osm:

see #7548 - upgrade cadastre-fr plugin to use ProjectionPreference instead of MapPreference

comment:150 by Don-vip, 3 years ago

In 35647/osm:

see #7548 - upgrade rasterfilters plugin to use ImageryPreference instead of MapPreference

comment:151 by Don-vip, 3 years ago

In 17315/josm:

see #7548 - deprecate MapPreference, not used anymore

comment:152 by Don-vip, 3 years ago

Resolution: fixed
Status: newclosed

This will be all for this ticket and this milestone. New issues or enhancements should be tracked by new tickets.

comment:154 by Don-vip, 3 years ago

In 17316/josm:

see #7548 - fix #20094 - fix NPE

in reply to:  152 comment:155 by Klumbumbus, 3 years ago

Replying to Don-vip:

This will be all for this ticket and this milestone. New issues or enhancements should be tracked by new tickets.

Thx. Follow up in #20097.

comment:156 by Don-vip, 3 years ago

In 17317/josm:

see #7548 - deprecate MapPreferenceTest until we remove MapPreference

in reply to:  134 ; comment:157 by skyper, 3 years ago

Replying to GerdP:

The memory leak isn't that easy to fix, but there is no need to handle it in this ticket.

Do we have a new ticket for that?

Replying to skyper:

Ok, the tab stays now, but the selection highlighting on the left is still lost.

Seems to be a problem of the flatlaf plugin, see #20086.

Replying to skyper:

The "OSM-Server" tab needs a scroll bar. With OAuth enabled, click on "Display Advanced OAuth Parameters" and context above and below turns unreachable.

See #20099.

in reply to:  157 comment:158 by GerdP, 3 years ago

Replying to skyper:

Replying to GerdP:

The memory leak isn't that easy to fix, but there is no need to handle it in this ticket.

Do we have a new ticket for that?

No. I looked at it for quite a while and I think I got confused by the behaviour of WeakReference. Sometimes multiple instances of the PreferenceDialog exist for quite a while and then they disappear when a different dialog is opened. So probably nothing to fix here.

comment:159 by simon04, 3 years ago

Big thanks to all involved in re-organizing the preference dialog!

I think we are ready to release milestone:20.10 since there are no major issues left. Unless anyone vetoes or someone gets there first, I'd do a release in the afternoon/evening today.

by mdk, 3 years ago

Attachment: EmptyPrefs.png added

comment:160 by mdk, 3 years ago

In the current version r17329 the list in the "Advanced Preferences" tab is empty:


Version 0, edited 3 years ago by mdk (next)

in reply to:  159 comment:162 by Don-vip, 3 years ago

Replying to simon04:

Big thanks to all involved in re-organizing the preference dialog!

I think we are ready to release milestone:20.10 since there are no major issues left. Unless anyone vetoes or someone gets there first, I'd do a release in the afternoon/evening today.

No objection, given the date, I will just rename 20.10 to 20.11 and 20.11 to 20.12.

comment:163 by Don-vip, 3 years ago

Milestone: 20.1020.11

Milestone renamed

comment:164 by GerdP, 3 years ago

@Simon: It seems the milestone 20.11 was not set. Is this still possible?

comment:165 by simon04, 3 years ago

What do you mean? This ticket has milestone=2011?

comment:166 by GerdP, 3 years ago

I mean "Mark milestone complete" in wiki:DevelopersGuide/Releasing
I am still offered to set milestone 20.11 for tickets.

comment:167 by simon04, 3 years ago

I cannot. I don't have the necessary Trac permissions.

comment:168 by Don-vip, 3 years ago

In 17482/josm:

see #7548 - remove deprecated MapPreference

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.