Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(635)

Issue 10617044: Decorate section headers in review list.

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 6 months ago by sadrul
Modified:
12 years, 6 months ago
Reviewers:
Andi
Visibility:
Public.

Description

Decorate section headers in review list.

Patch Set 1 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -6 lines) Patch
M templates/user.html View 6 chunks +6 lines, -6 lines 0 comments Download

Messages

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
Andi
LGTM Thanks for pointing me to the discussion on chromium-dev. After skimming through the thread ...
12 years, 6 months ago (2013-06-27 06:50:40 UTC) #2
sadrul
On 2013/06/27 06:50:40, Andi wrote: > LGTM > > Thanks for pointing me to the ...
12 years, 6 months ago (2013-06-27 15:40:31 UTC) #3
Andi
Am Donnerstag, 27. Juni 2013 schrieb : > On 2013/06/27 06:50:40, Andi wrote: > >> ...
12 years, 6 months ago (2013-06-27 15:53:52 UTC) #4
sadrul
12 years, 6 months ago (2013-06-27 15:56:50 UTC) #5
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!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b