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

ThemeChanging Functionality Added #2723

Open
wants to merge 6 commits into
base: development
Choose a base branch
from

Conversation

revanthkumarJ
Copy link
Contributor

Fixes #Issue_Number

Please Add Screenshots If there are any UI changes.

Before:

WhatsApp.Video.2024-12-22.at.2.05.59.PM.mp4

Now:

Record_2024-12-22-14-33-57_0b55964ac4df2babebf0d80487f8a230.mp4

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

Copy link
Collaborator

@niyajali niyajali left a comment

Choose a reason for hiding this comment

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

@revanthkumarJ Requested few changes, have a look

@revanthkumarJ
Copy link
Contributor Author

@niyajali i have done the changes you told. Review it once

@therajanmaurya therajanmaurya self-assigned this Dec 24, 2024
@revanthkumarJ
Copy link
Contributor Author

@niyajali used edgeToedge API instead of window API can you review it once

Copy link
Collaborator

@niyajali niyajali left a comment

Choose a reason for hiding this comment

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

@revanthkumarJ Looks good to me, should be merged, and can you upload the latest video/image.

@revanthkumarJ
Copy link
Contributor Author

Status Bar Current Images

Light Mode DarkMode
Light 1 Dark 1

Theme Changing Current Video

WhatsApp.Video.2024-12-27.at.2.17.28.AM.mp4

@revanthkumarJ
Copy link
Contributor Author

@therajanmaurya sir , i have done all the changes told by niyaj bro and the theme functionality is working.
Can you review it once and Merge this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants