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

Form fields not aligned in chart classification #9726

Closed
1 task done
tdipisa opened this issue Nov 22, 2023 · 4 comments · Fixed by #9731 or #9742
Closed
1 task done

Form fields not aligned in chart classification #9726

tdipisa opened this issue Nov 22, 2023 · 4 comments · Fixed by #9731 or #9742

Comments

@tdipisa
Copy link
Member

tdipisa commented Nov 22, 2023

Description

Classification fields (class value drop down) are no longer aligned properly in the new chart classification panel. Release 2023.02.00 not affected!

How to reproduce

  • Open this map
  • Open the chart editor of US Population chart widget
  • Switch to Persons Trace
  • Open the Custom Color Settings panel of Color ramp property (see screenshot below)

Expected Result

Class value drop down are properly aligned with other fields of the same row

Current Result

Class value drop down are not properly aligned with other fields of the same row

image

Also QA is affected for the same form fields even if in master there has been an huge refactor of charts, therefore most probably the cause is the same.

image

  • Not browser related
Browser info (use this site: https://www.whatsmybrowser.org/ for non expert users)
Browser Affected Version
Internet Explorer
Edge
Chrome
Firefox
Safari

Other useful information

We need to understand what introduced this regression in QA.

@MV88
Copy link
Contributor

MV88 commented Nov 22, 2023

@tdipisa i think this issue is similar to #9728 , we can test this again after merging this #9731

@tdipisa
Copy link
Member Author

tdipisa commented Nov 22, 2023

@MV88 ok let's check and confirm once deployed.

@MV88 MV88 linked a pull request Nov 23, 2023 that will close this issue
12 tasks
@MV88
Copy link
Contributor

MV88 commented Nov 23, 2023

closing it because it is fixed in DEV

@MV88 MV88 closed this as completed Nov 23, 2023
@MV88
Copy link
Contributor

MV88 commented Nov 23, 2023

@ElenaGallo please double test this in DEV

@MV88 MV88 added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 23, 2023
@ElenaGallo ElenaGallo self-assigned this Nov 23, 2023
@ElenaGallo ElenaGallo removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment