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

Single-select Combobox is 2px taller than other form elements #6421

Closed
mitc7862 opened this issue Feb 3, 2023 · 7 comments
Closed

Single-select Combobox is 2px taller than other form elements #6421

mitc7862 opened this issue Feb 3, 2023 · 7 comments
Assignees
Labels
4 - verified Issues that have been released and confirmed resolved. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. c-combobox Issues that pertain to the calcite-combobox and related components design Issues that need design consultation prior to development. estimate - 2 Small fix or update, may require updates to tests. p - low Issue is non core or affecting less that 10% of people using the library ready for dev Issues ready for development implementation.

Comments

@mitc7862
Copy link

mitc7862 commented Feb 3, 2023

Actual Behavior

The Combobox component at medium scale is 34px tall, whereas all other form elements are 32px (at the same scale). This isn't a perceptible difference in vertical form layouts, but it makes alignment impossible for horizontal designs. I think this is a bug since scales seem to be based on multiples of 4.

image
(screenshot from Figma)

The same difference is apparent at small and large scales as well.

Expected Behavior

When the Combobox is set to single-select, its height should be the same as that of all other form elements with the same scale.

Reproduction Sample

https://codepen.io/MitchelCox/pen/GRBzMKa

Reproduction Steps

Place Combobox alongside other form elements
Use flex to position them horizontally
Ensure that the combo box is set to single-select

Reproduction Version

1.0.3

Relevant Info

No response

Regression?

No response

Impact

As described above, this issue affects any application where horizontal alignment is important. This is not a functional bug, so it's not high-impact. However, the single-select Combobox is one of the most-used form elements in Web Analysis and is sometimes shown side-by-side with other inputs.

Esri team

ArcGIS Map Viewer

@mitc7862 mitc7862 added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Feb 3, 2023
@github-actions github-actions bot added the ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. label Feb 3, 2023
@alisonailea alisonailea added needs triage Planning workflow - pending design/dev review. airtable and removed needs triage Planning workflow - pending design/dev review. airtable labels Feb 11, 2023
@macandcheese macandcheese added the c-combobox Issues that pertain to the calcite-combobox and related components label Mar 22, 2023
@brittneytewks brittneytewks added design Issues that need design consultation prior to development. p - low Issue is non core or affecting less that 10% of people using the library estimate - 2 Small fix or update, may require updates to tests. needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. and removed needs triage Planning workflow - pending design/dev review. labels Apr 11, 2023
@geospatialem geospatialem added this to the 2023 May Priorities milestone Apr 11, 2023
@geospatialem geospatialem added 1 - assigned Issues that are assigned to a sprint and a team member. and removed needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. 0 - new New issues that need assignment. labels Apr 11, 2023
@ashetland
Copy link
Contributor

@geospatialem Another add for the Combobox Redesign Epic.

@ashetland
Copy link
Contributor

Proposed updates to match Combobox height to other inputs:

  • For single-select, heights should be reduced by 2px for no selection and selected.
  • For multi-select, heights should be reduced by 2px when no selection is made. Combobox should be allowed to grow in height as it currently does when selections are made.

See image:
image

@ashetland ashetland added ready for dev Issues ready for development implementation. and removed design Issues that need design consultation prior to development. labels May 1, 2023
@ashetland ashetland removed their assignment May 1, 2023
@geospatialem geospatialem added the 0 - new New issues that need assignment. label May 1, 2023
@geospatialem geospatialem removed this from the 2023 May Priorities milestone May 1, 2023
@geospatialem geospatialem added the needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. label May 1, 2023
@macandcheese
Copy link
Contributor

We'll be able to use the new focus offset css variable to prevent the overlap of focus from Chips.

@ashetland ashetland added the figma changes Issues that require additions or updates to the Figma UI Kit where no `design` label exists label May 10, 2023
@ashetland
Copy link
Contributor

We'll be able to use the new focus offset css variable to prevent the overlap of focus from Chips.

Do we need to use the focus offset variable to correct anything? The idea with what's outlined above is that once chips populate the input, it would grow in height as it currently does. The spec'd update to the height is only to the initial unpopulated height of the input.

@macandcheese
Copy link
Contributor

Yeah, maybe we could bump the y-padding on the combobox when populated - if its growing in height anyway it could have some more room. It could also be balanced with the padding to the left border of the input.

@geospatialem geospatialem added this to the 2023 August Priorities milestone Jun 22, 2023
@geospatialem geospatialem removed the needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. label Jun 22, 2023
@jcfranco jcfranco self-assigned this Aug 21, 2023
@jcfranco jcfranco added 2 - in development Issues that are actively being worked on. and removed 0 - new New issues that need assignment. labels Aug 21, 2023
jcfranco added a commit that referenced this issue Aug 23, 2023
@jcfranco jcfranco added 3 - installed Issues that have been merged to master branch and are ready for final confirmation. and removed 2 - in development Issues that are actively being worked on. labels Aug 23, 2023
@github-actions
Copy link
Contributor

Installed and assigned for verification.

@geospatialem geospatialem added 4 - verified Issues that have been released and confirmed resolved. and removed 3 - installed Issues that have been merged to master branch and are ready for final confirmation. labels Aug 27, 2023
@geospatialem
Copy link
Member

Verified in 1.7.0-next.9.

image

@ashetland ashetland added the design Issues that need design consultation prior to development. label Sep 18, 2023
@SkyeSeitz SkyeSeitz removed the figma changes Issues that require additions or updates to the Figma UI Kit where no `design` label exists label Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - verified Issues that have been released and confirmed resolved. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. c-combobox Issues that pertain to the calcite-combobox and related components design Issues that need design consultation prior to development. estimate - 2 Small fix or update, may require updates to tests. p - low Issue is non core or affecting less that 10% of people using the library ready for dev Issues ready for development implementation.
Projects
None yet
Development

No branches or pull requests

8 participants