Skip to content

Conversation

@GomezIvann
Copy link
Collaborator

@GomezIvann GomezIvann commented Nov 3, 2022

Checklist
(Check off all the items before submitting)

  • Build process is done without errors and all tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Description
To solve this issue, I just had to add position: relative to the checkbox parent container, so the position: absolute of the tick is calculated based on the actual positioning of the component as a whole. When an element is has position absolute, it is positioned relative to its closest positioned ancestor, if any; otherwise, it is placed relative to the initial containing block.

Screenshots

image

image

Closes #1355

@GomezIvann GomezIvann changed the title Fix Checkbox tick sticky positioning Fix Checkbox tick with sticky positioning Nov 3, 2022
@GomezIvann GomezIvann marked this pull request as ready for review November 3, 2022 12:51
@Jialecl Jialecl self-requested a review November 3, 2022 13:43
@Jialecl Jialecl self-assigned this Nov 3, 2022
@GomezIvann GomezIvann requested a review from Jialecl November 10, 2022 08:37
@Jialecl Jialecl merged commit a5074e7 into master Nov 11, 2022
@Jialecl Jialecl deleted the gomezivann-checkbox-tickFix branch November 11, 2022 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Checkbox position in scrollable multiselect

3 participants