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

feat(pie-icons-webc): DSW-000 set height and width on icon hosts #1367

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jamieomaguire
Copy link
Contributor

@jamieomaguire jamieomaguire commented Mar 28, 2024

Describe your changes (can list changeset entries if preferable)

Applies height and width to web component host along with a display inline block. Otherwise, the web component hosts are incorrectly sized. Image attached to demo

Screenshot 2024-03-28 at 15 14 02

Author Checklist (complete before requesting a review)

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests
  • If it is a PIE Docs change, I have reviewed the Docs site preview
  • If it is a component change, I have reviewed the Storybook preview
  • If there are visual test updates, I have reviewed them properly before approving

Reviewer checklists (complete before approving)

Reviewer 1

  • If it is a PIE Docs change, I have reviewed the PR preview
  • If there are visual test updates, I have reviewed them

Reviewer 2

  • If it is a PIE Docs change, I have reviewed the PR preview
  • If there are visual test updates, I have reviewed them

Copy link

changeset-bot bot commented Mar 28, 2024

🦋 Changeset detected

Latest commit: 03e9232

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages
Name Type
@justeattakeaway/pie-icons-webc Minor
wc-vanilla Patch
pie-storybook Patch
@justeattakeaway/pie-assistive-text Patch
@justeattakeaway/pie-chip Patch
@justeattakeaway/pie-icon-button Patch
@justeattakeaway/pie-modal Patch
@justeattakeaway/pie-notification Patch
@justeattakeaway/pie-switch Patch
@justeattakeaway/pie-input Patch
@justeattakeaway/pie-cookie-banner Patch
@justeattakeaway/pie-webc Patch
wc-next10 Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the tools label Mar 28, 2024
@pie-design-system-bot
Copy link
Contributor

pie-design-system-bot commented Mar 28, 2024


Fails
🚫 You currently have an unchecked checklist item in your PR description.

Please confirm this check has been carried out – if it's not relevant to your PR, delete this line from the PR checklist.

Generated by 🚫 dangerJS against 03e9232

dandel10n
dandel10n previously approved these changes Mar 29, 2024
Copy link
Contributor

@dandel10n dandel10n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, the only comment I have is about the comment that would give more context about current implementation

…low up) (#1368)

feat(pie-icons-webc): DSW-000 removes --icon-display-override var

--icon-display-override var is not needed after adding display inline-block to a host
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants