Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI Improvement - Table Layout #23

Open
Slind14 opened this issue Dec 14, 2016 · 3 comments
Open

UI Improvement - Table Layout #23

Slind14 opened this issue Dec 14, 2016 · 3 comments

Comments

@Slind14
Copy link

Slind14 commented Dec 14, 2016

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
@aikar
Copy link
Owner

aikar commented Jan 29, 2017

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
Copy link
Owner

aikar commented Feb 9, 2017

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/

@Slind14
Copy link
Author

Slind14 commented Feb 13, 2017

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants