opendev/gerrit
Code Issues Proposed changes

Remove red color for 'Abandon' and 'Restore' buttons

Browse Source 
There is nothing dangerous about these operations that justifies
highlighting the buttons in red color. When the buttons are clicked
there is a popup where the user must confirm the operation, so it can
still be cancelled.

Change-Id: I7b8318b02067e749d99ac6a63874f7599152421b
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2015-01-08 08:33:22 +01:00
parent 020bcf8872
commit a332ae6d56
1 changed files with 2 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Actions.ui.xml
View File

@@ -45,12 +45,6 @@ limitations under the License.
line-height: BUTTON_HEIGHT; line-height: BUTTON_HEIGHT;
} }
#change_actions button.red {
color: #d14836;
background-color: #d14836;
}
#change_actions button.red div {color: #fff;}
#change_actions button.submit { #change_actions button.submit {
float: right; float: right;
background-color: #4d90fe; background-color: #4d90fe;
@@ -94,10 +88,10 @@ limitations under the License.
<div><ui:msg>Rebase Edit</ui:msg></div> <div><ui:msg>Rebase Edit</ui:msg></div>
</g:Button> </g:Button>
<g:Button ui:field='abandon' styleName='{style.red}' visible='false'> <g:Button ui:field='abandon' styleName='' visible='false'>
<div><ui:msg>Abandon</ui:msg></div> <div><ui:msg>Abandon</ui:msg></div>
</g:Button> </g:Button>
<g:Button ui:field='restore' styleName='{style.red}' visible='false'> <g:Button ui:field='restore' styleName='' visible='false'>
<div><ui:msg>Restore</ui:msg></div> <div><ui:msg>Restore</ui:msg></div>
</g:Button> </g:Button>
<g:Button ui:field='followUp' styleName='' visible='false'> <g:Button ui:field='followUp' styleName='' visible='false'>