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

feat: redesign login UI #110

Open
wants to merge 1 commit into
base: redesign
Choose a base branch
from
Open

feat: redesign login UI #110

wants to merge 1 commit into from

Conversation

laxyapahuja
Copy link

Issue Fix

Fixes #100

Screenshots

image

  • 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.

@rchtgpt
Copy link
Member

rchtgpt commented Jan 7, 2020

@laxyapahuja hey! Please send your PR on redesign branch :)

@laxyapahuja laxyapahuja changed the base branch from master to redesign January 7, 2020 08:50
@rchtgpt
Copy link
Member

rchtgpt commented Jan 7, 2020

Also, I think it would be better if we import the fonts in a different PR because if everyone started adding font files with their PRs then it can lead to merge conflicts.

@luckyman20
Copy link
Collaborator

@laxyapahuja Yes, @Rachittt is correct please do so.

@luckyman20 luckyman20 closed this Jan 9, 2020
@luckyman20 luckyman20 reopened this Jan 9, 2020
@laxyapahuja
Copy link
Author

@laxyapahuja Yes, @Rachittt is correct please do so.

Already fixed :)

@luckyman20
Copy link
Collaborator

I can still see .ttf files.

@laxyapahuja
Copy link
Author

I can still see .ttf files.

updated :D

app/src/main/res/values/styles.xml Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_login.xml Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_login.xml Outdated Show resolved Hide resolved
app/src/main/res/values/styles.xml Outdated Show resolved Hide resolved
app/src/main/res/values/styles.xml Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_login.xml Outdated Show resolved Hide resolved
@laxyapahuja
Copy link
Author

updated :)

@rchtgpt rchtgpt mentioned this pull request May 16, 2020
2 tasks
@garvit984
Copy link
Collaborator

@laxyapahuja please look into the changes requested and conflicting files.

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.

4 participants