OpenCalc icon indicating copy to clipboard operation
OpenCalc copied to clipboard

Outdated and inconsistent dialogs in settings

Open meenbeese opened this issue 1 year ago • 7 comments

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:

  1. Go to 'Settings'
  2. Click on 'Number of decimal places' or 'Results saved in history'
  3. See the old dialog/fragment

Expected behavior They should be similar to the dialog menu used for the 'Theme'.

Screenshots/videos Screenshot_20230728_211853_Calculator_1 Screenshot_20230728_211900_Calculator_1

Smartphone (please complete the following information):

  • Device: Galaxy S20+
  • OS: Android 13
  • Version: 3.0.0-beta3
  • Language: English

Additional context I saw this issue in only 2 of the 3 menus found in the app which are mentioned above.

meenbeese avatar Jul 29 '23 01:07 meenbeese

I made a branch to fix that: https://github.com/Darkempire78/OpenCalc/tree/fix-%23299 It's now using the material design 3, but not with the good colors

Darkempire78 avatar Nov 10 '23 17:11 Darkempire78

I made a branch to fix that: https://github.com/Darkempire78/OpenCalc/tree/fix-%23299

It's now using the material design 3, but not with the good colors

Thanks, it is still a clear improvement.

meenbeese avatar Nov 10 '23 21:11 meenbeese

@Darkempire78 you could also add screenshots. It could save few minutes.

inson1 avatar Nov 11 '23 00:11 inson1

@Darkempire78 you could also add screenshots. You could save as 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 avatar Nov 11 '23 01:11 meenbeese

@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.

studio64_xYuFmQhDkV studio64_Et08wDp60y studio64_dqaKN2vNi0 studio64_uAsL5O6v32

inson1 avatar Nov 11 '23 02:11 inson1

@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 avatar Nov 11 '23 18:11 meenbeese

@meenbeese To be clear I wanted to add the screenshots when I added the first comment. So that could changed the meaning. I didnt change the comment to be nicer, it just souded better. I didnt meant to sound condescending, its just nice when people share also the screenshots. So I want luxury treatmeant for everyone from everyone xD People can forget and that is ok.

inson1 avatar Nov 11 '23 18:11 inson1