eclipse-cdt-cloud / theia-trace-extension

Eclipse Theia trace viewer extension using the Trace Server Protocol (TSP), through the tsp-typescript-client. Also the home for reusable JavaScript libraries: traceviewer-base, traceviewer-react-components
MIT License
49 stars 61 forks source link

Open some default views when a trace is opened #510

Open ebugden opened 3 years ago

ebugden commented 3 years ago

Opening a couple views that go from high-level to low-level has several benefits for users:

On the other hand, opening default views can be annoying for experienced users who don't need this additional guidance.

The default views are opened once and afterwards they can be moved, deleted, etc. as any other view would be (they have no special status).

Default views

These views were chosen because:

However, they're not based on user data or on the observation of analysis experts and may need to be adjusted in the future based on this.


Good default layouts are:

mirsky-work commented 3 years ago

@ebugden, can you please clarify how this one differs from #268?

ebugden commented 3 years ago

@ebugden, can you please clarify how this one differs from #268?

Good point. I adjusted #268's title and description as well as this issue's description. It needed to be updated because of changes in plan and because this issue exists now (more detailed explanation below). Thanks for mentioning this. Let me know if it's clear now!


268 was vague because, before #510 was created, the step of having just one basic set of views was implicitly included in #268 (as a first step). It also didn't have much detail because we initially planned for a basic set of default views to come later in the project (after the MVP, based on statistics of which views experts were using the most), but it's now happening earlier (to guide new users and to tutorialize the linked selection/navigation between charts).

mirsky-work commented 3 years ago

Thanks, it's clear now.