[PATCH] [LNT] Various LNT web interface improvements

Yi Kong kongy.dev at gmail.com
Fri May 16 12:22:03 PDT 2014


On 16 May 2014 18:37, Chris Matthews <chris.matthews at apple.com> wrote:
> 0001: LGTM.
> 0002: LGTM.
> 0003: LGTM.
> 0004: Full sentence in the comment. LGTM.
> 0005: LGTM. This is a really good idea.
> 0006: I like this fix.  I’d rather render local times in as many places as possible.  LGTM.

Just to be clear, it's in server's time zone instead of users'. It's
useful if used internally, but may cause confusion for public servers?

>
> On May 16, 2014, at 9:58 AM, Yi Kong <kongy.dev at gmail.com> wrote:
>
>> Hi Chris Matthews,
>>
>> I've made several fixes and improvements to LNT web interface. Some of
>> them are suggested in [LLVMdev] Improving the usability of LNT. Most
>> of the patches are pretty straightforward.
>>
>> The server currently reports wrong time information. It's reporting
>> local time but always shows PST. 0006-support-alternate-timezones
>> fixes the problem by showing correct time zone info. Another solution
>> is always show in UTC(like the rest of LNT does). Which solution do
>> you prefer?
>>
>> Regards,
>> Yi Kong
>> <0001-Also-fix-background-color-rendering-for-run-status.patch><0002-Add-select-all-checkbox-for-report-tables.patch><0003-Make-compile_time-and-exec_time-tables-sortable.patch><0004-Activate-navbar-items-upon-hovering-the-mouse.patch><0005-Unify-the-use-of-jQuery.patch><0006-Support-alternate-timezones.patch>
>




More information about the llvm-commits mailing list