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

Display Query name per each timeline entry #395

Open
dcaldastp opened this issue Aug 1, 2022 · 5 comments
Open

Display Query name per each timeline entry #395

dcaldastp opened this issue Aug 1, 2022 · 5 comments
Labels
Feature New feature or request

Comments

@dcaldastp
Copy link

dcaldastp commented Aug 1, 2022

About

Here's a sketch of the required addition - see myQueryX written in read near the Q icon, this is just a suggestion of course, placement of the label could be some other more suitable place.

Screenshot 2022-08-01 at 14 03 54

Each entry in the timeline would have somewhere displayed the query name for better exploration. This will save us the effort of clicking each entry to figure out which query we're looking at, which IMO carries an additional obstacle for the developer trying to wrap their head around the timeline.

@dcaldastp dcaldastp added the Feature New feature or request label Aug 1, 2022
@andyrichardson
Copy link
Collaborator

Hey @dcaldastp, thanks for the suggestion!

How would you feel about being able to see the query name when hovering over the Q / M icon?

@dcaldastp
Copy link
Author

dcaldastp commented Aug 1, 2022

How would you feel about being able to see the query name when hovering over the Q / M icon?

Hey @andyrichardson, thanks for the quick reply. Honestly, hover would not be as helpful since I would like to look at the timeline as a whole without having to keep track of a mental map of the timeline position to query/mutation name (I'm looking at a timeline with 20+ queries, it becomes a bit hard to keep track of it, even with the hover).

@andyrichardson
Copy link
Collaborator

Thanks for explaining your reasoning - that does make a lot of sense! For the time being, I think tool-tips would be the best we can do with the current layout due to the lack of space.

I did have a mess around with an alternative layout (below) - might make it easier to distinguish and find queries but would take more time than I have available to implement. Maybe something for the future 🤞

Alternative layout

Frame 8

Frame 9

Frame 10

@dcaldastp
Copy link
Author

I have another alternative for you to consider in order to tackle the lack of space. What about adding the query/mutation name within the event line itself 👇 (see the "myLongQueryName" on top of one of the lines)

Screenshot 2022-08-02 at 10 14 41

You can play with font-size/color to make it fit nicely I feel. What do you think?

@andyrichardson
Copy link
Collaborator

I like your thinking, although for information heirarchy, I think this might be a little tricky. The whole "row" is the query and ideally the name would be associated with the row rather than a "duration".

There are also some additional situations where this might not quite work:

  • There are no query durations in view for that row
  • There are multiple query durations in view for that row (do we show the name in all of them?)
  • The query duration is too short to show the query name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants