commit 31b89db66f73892d02ab078965a4f5f99a243a1e
author Andrew Bonventre <andybons@chromium.org> Thu May 05 20:32:03 2016 -0400
committer Andrew Bonventre <andybons@chromium.org> Fri May 06 12:36:07 2016 -0400
tree 2a3426b7ddb7e45245e43cc3b0cb9908a055067f
parent 2e984564101c6e9578627de10a3f2aba37382a87

Fix hit target size for change file path

In a previous change[1], the clickable hit target was
reduced to the width of the containing text with its
container taking up the desired width. Make the anchor
tag the container while still preserving the original
intent of [1].

[1] https://gerrit-review.googlesource.com/77070

Change-Id: I7d1addf4988eaca1416bcc25cc3dbc2fc6abbaf9

polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html

diff --git a/polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html b/polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html
index 8134704..f42cdb6 100644
--- a/polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html
+++ b/polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html
@@ -61,13 +61,11 @@
         flex: 1;
         overflow: hidden;
         padding-left: .35em;
+        text-decoration: none;
         text-overflow: ellipsis;
         white-space: nowrap;
       }
-      .path a {
-        text-decoration: none;
-      }
-      .path a:hover {
+      .path:hover :first-child {
         text-decoration: underline;
       }
       .oldPath {
@@ -136,16 +134,15 @@
         <div class$="[[_computeClass('status', file.__path)]]">
           [[_computeFileStatus(file.status)]]
         </div>
-        <div class="path">
-          <a href$="[[_computeDiffURL(changeNum, patchNum, file.__path)]]"
-              title="[[_computeFileDisplayName(file.__path)]]">
+        <a class="path" href$="[[_computeDiffURL(changeNum, patchNum, file.__path)]]">
+          <div title$="[[_computeFileDisplayName(file.__path)]]">
             [[_computeFileDisplayName(file.__path)]]
-          </a>
+          </div>
           <div class="oldPath" hidden$="[[!file.old_path]]" hidden
-              title="[[file.old_path]]">
+              title$="[[file.old_path]]">
             [[file.old_path]]
           </div>
-        </div>
+        </a>
         <div class="comments">
           <span class="drafts">[[_computeDraftsString(drafts, patchNum, file.__path)]]</span>
           [[_computeCommentsString(comments, patchNum, file.__path)]]