|
|
|
Decorate section headers in review list.
Patch Set 1 : . #
Total messages: 5
|
sadrul
Hi! Having this should make it possible for extensions to move the issue-blocks around, so ...
|
12 years, 6 months ago (2013年06月26日 16:58:07 UTC) #1 | |||||||||||||||
Hi! Having this should make it possible for extensions to move the issue-blocks around, so change-averse devs like myself can continue to use the old layout! :) [ https://codereview.appspot.com/9971043/ seems to have stalled, and this is a simple workaround. The relevant chromium-dev@ thread is here: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/92t4yyGOm-M ]
LGTM Thanks for pointing me to the discussion on chromium-dev. After skimming through the thread I understand why there are some discussions regarding the UI changes and I agree to some of the arguments there.
On 2013年06月27日 06:50:40, Andi wrote: > LGTM > > Thanks for pointing me to the discussion on chromium-dev. After skimming through > the thread I understand why there are some discussions regarding the UI changes > and I agree to some of the arguments there. Thanks! Mind committing this patch (or is there a better way to commit patches for rietveld)? :)
Am Donnerstag, 27. Juni 2013 schrieb : > On 2013年06月27日 06:50:40, Andi wrote: > >> LGTM >> > > Thanks for pointing me to the discussion on chromium-dev. After >> > skimming through > >> the thread I understand why there are some discussions regarding the >> > UI changes > >> and I agree to some of the arguments there. >> > > Thanks! Mind committing this patch (or is there a better way to commit > patches for rietveld)? :) It's already committed and live on this instance. Let me know if you're missing something! --Andi > > https://codereview.appspot.**com/10617044/<https://codereview.appspot.com/106... >
On 2013年06月27日 15:53:52, Andi wrote: > Am Donnerstag, 27. Juni 2013 schrieb : > > > On 2013年06月27日 06:50:40, Andi wrote: > > > >> LGTM > >> > > > > Thanks for pointing me to the discussion on chromium-dev. After > >> > > skimming through > > > >> the thread I understand why there are some discussions regarding the > >> > > UI changes > > > >> and I agree to some of the arguments there. > >> > > > > Thanks! Mind committing this patch (or is there a better way to commit > > patches for rietveld)? :) > > > It's already committed and live on this instance. Let me know if you're > missing something! Oh, sweet! Thanks!