Fix star styling in Firefox
+ An SVG requires a width and height to be set in FF.
+ Also tighten up the sizing in the dashboard view.
Change-Id: I32cf74f2d4f0861cc3be922944a326f51af1a3f4
diff --git a/polygerrit-ui/app/elements/gr-change-star.html b/polygerrit-ui/app/elements/gr-change-star.html
index 72a75ff..b3ff00d 100644
--- a/polygerrit-ui/app/elements/gr-change-star.html
+++ b/polygerrit-ui/app/elements/gr-change-star.html
@@ -22,19 +22,21 @@
<style>
:host {
display: inline-block;
+ overflow: hidden;
}
.starButton {
background-color: transparent;
border-color: transparent;
cursor: pointer;
- font-size: 1.2em;
- vertical-align: -.2em;
- margin-right: .25em;
+ font-size: 1.1em;
width: 1.2em;
+ height: 1.2em;
outline: none;
}
.starButton svg {
fill: #aab8c2;
+ width: 1em;
+ height: 1em;
}
.starButton-active svg {
fill: #ffac33;
diff --git a/polygerrit-ui/app/elements/gr-change-view.html b/polygerrit-ui/app/elements/gr-change-view.html
index ed1b820..04c8e5c 100644
--- a/polygerrit-ui/app/elements/gr-change-view.html
+++ b/polygerrit-ui/app/elements/gr-change-view.html
@@ -64,6 +64,10 @@
text-overflow: ellipsis;
white-space: nowrap;
}
+ gr-change-star {
+ margin-right: .25em;
+ vertical-align: -.425em;
+ }
.patchSelectLabel {
margin-left: var(--default-horizontal-margin);
}
diff --git a/polygerrit-ui/app/styles/gr-change-list-styles.html b/polygerrit-ui/app/styles/gr-change-list-styles.html
index 5a96b6c..51c91cb 100644
--- a/polygerrit-ui/app/styles/gr-change-list-styles.html
+++ b/polygerrit-ui/app/styles/gr-change-list-styles.html
@@ -18,8 +18,15 @@
<style>
.keyboard,
.star {
+ align-items: center;
+ display: flex;
+ justify-content: center;
+ padding: 0;
width: 2em;
}
+ .star {
+ padding-top: .05em;
+ }
.subject {
flex-grow: 1;
flex-shrink: 1;
