-
Notifications
You must be signed in to change notification settings - Fork 224
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
Continued styles and text tidying #892
Continued styles and text tidying #892
Conversation
Looks like you're reintroducing the bug fixed in #891. You might want to consider rebasing this branch onto master. |
assets/less/formstyles.less
Outdated
color: @buttonText; | ||
background-color: @primaryGreen; | ||
border-color: @primaryGreen; | ||
&:disabled, &[disabled]:hover { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor but my opinion is using pointer-events: none;
instead of specifying &[disabled]:hover
could be a bit cleaner.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
assets/less/formstyles.less
Outdated
color: @buttonText; | ||
background-color: @secondaryBlue; | ||
border-color: @secondaryBlue; | ||
&:disabled, &[disabled] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks a bit redundant without the :hover
. Also consider my suggestion in the previous comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
assets/less/formstyles.less
Outdated
color: @secondaryBlue; | ||
background-color: @buttonText; | ||
border: 2px solid @secondaryBlue; | ||
&:disabled, &[disabled] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
assets/less/templates.less
Outdated
@@ -177,13 +177,12 @@ | |||
font-weight: normal; | |||
font-family: helvetica; | |||
.box-sizing(border-box); | |||
background-color: #564E4C; | |||
background-color: @brandDark1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs a semi colon
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Personal opinion is it'd be nice to have some kind of hover state change (other than a change of cursor) for the database names on the |
color: @buttonText; | ||
background-color: @dangerRed; | ||
border-color: @dangerRed; | ||
&:disabled { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need to consider hover state here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
…ibm/couchdb-fauxton into colorclysm-cleanup-couch
I've edited strings for clarity/consistency in a number of places.
Updated primary/secondary-action buttons to be consistently styled.
Added hover states to a number of buttons that didn't previously have them.
Made some minor style changes (rule colors, heading spacing)
Removed underline effects on links, but maintained hover-color and cursor changes.