Dsw 3992 light dom

[jamieomaguire]

ignore for now please

[changeset-bot]

🦋 Changeset detected

Latest commit: 53e7e0f

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

This PR includes changesets to release 4 packages

Name	Type
@justeattakeaway/pie-lite-radio	Minor
@justeattakeaway/pie-webc	Minor
@justeattakeaway/pie-storybook	Minor
@justeattakeaway/pie-docs	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

[pie-design-system-app]

	Fails
🚫	❌ PR Title Validation Failed The PR title should follow the same convention used for commits, e.g.: type(scope): TICKET-123 title where TICKET-123 is a valid ticket ID. Note: Ticket IDs cannot be all zeros (e.g. DSW-000 is not allowed). Current title: Dsw 3992 light dom

Generated by 🚫 dangerJS against 53e7e0f

[ashleynolan]

This is a really nice pattern 😄

Q – could this use a default selector if it were to be radio specific?

@mixin radio-slotted-input-base($selector: 'input[type="radio"]') {

and then called just with:

@include radio-slotted-input-base();

Or could even set it as an optional variable, so that all the mixins can make use of it, and it be overridden by the user if they needed to:

$pRadioSelector: 'input[type="radio"]' !default;

@mixin radio-slotted-input-base() {
	::slotted(#{$pRadioSelector}) {
        …
    }
}

[jamieomaguire]

yeah you totally could add a default value! both are quite smooth ideas