[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Change the all icons color in home.dart #203

Closed
wants to merge 3 commits into from
Closed

Change the all icons color in home.dart #203

wants to merge 3 commits into from

Conversation

Neha62-lit
Copy link
Contributor

1.Change the icons color.
2.Add a box-shadow to card.
3.Change the card color to white.

home_docLense

@Neha62-lit
Copy link
Contributor Author

@smaranjitghose Please review and merge this PR.

@Neha62-lit
Copy link
Contributor Author

@anushbhatia @smaranjitghose I renamed all file names to lowercase_with_underscore, because I'm getting some errors and according to Lint rules, all file names should be in lowercase. So I did it.
Please review and merge this PR.

arteevraina
arteevraina approved these changes Mar 18, 2021
View reviewed changes
Copy link
@arteevraina arteevraina 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!

@arteevraina
Copy link

Hey @Neha62-lit ! You will need to resolve conflicts after that it can be merged.

@Neha62-lit
Copy link
Contributor Author

Hey @Neha62-lit ! You will need to resolve conflicts after that it can be merged.

Okay!

@Neha62-lit
Copy link
Contributor Author

@smaranjitghose I am unable to resolve these conflicts. Please help me resolving those.

@anushbhatia
Copy link
Collaborator

https://www.youtube.com/watch?v=iRyUUGsNHrU use this. Also, you can refer to the documentation for this or else you can create a new PR.

@Neha62-lit
Copy link
Contributor Author

https://www.youtube.com/watch?v=iRyUUGsNHrU use this. Also, you can refer to the documentation for this or else you can create a new PR.

Okay..

@Neha62-lit Neha62-lit closed this Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arteevraina arteevraina Awaiting requested review from arteevraina

Assignees
No one assigned
Labels
deprecated
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Neha62-lit @arteevraina @anushbhatia @smaranjitghose