-
Notifications
You must be signed in to change notification settings - Fork 25
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
Inspiration for new UX features #28
Comments
I think it might be cool to group certain metrics together. Maybe just a visual line not necessarily hiding metrics. I like the ? icon though. Very helpful for metrics I'm less familiar with. It would be nice upon click to dive into more info (or perhaps link to another site with info) as well. Thanks for making this btw, it's an awesome tool. |
Kitchen sink of ideas... Go if I weren't lazy I'd be sending a pull request. There are quite a few charts that are flat lined. It would be cool to toggle those off. It would also be cool to see % backend vs % front end as a chart. Lastly a trimmed down view with say 3-6 charts good for sharing with non technical team members. |
Cool, thx! It's not really important, but might be of use from time to time: The JS script at the and of the index.html could use line breaks for every new entry. I recently had a (it really is a rare special case) merge conflict and then opened the differing index.html files in the meld diff viewer and it almost made my Ubuntu coming to a halt because that one differing JS line was simply much too long for meld. |
-> already implemented in new version. 😄 |
@Augenfeind
Hmm, nope. would like to stick to this timestamp. But what about displaying the date in graphs more clearly?
Have to check on that, later. |
Thanks anyone for feedback so far!!! That's awesome. |
Non UI wise - I have multiple folders that measure different routes on my server: in each one of them the d3, phantomas and css are re-copied creating many redundant files. I would go for an index.html that would allow you to select by tabs which dir you want to load, and then you can see the stats for those route tests |
@stefanjudis Regarding human readable timestamps: I don't mean to replace the current timestamps, but - if using line breaks in the JS code - adding something like an additional comment above every line telling the date & time for the following code (if this is the way the code works - didn't yet have a deeper look into the internals). |
Phew... That's a tricky (and big) one. Not sure when to start that, but will create a ticket for that. Maybe you could specify in there with complete specs? @Augenfeind Right now it's simple |
One more question. Is any one interested in the table at the bottom?! I'm using grunt-phantomas daily and have to say, I'm not checking the table... 😄 |
I really prefer the table to the graphs. Tables are very concise, and for the graphs I have to hover over data-points to see them. |
What about tables next to graph then? |
That could work. You can also allow for optional generation of graphs/table through the grunt plugin options options: {
graphs: true,
table: true
} |
Setting a new option is a good idea 👍 To be honest I want to get rid of the templating at the bottom of the page. Grouping the metrics for table generation wasn't done pretty by me. :( But basically, when it would be possible to group metrics together and the table would by right under the graph, it should work out for you? I mean, the table right now is not sorted and for you probably not nice to read? |
Oh noes, late for the game. Graph x values rotated so in case they are long (as I had them so far), they can be read. |
How about not tabs, but - as an intermediate solution - checkboxes for turning on and off single stats? |
@Augenfeind Please check #32 ;) |
Shouldn't be a big deal. Wanne create an issue? ;) |
Hey guys, sorry for bothering. I'm close to shipping the next version of grunt-phantomas and I'm wondering what my next steps should be.
I'm thinking of UX improvements and am just curious what some people would like to have. I just want to collect some ideas.
If you're not using it, just ignore this message. 😄 I'm just collecting people, that filled issues...
Thanks a lot.
@pedro-teixeira, @nickpresta, @paazmaya, @pgilad, @shama, @mkehlmann, @Augenfeind, @davidlinse, @Greg-Boggs
The text was updated successfully, but these errors were encountered: