-
Notifications
You must be signed in to change notification settings - Fork 27
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
Make the "Deploy to OpenShift" command more discoverable #406
Comments
Currently, we only generate a notification when a user debugs a Quarkus application, and only the first time, so very limited. A few places we could put this :
|
- Fixes redhat-developer#406 Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
- Add command to editor title menu bar - Add command to the view title menu - Fixes redhat-developer#406 Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
- Add command to editor title menu bar, and view title menu - Also make "Debug Quarkus Application" more discoverable by adding to title run menu - Create a duplicate command, ${id}.short whose title is a shorter version suitable for context menus - Fixes redhat-developer#406 Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
- Add command to editor title menu bar, and view title menu - Also make "Debug Quarkus Application" more discoverable by adding to title run menu - Create a duplicate command, ${id}.short whose title is a shorter version suitable for context menus - Fixes #406 Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
How can I hide the icon though? |
Currently it isn't possible to hide this. The icon should only show up in a Java source file when a Quarkus project is detected. I noticed we don't check whether vscode-openshift is installed but the icon does prompt to handle that if clicked. @fbricon , is it worth having a setting just to disable the icon ? The only other options I can think of are :
|
Today, only people who know the "Deploy to OpenShift" command exist will be able to use it. We need to figure out a way to make it more obvious
The text was updated successfully, but these errors were encountered: