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

trace-explorer: add to default layout #82

Merged

Conversation

vince-fugnitto
Copy link
Contributor

What it does

Fixes: #80

The following commit adds the Trace Explorer to the default layout of the application.
Adding the view to the default layout will make it more discoverable and accessible to end-users, and can later be automatically opened if a trace is identified.

How to test

  1. checkout the branch, build the application and start: yarn && (cd browser-app && yarn start)
  2. execute the command Reset Workbench Layout
    (using the command palette ctrlcmd+shift+p)
    (the command resets the layout to it's default (without local-storage storing the previous layout))
  3. when the application is reloaded, the view should be present in the sidebar.

Screenshot from 2020-06-18 10-44-03

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com

Fixes: eclipse-cdt-cloud#80

The following commit adds the `Trace Explorer` to the default layout of
the application. This will make the view more discoverable and
accessible to end-users.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
Copy link
Collaborator

@bhufmann bhufmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks

Copy link
Contributor

@tahini tahini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good! Thanks!

@vince-fugnitto
Copy link
Contributor Author

I don't have the necessary permissions to the repo (reason in which I'm working from a fork) so please don't hesitate to merge if the changes are acceptable :)

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

Successfully merging this pull request may close these issues.

Trace Explorer view not part of left-side panel by default
4 participants