Modify

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#22378 closed enhancement (fixed)

[Patch] Preview object properties on mouse hover

Reported by: Woazboat Owned by: team
Priority: normal Milestone: 22.10
Component: Core Version:
Keywords: mouse, hover, object, preview, iD Cc:

Description (last modified by taylor.smock)

Show tags and relation memberships of an object in the properties dialog when moving the mouser pointer over it (similar to iD).

Can be enabled/disabled via settings (enabled by default)

https://github.com/JOSM/josm/pull/104
https://github.com/JOSM/josm/pull/104.patch


Attachments (2)

josm_mouse_hover_object_preview.gif (369.5 KB ) - added by Woazboat 2 years ago.
Peek 2022-12-12 00-26.gif (399.3 KB ) - added by Woazboat 2 years ago.

Download all attachments as: .zip

Change History (29)

by Woazboat, 2 years ago

comment:1 by Woazboat, 2 years ago

Description: modified (diff)

comment:2 by Woazboat, 2 years ago

Summary: [Patch] Preview object proerties on mouse hover[Patch] Preview object properties on mouse hover

comment:3 by taylor.smock, 2 years ago

Description: modified (diff)

comment:4 by taylor.smock, 2 years ago

Resolution: fixed
Status: newclosed

In 18574/josm:

Fix #22378: Preview object properties on mouse hover (patch by Woazboat, modified)

Show tags and relation memberships of an object in the properties dialog when
moving the mouser pointer over it (similar to iD).

Can be enabled/disabled via settings (enabled by default)

comment:5 by taylor.smock, 2 years ago

Milestone: 22.10

comment:6 by Patrik_B, 2 years ago

Hey guys, thank you for keep adding new features into JOSM. I am personally not a fan of this one, how can I disable it please? :)
I tried to change "propertiesdialog.preview-on-hover" and "propertiesdialog.preview-on-hover.always-show-selected" as "false" but didn't worked.

Thank you

in reply to:  6 comment:7 by taylor.smock, 2 years ago

Replying to Patrik_B:

Hey guys, thank you for keep adding new features into JOSM. I am personally not a fan of this one, how can I disable it please? :)
I tried to change "propertiesdialog.preview-on-hover" and "propertiesdialog.preview-on-hover.always-show-selected" as "false" but didn't worked.

Thank you

Can you try via JOSM Preferences -> Display -> toggle Preview object properties on mouse hover? It is working for me.

comment:8 by Patrik_B, 2 years ago

Yes that worked! Thank you very much :) sorry I didn't checked that setting myself

comment:9 by skyper, 2 years ago

Well propertiesdialog.preview-on-hover is the correct advanced preference. I have updated the wiki (diff).

comment:10 by Patrik_B, 2 years ago

The disabling doesn't work properly for me. Every time I restart JOSM or just delete all layers and then download OSM data it is showing the preview even tho I have the option unchecked in Display settings and also in Advanced Preferences "propertiesdialog.preview-on-hover" is set to false.

I have to "turn on" the feature in Preferences, close preferences and then turn it off in Preferences to disable it, every time I restart JOSM.

I recorded my screen how to replicate the issue https://sendspark.com/share/bl90oab1vxs80e91

comment:11 by Woazboat, 2 years ago

Hmmm, I'll look into it but I'm away from home at the moment so it could take a couple of days.

comment:12 by Woazboat, 2 years ago

I actually just had a bit of time to take a look at it. Turns out the layer visibility change listeners I had to add to fix this are missing a check and ended up effectively re-enabling the hover preview on layer (visibility) change.

https://github.com/JOSM/josm/pull/105

comment:14 by skyper, 2 years ago

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

comment:15 by anonymous, 2 years ago

It would be fantastic to have the same thing when hovering the objects list in the upload window to do a (quick) verify of tagging completeness before committing to the server or to just a check on added/changed objects in the current open change set.

comment:16 by GerdP, 2 years ago

I find this behaviour very confusing. I am used to see the properties of the selected element. I really think the "hover over" should only do it's thing when nothing is selected.

in reply to:  16 ; comment:17 by skyper, 2 years ago

Replying to GerdP:

I find this behaviour very confusing. I am used to see the properties of the selected element. I really think the "hover over" should only do it's thing when nothing is selected.

Please, take a look at advanced preference propertiesdialog.preview-on-hover.always-show-selected to disable it with selected objects.
I would say this should be set to false per default.

Last edited 2 years ago by skyper (previous) (diff)

in reply to:  17 comment:18 by GerdP, 2 years ago

Thanks for the hint.

I would say this should be set to false per default.

Depends on what false mean. I have to try.

Last edited 2 years ago by GerdP (previous) (diff)

comment:19 by GerdP, 2 years ago

Just tried with r18612 and it seems changing the setting doesn't have an effect unless I create a new layer.
Besides that the defaults seem to work as I want them:

propertiesdialog.preview-on-hover	true
propertiesdialog.preview-on-hover.always-show-selected	true

in reply to:  16 comment:20 by Woazboat, 2 years ago

Replying to GerdP:

I find this behaviour very confusing. I am used to see the properties of the selected element. I really think the "hover over" should only do it's thing when nothing is selected.

That should already be the default behavior. Did you change any of the settings?

https://josm.openstreetmap.de/browser/josm/trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java?rev=18574#L269

comment:21 by GerdP, 2 years ago

Sorry, I am not sure what version and settings I used when I wrote comment:7. I disabled the hover over function and my preferences have both values set to false. I'll set both values to true again and see if I can reproduce my problem with latest.

in reply to:  19 comment:22 by skyper, 2 years ago

Replying to GerdP:

Thanks for the hint.

I would say this should be set to false per default.

Depends on what false mean. I have to try.

Oh no, I mixed it up. true is correct as default, e.g. show on hover is disabled with selection.

Replying to GerdP:

Just tried with r18612 and it seems changing the setting doesn't have an effect unless I create a new layer.

Yes, same for me, the behavior is only changed with new layers.

by Woazboat, 2 years ago

Attachment: Peek 2022-12-12 00-26.gif added

comment:23 by Woazboat, 2 years ago

Replying to skyper:

Replying to GerdP:

Just tried with r18612 and it seems changing the setting doesn't have an effect unless I create a new layer.

Yes, same for me, the behavior is only changed with new layers.

I can't reproduce the incorrect behavior you're seeing. It works as expected for me without changing to a new layer (r18612)


Last edited 2 years ago by Woazboat (previous) (diff)

comment:24 by GerdP, 2 years ago

I can reproduce like this:
1) select an object and hover
2) change option without changing the selection
3) hover again and nothing changes in the behaviour

You change the selection and I guess that's the difference?

comment:25 by skyper, 2 years ago

Yes, selected objects seems to be the problem

  1. start with fresh, empty preferences
  2. download some data
  3. select a object
  4. change propertiesdialog.preview-on-hover.always-show-selected to false

Now the feature does not work at all.

Changing the preference without selection works as expected.

in reply to:  25 comment:26 by Woazboat, 2 years ago

Replying to skyper:

Yes, selected objects seems to be the problem

  1. start with fresh, empty preferences
  2. download some data
  3. select a object
  4. change propertiesdialog.preview-on-hover.always-show-selected to false

Now the feature does not work at all.

Changing the preference without selection works as expected.

Fixed in https://github.com/JOSM/josm/pull/107

comment:27 by taylor.smock, 2 years ago

In 18631/josm:

See #22378: Fix hover preview setting changes not being applied immediately (patch by Woazboat)

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.