What steps will reproduce the problem?
(1) Use the "non-deprecated" Rietveld UI
(2) Visit https://codereview.chromium.org/2272793007#msg17
What is the expected output?
Comment #14 and #17 indicate LGTM in the consistent way.
What do you see instead?
For #17, the fact that comment #17 is linked in the URL causes it to be highlighted.
I thought for a moment that Rietveld now detects "LGTM with nits" and marks it as yellow.
But now, the hashlink simply trumps the existing color, and there is no cue to indicate this.
The same also applies to "not LGTM", which would simply be a little confusing, except that the white text color (meant for a red background) now shows up on beige. (not-lgtm.png)
https://codereview.chromium.org/2231383003#msg6
Rare edge cases, but I let no bug go unfiled! ;-)
Comment 1 by andyb...@chromium.org
, Sep 8 2016Status: Available (was: Untriaged)