opendev/gerrit
Code Issues Proposed changes

Remove background change on hover/active in gr-dropdown

Browse Source 
+ Also remove redundant gr-button within gr-account-dropdown.

Change-Id: I5cb4020355289b48be98fa74150092009f07544b
This commit is contained in:
Andrew Bonventre
2017-01-10 12:24:51 -08:00
parent 87cb82f5a1
commit 1bfefed941
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/elements/core/gr-account-dropdown/gr-account-dropdown.html
View File

@@ -36,11 +36,9 @@ limitations under the License.
</style> </style>
<gr-dropdown items=[[links]] top-content=[[topContent]] <gr-dropdown items=[[links]] top-content=[[topContent]]
horizontal-align="right"> horizontal-align="right">
<gr-button link class="dropdown-trigger" id="trigger">
<span hidden$="[[_hasAvatars]]" hidden>[[account.name]]</span> <span hidden$="[[_hasAvatars]]" hidden>[[account.name]]</span>
<gr-avatar account="[[account]]" hidden$="[[!_hasAvatars]]" hidden <gr-avatar account="[[account]]" hidden$="[[!_hasAvatars]]" hidden
image-size="56"></gr-avatar> image-size="56"></gr-avatar>
</gr-button>
</gr-dropdown> </gr-dropdown>
<gr-rest-api-interface id="restAPI"></gr-rest-api-interface> <gr-rest-api-interface id="restAPI"></gr-rest-api-interface>
</template> </template>

5
polygerrit-ui/app/elements/shared/gr-dropdown/gr-dropdown.html
View File

@@ -46,6 +46,11 @@ limitations under the License.
gr-button[link] { gr-button[link] {
padding: 1em 0; padding: 1em 0;
} }
gr-button:focus:not([primary]):not([secondary]),
gr-button:hover:not([primary]):not([secondary]) {
background-color: transparent;
border-color: transparent;
}
ul { ul {
list-style: none; list-style: none;
} }