radar_pi icon indicating copy to clipboard operation
radar_pi copied to clipboard

Unable to re open PPI after dismissal

Open rszemeti opened this issue 1 year ago • 9 comments

I am running OpenCPN 5.8.4, radar 5.5.0 ... with a Garmin GMR 18HD+ (note the plus ...) scanner.

Closing the PPI via the [x] icon in the PPI window leaves the system in a state where neither the PPI nor the menu can be reovered. Going into plugi option via setting->plugins and selecting/selecting radar types can remove the and replace with default settings to restore operation.

rszemeti avatar Jan 17 '24 02:01 rszemeti

I think I understand the issue now ... the "icon" toggles between show radars (any individual PPIs or overlays marked as shown) and globally hides/shows them ... but this makes no sense in a situation where ALL the idividual PPIs and overlays are hidden.

For a user who has hidden the PPI the "wrong way" by clicking the [x] rather than the icon, it is only possible to get it back if you can right click, which I am finding rather tricky on a touchscreen. I appreciate the idea of the [x] was to allow multi-scanner users to remove individual PPI's

Would it not make more sense, in a situation where ALL PPI displays and overlays are individually hidden to show something when the global icon is clicked ... otherwise it just does nothing.

rszemeti avatar Jan 18 '24 16:01 rszemeti

Have you tried the canvas menu? Right click the chart window. There are radar items as well.

Hakansv avatar Jan 18 '24 17:01 Hakansv

I see same issue here. If I close radar screen using top right [x] I can’t bring it back

mgrouch avatar Jan 29 '24 11:01 mgrouch

Right mouse click on the chart, then click "Show radar" or "Radar Control..." followed by "Window > Show PPI".

miluk-nl avatar Jan 29 '24 11:01 miluk-nl

I have a touchscreen, right click is kinda difficult.

To my mind, if you turned off the PPI and you turned off the overlay .. now the icon in the menu does nothing, that seems, from a UI perspective, wrong. I would expect clicking the icon to do something maybe if both PPI and overlay are Off, when you toggle the menu item to "on" it should enable them both.

On Mon, 29 Jan 2024 at 11:57, miluk-nl @.***> wrote:

Right mouse click on the chart, then click "Show radar" or "Radar Control..." followed by "Window > Show PPI".

— Reply to this email directly, view it on GitHub https://github.com/opencpn-radar-pi/radar_pi/issues/255#issuecomment-1914543790, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALFTBRE4BAVOY6OU6I7L3T3YQ6FBBAVCNFSM6AAAAABB5YTLPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUGU2DGNZZGA . You are receiving this because you authored the thread.Message ID: @.***>

rszemeti avatar Jan 29 '24 13:01 rszemeti

A possible solution could be to adapt the function of the radar icon on the menubar. Currently this button is mainly a hide / show radar button. And the "show" click will get back the same what you hided, if there were no radar items shown, you get nothing back. It is simple to make the button always show the radar controls. So if there are no radar items visible, click the icon, hide the nothing. Click again, and the radar controls for all radars are show. This would also help users that don't know about the canvas menu, and ask questions of the forum how to show the radar. But my suggested solution will irritate users who use the icon to hide / show the radar and get back exactly what they have hidden. Now they will also get the radar controls. But these users could switch to the canvas menu hide / show radar. I have pushed a test version of this change in the branch "menubar". This change could possibly be implemented in a next release.

douwefokkema avatar Jan 31 '24 09:01 douwefokkema

I have a touchscreen, right click is kinda difficult.

Long hold on the screen should give a square grey outline under your finger, then release. This should bring up the context menu.

miluk-nl avatar Jan 31 '24 10:01 miluk-nl

The solution I proposed does not require a right click, it only makes use of the icon in the menu bar.

douwefokkema avatar Jan 31 '24 10:01 douwefokkema

A solution that would solve this issue and not irritate the hide show/users is simple, there are three setting to take into account:

  1. the global visibility setting

  2. the individual PPI visibility

  3. the individual overlay visibility

Clicking the icon toggles the global visibility. This is the current situation

Proposed solution: if either PPI or overlay are individually marked as shown, then clicking the icon toggles global visibility, same as current situation

if both PPI and Overlay are both marked individually as hidden: Current situation: Clicking icon toggles global visibility, but no effect because both items are individually marked as hidden. Proposed solution: Clicking icon sets both PPI and overlay to visible, toggles global visibility

On Wed, 31 Jan 2024 at 09:42, Douwe Fokkema @.***> wrote:

A possible solution could be to adapt the function of the radar icon on the menubar. Currently this button is mainly a hide / show radar button. And the "show" click will get back the same what you hided, if there were no radar items shown, you get nothing back. It is simple to make the button always show the radar controls. So if there are no radar items visible, click the icon, hide the nothing. Click again, and the radar controls for all radars are show. This would also help users that don't know about the canvas menu, and ask questions of the forum how to show the radar. But my suggested solution will irritate users who use the icon to hide / show the radar and get back exactly what they have hidden. Now they will also get the radar controls. But these users could switch to the canvas menu hide / show radar. I have pushed a test version of this change in the branch "menubar". This change could possibly be implemented in a next release.

— Reply to this email directly, view it on GitHub https://github.com/opencpn-radar-pi/radar_pi/issues/255#issuecomment-1918738445, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALFTBRFRDDKJTKQNNYZPR4TYRIGW5AVCNFSM6AAAAABB5YTLPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJYG4ZTQNBUGU . You are receiving this because you authored the thread.Message ID: @.***>

rszemeti avatar Jan 31 '24 11:01 rszemeti

Fixed in v5.5.4 commit d1b32e42c769ebf55cccfadf97e4e1d26330e7c0

Just press the Radar icon in the icon list, when radars are hidden it will show the controls.

keesverruijt avatar Aug 14 '24 10:08 keesverruijt