Skip to content
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

CHE-2813 update style for popups in Dashboard #3818

Merged
merged 1 commit into from
Jan 25, 2017
Merged

CHE-2813 update style for popups in Dashboard #3818

merged 1 commit into from
Jan 25, 2017

Conversation

olexii4
Copy link
Contributor

@olexii4 olexii4 commented Jan 19, 2017

Signed-off-by: Oleksii Orel oorel@codenvy.com

What does this PR do?

Update style for popups in Dashboard

What issues does this PR fix or reference?

#2813

selection_001
selection_002
selection_003
selection_006
selection_007
selection_008

Changelog and Release Note Information

Changelog

Update style for popups.

Release notes:

  • Updated the styles for popups by adding header and making content look common

@codenvy-ci
Copy link

@ashumilova
Copy link
Contributor

@olexii4 please check the docs on modified popups - to be sure docs contain the same one.

@olexii4
Copy link
Contributor Author

olexii4 commented Jan 23, 2017

@ashumilova the docs doesn't contain these popups

@olexii4 olexii4 force-pushed the CHE-2813 branch 2 times, most recently from 7e94c94 to 8b85d69 Compare January 23, 2017 17:50
@codenvy-ci
Copy link

}, (error) => {
this.cheNotification.showError(error.data && error.data.message ? error.data.message : 'Edit registry error.');
}, (error: any) => {
this.cheNotification.showError(error.data && error.data.message ? error.data.message : 'Error.');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'Error.' - add more description

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@codenvy-ci
Copy link

@ashumilova
Copy link
Contributor

Following dialogs also should be fixed:

components
env_variables
machine
projects

@olexii4
Copy link
Contributor Author

olexii4 commented Jan 24, 2017

selection_024
selection_023
selection_022
selection_021
selection_020
Done

@olexii4 olexii4 requested a review from slemeur January 24, 2017 17:22
@codenvy-ci
Copy link

@ashumilova ashumilova added this to the 5.2.0 milestone Jan 25, 2017
Signed-off-by: Oleksii Orel <oorel@codenvy.com>
@olexii4 olexii4 merged commit b229694 into master Jan 25, 2017
@olexii4 olexii4 deleted the CHE-2813 branch January 25, 2017 16:19
@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Jan 26, 2017
@JamesDrummond JamesDrummond mentioned this pull request Feb 6, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
CHE-2813 update style for popups in Dashboard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants