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

Reduce Overdraw on About activity #4222

Open
wants to merge 7 commits into
base: release/4.0
Choose a base branch
from

Conversation

hammadrfq3
Copy link
Contributor

Description

Overdraw issue has been fixed by using constraintLayout instead of Nested Relative and LinearLayouts and also added recyclerview for translation section.

Issue tracker

Fixes #3109

Manual tests

  • Done

Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

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

@hammadrfq3 thanks for contributing to amaze! your PR looks great; but, there are some minor things to be addressed before this is ready for merging. please take a look at the review comments.

@hammadrfq3 hammadrfq3 requested a review from VishnuSanal July 24, 2024 10:30
Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

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

awesome! looks good to me. :)

@hammadrfq3
Copy link
Contributor Author

awesome! looks good to me. :)

Thanks

@VishnuSanal VishnuSanal added the pr-awaiting-final-review this PR is awaiting a final review/approval label Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-awaiting-final-review this PR is awaiting a final review/approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce overdraw on About activity
3 participants