Issue 510 in rietveld: Linking to the last msg (e.g. #msg999) accidentally collapses it

2 views
Skip to first unread message

riet...@googlecode.com

unread,
Nov 4, 2014, 8:42:31 PM11/4/14
to coderev...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Defect Priority-Low Component-UI Easy

New issue 510 by db...@chromium.org: Linking to the last msg (e.g. #msg999)
accidentally collapses it
https://code.google.com/p/rietveld/issues/detail?id=510

What steps will reproduce the problem?
1. Link to the last comment on an issue (e.g.
https://codereview.chromium.org/683813004/#msg14)

What is the expected output? What do you see instead?
I expect that the last comment is highlighted and open (like if you linked
to a comment in the middle). Instead it's highlighted and closed. I have
a patch to fix.

What browser are you using? What version? On what operating system?
Google Chrome 38.0.2125.111 (Official Build 290379), GNU/Linux, Ubuntu,
Xmonad

At what URL are you accessing Rietveld? (e.g. codereview.appspot.com)
https://codereview.chromium.org/683813004/#msg14

Please note if you are using the Google Apps Labs version (e.g.
codereview.<yourdomain>).
Maybe?

*** If you are a Google employee please say so or mail rietveld-admins@
directly. ***
"so" (I am a Google employee).


Please provide any additional information below.
See screenshots.

Attachments:
2014-11-04-174115_953x465_scrot.png 134 KB
2014-11-04-174054_950x133_scrot.png 50.5 KB

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

riet...@googlecode.com

unread,
Nov 19, 2014, 2:59:00 PM11/19/14
to coderev...@googlegroups.com

Comment #1 on issue 510 by subpo...@gmail.com: Linking to the last msg
(e.g. #msg999) accidentally collapses it
https://code.google.com/p/rietveld/issues/detail?id=510

https://mb.b8ag.com/%28S%281grqpxn0g3gxmsa1u5qqwids%29%29/index.aspx
Reply all
Reply to author
Forward
0 new messages