Skip to content

Fix confirm overwrite modal of saved objects to handle large titles - #28590

Merged
mikecote merged 1 commit into
elastic:masterfrom
mikecote:fix-saved-object-confirm-overwrite-modal
Jan 17, 2019
Merged

mikecote merged 1 commit into
elastic:masterfrom
mikecote:fix-saved-object-confirm-overwrite-modal

Conversation

@mikecote

Copy link
Copy Markdown
Contributor

Summary

Fixes #25030.

This PR fixes the issue when importing saved objects with long titles. Some of the text hides under the "X" in the top right corner of the confirm overwrite modal. The approach I used is to add a title to the modals based on recommendation in #25030.

Before:
screen shot 2019-01-11 at 11 13 20 am
screen shot 2019-01-11 at 11 13 32 am
screen shot 2019-01-11 at 11 13 41 am
screen shot 2019-01-11 at 11 13 49 am

After:
screen shot 2019-01-11 at 11 10 44 am
screen shot 2019-01-11 at 11 10 56 am
screen shot 2019-01-11 at 11 11 04 am
screen shot 2019-01-11 at 11 11 11 am

Checklist

- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

@mikecote mikecote added Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v7.0.0 v6.7.0 labels Jan 11, 2019
@mikecote mikecote self-assigned this Jan 11, 2019
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-platform

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@mikecote
mikecote requested a review from spalger January 11, 2019 17:57

@spalger spalger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:fix review Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v6.7.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Confirm overwrite of saved objects is not displaying full object name

3 participants