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

Improve UX of Visual Style Editor for WFS/Vector layers #8935

Closed
1 of 6 tasks
allyoucanmap opened this issue Jan 26, 2023 · 0 comments · Fixed by #8955
Closed
1 of 6 tasks

Improve UX of Visual Style Editor for WFS/Vector layers #8935

allyoucanmap opened this issue Jan 26, 2023 · 0 comments · Fixed by #8955

Comments

@allyoucanmap
Copy link
Contributor

allyoucanmap commented Jan 26, 2023

Description

There are some fields inside the visual style editor available for 2D but not for 3D and vice-versa. The current solution is to hide the field not available in a specific 2D/3D viewer but it's confusing for user that are not able to find those properties. We should instead keep them visible with a disabled state.

An example of current state

image

This could be a possible solution (we could think on something better while developing)

image

Also the symbolizer not available (eg: 3D model) should be disabled but visible

image

All symbolizer should be reviewed

What kind of improvement you want to add? (check one with "x", remove the others)

  • Minor changes to existing features
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Other useful information

@allyoucanmap allyoucanmap added this to the 2023.01.00 milestone Jan 26, 2023
@tdipisa tdipisa assigned allyoucanmap and unassigned tdipisa Jan 31, 2023
@tdipisa tdipisa changed the title Improve UX of Visula Style Editor for WFS/Vector layers Improve UX of Visual Style Editor for WFS/Vector layers Feb 3, 2023
allyoucanmap added a commit to allyoucanmap/MapStore2 that referenced this issue Feb 7, 2023
@allyoucanmap allyoucanmap linked a pull request Feb 7, 2023 that will close this issue
5 tasks
@tdipisa tdipisa removed their assignment Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants