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

Add titlebar in about page #137

Closed
wants to merge 1 commit into from
Closed

Add titlebar in about page #137

wants to merge 1 commit into from

Conversation

Biswa96
Copy link
Contributor

@Biswa96 Biswa96 commented Nov 18, 2023

This addition will provide easy navigation back to the main menu, as well as improve the user experience and consistent design.

  • Before:

before

  • After:

after

This addition will provide easy navigation back to the main menu,
as well as improve the user experience and consistent design.
@tmo1
Copy link
Owner

tmo1 commented Nov 19, 2023

Thank you! It looks okay, but I'm just considering whether it's worth adding and whether it clutters up the About page unnecessarily..

@Biswa96
Copy link
Contributor Author

Biswa96 commented Nov 19, 2023

As author of this pull request, my opinion would be a bit subjective. I frequently make use of the back button in title bar of other apps and find this feature to be a convenient way to navigate.

Feel free to merge or reject the pull request as you wish.

@Biswa96
Copy link
Contributor Author

Biswa96 commented Dec 12, 2023

I sincerely appreciate your time and effort in reviewing this pull request. Thank you.

@Biswa96 Biswa96 closed this Dec 12, 2023
@Biswa96 Biswa96 deleted the add-about-toolbar branch December 12, 2023 09:12
@tmo1
Copy link
Owner

tmo1 commented Dec 12, 2023

Thank you for your patience. I've procrastinated making a final decision, but I'm still thinking about it. Thank you again for your various contributions to the app!

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 this pull request may close these issues.

2 participants