Modify

Opened 9 months ago

Closed 9 months ago

Last modified 9 months ago

#23519 closed enhancement (fixed)

[Patch] Don't automatically enlarge "Validation Results" panel

Reported by: PPete2 Owned by: GerdP
Priority: normal Milestone: 24.02
Component: Core validator Version: latest
Keywords: Cc:

Description (last modified by PPete2)

I'm using JOSM v18969 on Windows 10, installed with Windows-setup file.

Right now each time JOSM detects a validation warning, the "Validation Results" panel is enlarged. Undependend if its completely minimized, or opened with a small height.
Please keep the panel as it is before, and let the user enlarge it on its own. Reason: This panel is regularly undesiredly taking important space from other opened panels.

=>

Attachments (3)

validation before.png (19.2 KB ) - added by PPete2 9 months ago.
validation after.png (20.6 KB ) - added by PPete2 9 months ago.
23519.patch (4.4 KB ) - added by GerdP 9 months ago.
implement new preference validator.force.unfurl.window with default value true , cleanup of duplicated / unnecessary code

Download all attachments as: .zip

Change History (15)

by PPete2, 9 months ago

Attachment: validation before.png added

comment:1 by PPete2, 9 months ago

Description: modified (diff)
Type: defectenhancement
Version: latest

by PPete2, 9 months ago

Attachment: validation after.png added

comment:2 by PPete2, 9 months ago

Description: modified (diff)

comment:3 by GerdP, 9 months ago

I can only reproduce that when the window is collapsed (as shown in the screen shot). When the window is opened the size is not changed.
It should be possible to add a new preference to keep a collapsed validation window in that state.

by GerdP, 9 months ago

Attachment: 23519.patch added

implement new preference validator.force.unfurl.window with default value true , cleanup of duplicated / unnecessary code

comment:4 by GerdP, 9 months ago

Milestone: 24.02
Owner: changed from team to GerdP
Summary: Don't automatically enlarge "Validation Results" panel[Patch] Don't automatically enlarge "Validation Results" panel

comment:5 by GerdP, 9 months ago

Furher thought:
A dialog that is collapsed is stored with a lastheight = 0 in the preferences, therefore it requires more code to unfurl a dialog (only) to its previous height.

comment:6 by PPete2, 9 months ago

I uninstalled JOSM, completly deleted its %APPDATA%\josm\ folder and reinsalled JOSM and all former used Plugins. Now the height of opened "Validation Results" panel is kept.
I support the idea to also keep a collapsed validation window in its collapsed state.

Last edited 9 months ago by PPete2 (previous) (diff)

comment:7 by GerdP, 9 months ago

Resolution: fixed
Status: newclosed

In 19000/josm:

fix #23519: Don't automatically enlarge "Validation Results" panel

  • implement new preference validator.force.unfurl.window with default value true, if set to false the window is not unfurled
  • code cleanup to remove duplicate or obsolete code

comment:8 by GerdP, 9 months ago

Resolution: fixed
Status: closedreopened

@PPete2: I am not sure if the new preference validator.force.unfurl.window works as you intended. In the German discussion you mentioned that this happens on upload, here you didn't.
So, what should happen with validator.force.unfurl.window=false

  • when there is no validator window and user starts the normal validation? With the current implementation, the validation window is not created.
  • when the window is collapsed? With the current implementation, the validation window is stays collapsed.

comment:9 by PPete2, 9 months ago

I have to say, that I just "use" automated validation during upload, I hardly use manual validation. But I think both variants can be treated the same way.

a) If there's no validation panel on the right => place panel and open it
b) If there's collapsed validation panel on the right => keep it collapsed
c) If there's opened validation panel on the right => keep it openend but don't change its height

comment:10 by GerdP, 9 months ago

OK, thanks, that's good for me as well. I'll try to find a solution because a) doesn't happen now.

comment:11 by GerdP, 9 months ago

Resolution: fixed
Status: reopenedclosed

In 19005/josm:

fix #23519: Don't automatically enlarge "Validation Results" panel

  • show and unfurl "Validation Results" panel if not yet visible unless the tree is empty and the validation was triggered by an upload (see #23187)

comment:12 by skyper, 9 months ago

Component: CoreCore validator

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain GerdP.
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.