@Zackster really, really nice!!! Just downloaded the latest snapshot to take a look into it. Very good for finding bootlenecks. Is there a way to add a css cursor: pointer to the head cells, so you can see that the headers are clickable? something like setting a css like e.g. adding this to line 238 of Modern.cfc?
I’ve seen there is a link to set a sorting for/after a page reload behind the table header AVG-Times and table header order links. That may be a little confusing. By the way… I know I can PR it myself, but I really still don’t feel that comfortable to do it right at the moment.
that fix looks good, only problem is that might affect the other debugging templates which didn’t get sorting added, but I think they don’t have THEAD so it should work if you add the THEAD into the CSS selector.
Just added my first PR Thanks for motivation and support Zac. Hope I did everything correctly. The guidelines state I should put LDEV number into commit, but I thought such a minor change shouldn’t have a JIRA ticket, or should I have created one first? Here is my PR:
When I first added the CSS I saw multiple table headers being affected (also in the search results of the reference tab). Then I added the css ID selector #-sign, isolating it to the “Execution-Tme” and “Implicit Variable Access” only.