-
Notifications
You must be signed in to change notification settings - Fork 125
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
SLE-981,SLE-980: Update labels and icons for rebranding #764
Conversation
Update all the user facing labels, including notifications, dialogs, views, popups and further visible information.
8645245
to
7e44993
Compare
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.
I understand that on specific locations you have to use SonarQube
instead of SonarQube for Eclipse
, but I believe that sometimes you could use the full name when you are not restricted by the size of the panel (in settings, notifications, etc.). WDYT?
....its.connected.sq/src/org/sonarlint/eclipse/its/connected/sq/SonarQubeConnectedModeTest.java
Outdated
Show resolved
Hide resolved
...ed/src/org/sonarlint/eclipse/its/shared/reddeer/dialogs/ConfirmConnectionCreationDialog.java
Outdated
Show resolved
Hide resolved
...int.eclipse.its.shared/src/org/sonarlint/eclipse/its/shared/reddeer/dialogs/TrustButton.java
Outdated
Show resolved
Hide resolved
...st/java/org/sonarlint/eclipse/core/internal/engine/AnalysisRequirementNotificationsTest.java
Outdated
Show resolved
Hide resolved
...se.core/src/org/sonarlint/eclipse/core/internal/engine/AnalysisRequirementNotifications.java
Outdated
Show resolved
Hide resolved
...g/sonarlint/eclipse/ui/internal/binding/assist/ConfirmSonarQubeConnectionCreationDialog.java
Outdated
Show resolved
Hide resolved
...rlint.eclipse.ui/src/org/sonarlint/eclipse/ui/internal/views/issues/SonarLintReportView.java
Show resolved
Hide resolved
@nquinquenel Most of the time when I use |
Update all the user facing icons. Remove the old GIMP files as they are not used anymore.
7e44993
to
716191a
Compare
Based on PR review and validation feedback.
1205520
to
f7524d8
Compare
Due to the Gradle integration the tests sometime intertwine and are therefore flaky.
Due to issues with the rebranding and Orchestrator we have to set a static version for now. Will be fixed later.
Quality Gate passedIssues Measures |
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.
LGTM
SLE-981
Update all the user-facing labels, including notifications, dialogs, views, popups, and further visible information. Additionally, add the new icons.