Skip to content

changes in UI#14

Open
ub2906 wants to merge 4 commits into
ashikvarma11:masterfrom
ub2906:new-branch
Open

changes in UI#14
ub2906 wants to merge 4 commits into
ashikvarma11:masterfrom
ub2906:new-branch

Conversation

@ub2906

@ub2906 ub2906 commented Oct 2, 2020

Copy link
Copy Markdown

No description provided.

@ub2906

ub2906 commented Oct 2, 2020

Copy link
Copy Markdown
Author

Hey! I have added some changes in UI as follows

  • linear gradient background color in header
  • change in logo
  • change in the style of inputs
  • change in dimensions/alignment etc of buttons along with hover effects
    Hope you like it :)

@ashikvarma11

ashikvarma11 commented Oct 2, 2020

Copy link
Copy Markdown
Owner

@ub2906 I'll review it ASAP. The PR has conflicts. This might have occurred due to a change in directory structure. Please make sure you have the updated code. Take update of master and merge it into your own branch.

@ashikvarma11

ashikvarma11 commented Oct 2, 2020

Copy link
Copy Markdown
Owner

@ub2906 Hey, I recommend using a color palette.

https://colorhunt.co/ Try using colors from any single color palette. Find the palette that works for this. The colors you have used now doesnot seem to work with each other. Checkout this article: https://websitesetup.org/website-color-schemes/

@ub2906

ub2906 commented Oct 2, 2020

Copy link
Copy Markdown
Author

Thanks for the quick review! I have updated and merged the code. I also used the sites you recommended for choosing the color scheme. Hope it's better now!

@ashikvarma11

Copy link
Copy Markdown
Owner

@ub2906 You need to take the latest update and create a PR. This PR has conflicts with the latest code.

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.

2 participants