-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Improvements for intelligent commands - Phase 2 #4611
Conversation
Signed-off-by: Artem Zatsarynnyi <azatsarynnyy@codenvy.com>
…open previous command's editors
@azatsarynnyy : Could you please add the proper changelog (you can have multiple lines) |
if (hotKey != null) { | ||
SpanElement spanElement = Elements.createSpanElement(); | ||
spanElement.getStyle().setMarginLeft("5px"); | ||
spanElement.getStyle().setColor("#aaaaaa"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How it will be on white theme?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
White theme is not handled yet.
Will be part of that issue: #4590
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW it looks good on both themes
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2315/ |
Intelligent commands UX and UI improvements: Changed flow of new command goal creation Processes and Previews dropdown lists made fully clickable Command editor made fullwidth Command editors now are restorable as well as text editors Added shortcuts for the default "Run" and "Debug" commands: Alt+R and Alt+D respectively Added tooltip for process ID in commands toolbar
What does this PR do?
This PR polishes intelligent commands UX and UI based on the various feedbacks received during the beta
What issues does this PR fix or reference?
#4442
Changelog
Intelligent commands UX and UI improvements:
Release Notes
N/A
Will be released with #4389
Docs PR
eclipse-che/che-docs#168