User/mattwar/more results view#38
Merged
Merged
Conversation
Results can be viewed in bottom panel, singleton view or document view. Add mode for display: chart, data, and all.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces several improvements and refactorings to the Kusto Explorer VS Code extension, focusing on enhancing the user experience, streamlining code organization, and updating documentation for clarity. The most significant changes include a major refactor of query document handling, updates to the results and chart display logic, improvements to the extension's configuration and commands, and expanded documentation.
Refactoring and Code Organization:
documentPanels.tsfile has been renamed and refactored toqueryDocuments.ts, consolidating query document logic and updating all relevant imports and activations to use the new module. This simplifies the codebase and clarifies responsibilities. [1] [2] [3] [4]Results and Chart Display Logic:
displayResultsPanelanddisplaySingletonResultViewfor consistent behavior, and redundant calls to separate results and chart display functions have been removed. Error handling for result display has also been streamlined. [1] [2] [3] [4] [5]Extension Configuration and Commands:
kusto.results.displayallows users to choose where query results are shown: either in the bottom panel or beside the query editor. This enhances customization and usability.Documentation Updates:
README.mdhas been expanded with clearer descriptions of features, including new sections for Kusto Connections, the Chart Panel, and improved explanations for result and chart handling. Typos and formatting have also been corrected. [1] [2] [3]UI Label Improvements: