opendev/gerrit
Code Issues Proposed changes

Merge "ChangeScreen2: Add mouse hover highlight to file table"

Browse Source
This commit is contained in:
Edwin Kempin
2013-12-11 09:27:53 +00:00
committed by Gerrit Code Review
parent 58f1eef142 de540bf17c
commit c9595bc90a
2 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/FileTable.java
View File

@@ -64,6 +64,7 @@ class FileTable extends FlowPanel {
interface FileTableCss extends CssResource {
String table();
String nohover();
String pointer();
String reviewed();
String status();
@@ -406,7 +407,7 @@ class FileTable extends FlowPanel {
}
private void header(SafeHtmlBuilder sb) {
sb.openTr();
sb.openTr().setStyleName(R.css().nohover());
sb.openTh().setStyleName(R.css().pointer()).closeTh();
sb.openTh().setStyleName(R.css().reviewed()).closeTh();
sb.openTh().setStyleName(R.css().status()).closeTh();
@@ -586,7 +587,7 @@ class FileTable extends FlowPanel {
}
private void footer(SafeHtmlBuilder sb) {
sb.openTr();
sb.openTr().setStyleName(R.css().nohover());
sb.openTh().setStyleName(R.css().pointer()).closeTh();
sb.openTh().setStyleName(R.css().reviewed()).closeTh();
sb.openTh().setStyleName(R.css().status()).closeTh();

6
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/file_table.css
View File

@@ -22,6 +22,12 @@
.table tr {
vertical-align: top;
}
.table tr:hover {
background: rgba(209, 245, 248, 0.32);
}
.table tr.nohover:hover {
background: transparent;
}
.status {
padding-right: 4px;