Skip to content

Conversation

@lizozom
Copy link
Contributor

@lizozom lizozom commented Feb 21, 2019

Fixing Issues with new Listing component #31599

@lizozom
Copy link
Contributor Author

lizozom commented Feb 21, 2019

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@lizozom lizozom added the v8.0.0 label Feb 21, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💔 Build Failed

@lizozom
Copy link
Contributor Author

lizozom commented Feb 25, 2019

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Tested on Chrome Linux, looks good to me. Code LGTM

@timroes timroes removed the v7.0.1 label Feb 25, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@lizozom lizozom merged commit 209069e into elastic:master Feb 25, 2019
lizozom added a commit to lizozom/kibana that referenced this pull request Feb 25, 2019
* Fixing bugs from elastic#31599
* Make listing item links instead of buttons
* restore dashboard edit button behavior
lizozom added a commit that referenced this pull request Feb 26, 2019
* Fixing bugs from #31599
* Make listing item links instead of buttons
* restore dashboard edit button behavior
@lizozom lizozom deleted the visualization-list-eui branch March 3, 2019 16:31
@cchaos cchaos added EUI and removed Feature:EUI labels Aug 18, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/eui-design (EUI)

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

Labels

bug Fixes for quality problems that affect the customer experience EUI v7.2.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants