Skip to content

Settings button should toggle between Settings and Notifications, and clicking “Refresh” when viewing Settings should show Notifications/hide Settings #552

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

Closed
hughlilly opened this issue Sep 23, 2022 · 0 comments · Fixed by #553

Comments

@hughlilly
Copy link
Collaborator

Sorry for the mouthful of a subject line. I’ve just noticed two minor UX issues I’d personally like to see remedied.

  • When on the Settings page, clicking the Settings button does not return to Notifications (you need to explicitly click the back arrow for this)
  • When on the Settings page, clicking the Refresh button does not return to Notifications (you need to explicitly click the back arrow and then click Refresh)

I don’t know how easy it would be to remedy these, or if anyone else would even find them useful or not. Just thought I would try and explain what I’m after. Here’s a screen recording that might help:

Screen.Recording.2022-09-23.at.3.38.51.PM.mov
hughlilly added a commit to hughlilly/gitify that referenced this issue Sep 23, 2022
Settings now toggles btw notifications & settings; Refresh shows notifs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant