timings icon indicating copy to clipboard operation
timings copied to clipboard

UI Improvement - Table Layout

Open Slind14 opened this issue 8 years ago • 3 comments

Hi there,

I find it really difficult to read through the numbers combined with letters and not being able to sort by values. Is there a reason the data isn't displayed in a table format?

Atm it looks like this:

1. Current UI

Empire::Event: c.e.l.EmpireBlockListener (BlockPhysicsEvent) count(140612379) total(17.32% 69.665s, 19.77% of tick) avg(0.00ms per - 9.89ms/19,771.14 per tick)

2. Basic Table

Current layout converted into a table. (way more clear, easy to read, could be sort-able).

name total < < < avg < <
/\ calls totalPct totalTime pctOfTick avgMsPerCall totalMsPerTick callsPerTick
Empire::Event: c.e.l.EmpireBlockListener (BlockPhysicsEvent) 140,612,379 17.32% 69.665s 19.77% 0.00ms 9.89ms 19,771.14

3. Simplified Table

Going a step further by grouping the same data (different time unit).

name share time time/call calls
Empire::Event: c.e.l.EmpireBlockListener (BlockPhysicsEvent) 17.32% (19.77%/Tick) 69,665ms (9.89ms/Tick) 0.00ms 140,612,379 (19,771.14/Tick)

4. Smart Table

A further improvement - in terms of clean ui - the total/tick data toggle-able. So it only displays the total or avgPerTick values based upon a toggle button to switch between those two.

Total Time

name share time time/call calls
Empire::Event: c.e.l.EmpireBlockListener (BlockPhysicsEvent) 17.32% 69,665ms 0.00ms 140,612,379

Avg. per Tick

name share time time/call calls
Empire::Event: c.e.l.EmpireBlockListener (BlockPhysicsEvent) 19.77% 9.89ms 0.00ms 19,771.14

Slind14 avatar Dec 14 '16 21:12 Slind14

This is def on the roadmap!

I am in the middle of rewriting the entire front end to use React. Once thats done we can explore design changes like this.

aikar avatar Jan 29 '17 03:01 aikar

status update on this. I currently have the data sortable in the dev environment.

Goal is to still try to get the data represented in a column fashion and move the labels up to a header, which is now sticky.

https://timings.aikar.co/dev/

aikar avatar Feb 09 '17 15:02 aikar

Looking good so far. I would still like to see a table layout and a less "messy" look :)

Slind14 avatar Feb 13 '17 16:02 Slind14