HomePhorge

Highlight update time in revision list

Description

Highlight update time in revision list

Summary:
This is another experiment for reducing reviewers response time.
I stole the idea (and colors) from ReviewBoard.
I actually quite like it (except when everything is red) and I can image that people will review just to have better color balance.

The code is not production ready for these reasons:

  • We load holidays again and again for each revision. I couldn't cache it to static variable because it could persist multiple requests, right?
  • I don't know how to expand height to the whole cell (I'm really bad in CSS).
  • CSS rules are probably in wrong file.
  • We probably want to use different colors.

This is how it looks:
{F16406}

Test Plan: Displayed revision list.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D3190

Details

Provenance
vranaAuthored on Aug 20 2012, 2:05 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP90f5f869ae78: Minor, updates from @chad:
Branches
Unknown
Tags
Unknown

Event Timeline