-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
Outdated and inconsistent dialogs in settings #299
Comments
I made a branch to fix that: https://github.com/Darkempire78/OpenCalc/tree/fix-%23299 |
Thanks, it is still a clear improvement. |
@Darkempire78 you could also add screenshots. It could save few minutes. |
Please be respectful. @Darkempire78 is doing free and volunteer work on an open source project. He literally has zero obligations to you, and if you want luxury treatment you better be donating him a handsome amount before speaking. |
@meenbeese I didnt meant that in any disrespectful way. Just pointing it out. It would be nice. Yes, in future I'm going to donate. + I'm going to provide the screenshots. Here they are. |
I'm sorry if I sounded harsh, but I see this attitude in many people who are new to open source. It only demotivates the developer and harms the community. Your original comment sounded very condescending, so thanks for modifying it @inson1. |
@meenbeese To be clear I wanted to add the screenshots when I added the first comment. So that could changed the meaning. |
Describe the bug
Some of the dialog menus used in the settings are outdated and look out of place. They don't follow the Material You color palette and instead have a Material 2 design.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
They should be similar to the dialog menu used for the 'Theme'.
Screenshots/videos
Smartphone (please complete the following information):
Additional context
I saw this issue in only 2 of the 3 menus found in the app which are mentioned above.
The text was updated successfully, but these errors were encountered: