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

fix: navigation drawer lag #2007

Merged
merged 1 commit into from
Nov 20, 2019
Merged

Conversation

harshshinde07
Copy link
Member

@harshshinde07 harshshinde07 commented Nov 15, 2019

Fixes #2001

Changes:

Changed the navigation drawer image from a png file to a vector drawable, thus drastically reducing the lag.

Checklist:

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing:

feature.zip

Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Can you upload a screenshot with the changes made?

@CloudyPadmal CloudyPadmal added Status: Review Required Requested reviews from peers and maintainers Enhancement Improvement to an existing feature labels Nov 15, 2019
@harshshinde07
Copy link
Member Author

@CloudyPadmal Here's the screenshot as requested.

@adityastic
Copy link
Collaborator

@neel1998 let's get this merged

@neel1998 neel1998 merged commit 57e3eae into fossasia:development Nov 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to an existing feature Status: Review Required Requested reviews from peers and maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation drawer opening closing animation is not smooth
4 participants