Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide overlay icons when photo detail is clicked #279

Closed
6 tasks done
damianmoore opened this issue Jun 16, 2021 · 5 comments
Closed
6 tasks done

Hide overlay icons when photo detail is clicked #279

damianmoore opened this issue Jun 16, 2021 · 5 comments
Assignees
Labels
Milestone

Comments

@damianmoore
Copy link
Collaborator

damianmoore commented Jun 16, 2021

When viewing a photo in detail it can de distracting to see icons on top of it. In most phone gallery apps you can tap once to make the icons disappear.

  • Tap once to hide icons
  • When icons hide, any bounding boxes should also hide
  • Tap again once shows the icons (and bounding boxes if shown previously) again
  • When the icons have disappeared you should still be able to pinch zoom in on the photo
  • Double tap action shouldn't show/hide icons as this is reserved for zoom in
  • User can exit the photo detail with their browser/hardware back button but the icons should show by default for the next photo detail viewed
@damianmoore damianmoore added this to the 1.0 milestone Jun 16, 2021
@GyanP
Copy link
Collaborator

GyanP commented Jun 21, 2021

I have completed the task and changes pushed on branch 279-hide-overlay-icons.

@damianmoore
Copy link
Collaborator Author

Hi @GyanP, I've ticked off the checkboxes above for the ones that I'm happy with.

The icons are not hiding exactly as I was intending. These are what should be hidden as well as the bounding boxes:

Screenshot 2021-07-12 at 23-03-55 Photonix Photo Manager

Having played with it a bit more I've realised that some other things need to be done with regard to the metadata info sidebar on the right (if it showing).

Screenshot 2021-07-12 at 23-14-00 Photonix Photo Manager

  • When the sidebar is shown, make sure the icons (and bonding boxes if turned on) are showing
  • If the area highlighted is clicked, hide the metadata info sidebar - do not hide the icons of bounding boxes as would happen if the sidebar was not showing.
  • Once sidebar is hidden again, single click and double-click actions are restored.

@GyanP
Copy link
Collaborator

GyanP commented Jul 13, 2021

modification done and changes pushed on branch 279-hide-overlay-icons

1 similar comment
@GyanP
Copy link
Collaborator

GyanP commented Jul 13, 2021

modification done and changes pushed on branch 279-hide-overlay-icons

@damianmoore
Copy link
Collaborator Author

Thanks @GyanP. Merged as of cecf477.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants