Skip to content

GAUD-5470: List item illustration gap is too large#6716

Open
EdwinACL831 wants to merge 5 commits intomainfrom
ecollazos/GAUD-5470_list_item_illustration_gap
Open

GAUD-5470: List item illustration gap is too large#6716
EdwinACL831 wants to merge 5 commits intomainfrom
ecollazos/GAUD-5470_list_item_illustration_gap

Conversation

@EdwinACL831
Copy link
Copy Markdown
Contributor

@EdwinACL831 EdwinACL831 commented Mar 26, 2026

Jira

GAUD-5470

Follow these designs
image

Issue

  • The gap between the illustration and its content was too large
  • The gap between the checkbox and the illustration was too large

Solution

  • Removes the CSS rules that were overridding the --d2l-list-item-illustration-margin-inline-end from 0.9rem (18px) to 1rem (20px). This is something that we are trying to get rid off since we want the consumer to actually style the elements withing the list item
  • Shifts to the left 0.1rem (2px) to match the 18px gaps between the checkbox and the illustration

Removes the CSS rules that were overridding the
--d2l-list-item-illustration-margin-inline-end from 0.9rem to 1rem
This is something that we are trying to get rid off since we want the
consumer to actually style the elements withing the list item
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/core/pr-6716/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

Edwin Collazos and others added 2 commits March 26, 2026 10:15
…tem's grid and the d2l-list-item-content too
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@EdwinACL831 EdwinACL831 marked this pull request as ready for review March 26, 2026 15:29
@EdwinACL831 EdwinACL831 requested a review from a team as a code owner March 26, 2026 15:29
@dbatiste
Copy link
Copy Markdown
Contributor

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.

2 participants