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

[Maps] Cannot select Solid fill-color when removing fields #70601

Closed
thomasneirynck opened this issue Jul 2, 2020 · 2 comments · Fixed by #70621
Closed

[Maps] Cannot select Solid fill-color when removing fields #70601

thomasneirynck opened this issue Jul 2, 2020 · 2 comments · Fixed by #70621
Assignees
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.9.0 v8.0.0

Comments

@thomasneirynck
Copy link
Contributor

The style-editor gets in an unusable state with this interaction pattern:

Steps:

  • add a vector tile layer
  • add a field in the source-settings
  • do by-value styling on that field for fill-color
  • remove that field from the source-settings

-> The "by value/solid" drop-down is disabled

image

@thomasneirynck thomasneirynck added bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation labels Jul 2, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@nreese
Copy link
Contributor

nreese commented Jul 2, 2020

VectorStyle.getDescriptorWithMissingStylePropsRemoved needs to be updated to convert dynamic style to static style when source updates result in no fields. I am working on a fix

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 [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants