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

Allow query result view to be toggled from command bar #1833

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

analogrelay
Copy link
Contributor

@analogrelay analogrelay commented May 2, 2024

Preview this branch

This PR adds a new dropdown to the Query Editor Command Bar:

image

The options in this menu allow you to toggle between Vertical (editor and results view are stacked vertically; matches the current view) or Horizontal (editor and results view are stacked horizontally, a new layout made available in this PR):

toggle-view-for-query.mp4

The default layout is Vertical, matching the current view. However, this too can be changed using a setting in the Settings pane:

image

Changing this option will affect the starting view for all future Query Editor panels opened on that Browser (the setting is stored in localStorage, along with all the existing settings on this pane). Existing opened Query Editors will not be affected:

toggle-default-view-for-query.mp4

@analogrelay analogrelay requested a review from a team as a code owner May 2, 2024 19:58
jawelton74
jawelton74 previously approved these changes May 2, 2024
also provides a default results view option that's stored in the
browser's local storage
@analogrelay analogrelay force-pushed the ashleyst/toggle-query-results-view branch from 50a1d10 to 31b097c Compare June 4, 2024 22:14
@analogrelay analogrelay merged commit 9b12775 into master Jun 5, 2024
20 checks passed
@analogrelay analogrelay deleted the ashleyst/toggle-query-results-view branch June 5, 2024 19:16
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.

None yet

4 participants