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

Introduce Dynamic constants, Theming option #177

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

Conversation

ajaman190
Copy link
Contributor

Introduce dynamic constants, theming option #172

Summary:
This pull request introduces a new feature of theming and making the app more dynamic to different screens.

Issues:

  1. Screens and components of the app are not dynamic, i.e., they are not changeable with the change in screen size, and all their values are hard-coded.
  2. The app does not have a feature of theming.

Issue resolved Checklists:

  • Introducing new dynamic constants that dynamically resize the components, fonts, and other elements on the screen.
  • Adding a theming option to the app, allowing users to change the look and feel of the app quickly and efficiently.
  • Revamping the login and signup pages to make them dynamic and customizable.

Recording:
I have tested these changes thoroughly to ensure that they work as expected.

Project.1.mp4

Ignore the first three commit they belong to the last pr # 176 (login and signup).

@shehand Please let me know if there are any issues or concerns with this pull request. I am happy to address any feedback or suggestions that you may have.

@ajaman190 ajaman190 requested a review from shehand as a code owner March 18, 2023 08:53
@ajaman190
Copy link
Contributor Author

@shehand kindly review this pull request. I am looking forward for feedback.

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.

1 participant