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

Redesign Add Graph Button #71

Closed
wants to merge 3 commits into from
Closed

Redesign Add Graph Button #71

wants to merge 3 commits into from

Conversation

VarsPhilos
Copy link

Fixes #69
Added fa-chart-bar icon and changed style to match other buttons.

Did I do this pull request anywhere near correct?

VarsPhilos and others added 3 commits July 22, 2019 18:25
Added fa-chart-bar to 'Add Graph' button and changed it's style to match other buttons.
@VarsPhilos VarsPhilos closed this Jul 24, 2019
@IshaGupta18
Copy link
Collaborator

I am so sorry to have missed it, could you please provide a screenshot of the new design?

@VarsPhilos
Copy link
Author

Thank you for the opportunity!

IMG_3554

@IshaGupta18
Copy link
Collaborator

This looks good! Could you change the color of this button though?

@VarsPhilos
Copy link
Author

It was blue originally, do you want it reverted back to that or do you have another color in mind?

@IshaGupta18
Copy link
Collaborator

IshaGupta18 commented Jul 27, 2019 via email

@IMRC21
Copy link

IMRC21 commented Aug 6, 2019

In my opinion, a gradient would look really nice.
Btn_1
background-image: linear-gradient( 359.8deg, rgba(56,104,178,1) 7.3%, rgba(31,67,122,1) 84.8% );

Btn_2
background-image: linear-gradient( 90.6deg, rgba(26,109,85,1) -0.1%, rgba(49,227,125,1) 105.9% );

Btn_3
background-image: linear-gradient( 90.6deg, rgba(77,34,109,1) -1%, rgba(90,142,253,1) 153.1% );

@IshaGupta18
Copy link
Collaborator

This looks good! Maybe you can try out the first one, along with the fa-icon given by @VarsPhilos ? Thanks a lot!

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.

Redesign Add Graph Button
3 participants