-
Notifications
You must be signed in to change notification settings - Fork 77
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
UI Improvements #58
base: master
Are you sure you want to change the base?
UI Improvements #58
Conversation
Can you please attach the apk file here? |
https://drive.google.com/file/d/1HnobYn6zcC2jElTljOxYHw4lH6I4TgVZ/view?usp=sharing @mishraaditya595 apk not supporting. So I have updated in drive. Kindly check the link |
Well, the navigation bar does look pretty good but does not fit into when dark mode is enabled. Plus the icons does not convey what the fragment is about. Try fixing these. |
@mishraaditya595 I have changed the svg file for all 4 parts. Please check and let me know about the UI. I am still working on dark mode. Please merge this PR as in priority as dark mode is new to me. So I am working on it. |
provide the apk for the same |
hey @av-arvind77, there are several merge conflicts. Can you update this branch with the latest commits. The last commits were your PR #61 |
Yes sure @mishraaditya595 |
@mishraaditya595 I am unable to resolve conflicts. Can you please guide me to resolve this ? |
UI Improvements of the application
Fixes: #10
Describe the changes you've made
I have changed the bottom navigation view with some chip navigation bar into it.