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

[7.x] [Graph] Save modal (#44261) #44918

Merged
merged 1 commit into from
Sep 6, 2019

Conversation

flash1293
Copy link
Contributor

Backports the following commits to 7.x:

* create graph listing page

* clean up app folder

* remove inline loading menu

* also add badge to workspace route

* fix tests

* fix graph spaces functional test

* generate documentation for new breadcrumb property

* fix test subject names

* remove unused translations

* start implementing save modal flow for Graph

* fix spaces functional test

* wip save modal

* wip save modal

* add and style save modal

* add placeholder to description field

* disable dirty check on breadcrumb navigation and fix delete function

* improve onClick typing on breadcrumb

* fix newline error and use new types in dashboard app controller

* fix translation errors

* fix i18n translation for real

* code review

* fix i18n phrases

* remove fragments

* remove unnecessary max-width and add commentary

* move to async syntax

* clean up implementation

* use description instead of title

* fix snapshot

* adress review comments and set width for all save modals

* fix bug and improve typing

* fix classname
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@flash1293 flash1293 merged commit cffc08a into elastic:7.x Sep 6, 2019
@flash1293 flash1293 deleted the backport/7.x/pr-44261 branch September 6, 2019 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants