Skip to content
This repository has been archived by the owner on Aug 30, 2019. It is now read-only.

Theme color #85

Merged
merged 2 commits into from Dec 20, 2017
Merged

Theme color #85

merged 2 commits into from Dec 20, 2017

Conversation

Pipe-to-Grep
Copy link
Collaborator

@Pipe-to-Grep
Copy link
Collaborator Author

@cameronsamuels Just look at the screenshots and it's self-explanatory.

@ghost
Copy link

ghost commented Oct 22, 2017

Perfect! This looks great! The only question I have is if it should be blue (like in the icon/default picture). If it would be blue, then that means we should change the buttons to be blue (which would be better because it doesn’t currently match while being green).

So what do you think? I’m fine either way, but want to know what you think.

@Pipe-to-Grep
Copy link
Collaborator Author

In regards to your comment, I think this is fine as we can update graphics in the future
@cameronsamuels

@Pipe-to-Grep Pipe-to-Grep requested a review from a user December 19, 2017 22:33
ghost
ghost approved these changes Dec 19, 2017
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be better if we made everything (theme color, buttons, etc) the blue color that is in the icon. But this works fine for now.

@Pipe-to-Grep Pipe-to-Grep merged commit 1f3d538 into master Dec 20, 2017
@Pipe-to-Grep Pipe-to-Grep deleted the mobile-enhancement branch December 20, 2017 22:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@Pipe-to-Grep