-
Notifications
You must be signed in to change notification settings - Fork 677
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
OmniSharp Status Bar Icon should be Workspace-level element #2146
Labels
Milestone
Comments
It makes sense to me |
Yes, vscode recognises the following set of icons: https://octicons.github.com/. We can use them at appropriate places. |
The text that we display will be very similar to the |
I think we can segregate the two into two separate status bar items:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The existing Status Bar element displayed by this extension is a bit confusing, both from a user's perspective as well as the maintainer's. For reference, the element is this:
The element is confusing because it represents workspace-wide state but attempts to do so in relation to the Active Text Editor. More specifically, the Status Bar element is only visible if the Active Text Editor is one that O# does/should care about. However, the status bar element communicates the status of the Workspace's O# server, something that is largely independent of the currently active document.
The status element's split intent leads to several undesirable characteristics which we should solve:
Switch Projects
though it is impossible to select a project that covers the active file. This results in an infinite loop in UX actions.Proposal
The O# status bar element should be converted to represent the status of the O# server at the VS Code context. This means:
The text was updated successfully, but these errors were encountered: