-
-
reviewboard/reviews/builtin_fields.py (Diff revision 1) I think we should include the title only in the abbreviated case. It's always weird to me when text elements have tooltips that are the same as the text.
Show the full commit ID in a tooltip for the Commit field.
Review Request #5455 — Created Feb. 11, 2014 and submitted
Show the full commit ID in a tooltip for the Commit field.
The Commit field shortens the commit ID if it's too long. It's still
useful to be able to see the full ID, though. Now, hovering over the
shortened ID will show a tooltip containing the full ID.
Posted a change with an abbreviated commit ID. Hovered over the abbreviated ID
and saw a tooltip with the full ID.Posted a change against Mercurial that didn't abbreviate the commit ID. Hovered
over and never saw a tooltip.
| Description | From | Last Updated |
|---|---|---|
|
I think we should include the title only in the abbreviated case. It's always weird to me when text elements … |
 david
|
Change Summary:
The tooltip is now only shown for abbreviated commit IDs.
Testing Done: |
|
|||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Commit: |
|
|||||||||||||||||||||
Diff: |
Revision 2 (+5 -2) |