-
Notifications
You must be signed in to change notification settings - Fork 156
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
Improved Settings screen #716
Conversation
@arvindnegi1 @bpross-52n sir can you please review this PR? |
@SebaDro sir please review it. |
@SebaDro @arvindnegi1 @bpross-52n can you please review it before it gets superseded? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work @Akash-Ramjyothi . Lets discuss about this with @SebaDro Sir ;)
@asaikarthikeya Thank you so much for approving my PR. I am always ready and open for discussions with @SebaDro sir. |
The settings screen appears to be a little bit overloaded with all the big sized buttons in black color. Maybe, decreasing the size and a less saturated color would be less intrusive. |
Ok @SebaDro sir, thank you for the suggestion. Please give me some time, I will make the changes and commit them. |
Closed, due to inactivity. |
Fixes: #715
Description:
Added improved Settings screen by implementing Materialized Icons preceding to every Checkbox options to improve the interactivity, user interface and experience.
Sample demo of current Settings screen:
Sample demo of improved Settings screen: