Skip to content

Commit

Permalink
fix: review
Browse files Browse the repository at this point in the history
  • Loading branch information
dauriamarco committed Oct 9, 2023
1 parent 6af415d commit 9bf123a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import './sbb-radio-button-group';

describe('sbb-radio-button-group', () => {
it('renders', async () => {
const root = await fixture(html`<sbb-radio-button-group />`);
const root = await fixture(html`<sbb-radio-button-group></sbb-radio-button-group>`);

expect(root).dom.to.be.equal(
`
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ export class SbbRadioButtonGroup extends LitElement {
return this._allowEmptySelection;
}
public set allowEmptySelection(value: boolean) {
// TODO: Validate logic
const oldValue = this._allowEmptySelection;
this._allowEmptySelection = value;
this._updateAllowEmptySelection();
Expand All @@ -54,7 +53,6 @@ export class SbbRadioButtonGroup extends LitElement {
return this._disabled;
}
public set disabled(value: boolean) {
// TODO: Validate logic
const oldValue = this._disabled;
this._disabled = value;
this._updateDisabled();
Expand All @@ -70,7 +68,6 @@ export class SbbRadioButtonGroup extends LitElement {
return this._required;
}
public set required(value: boolean) {
// TODO: Validate logic
const oldValue = this._required;
this._required = value;
this._updateRequired();
Expand All @@ -86,7 +83,6 @@ export class SbbRadioButtonGroup extends LitElement {
return this._value;
}
public set value(value: any | null | null) {
// TODO: Validate logic
const oldValue = this._value;
this._value = value;
this._valueChanged(this._value);
Expand All @@ -102,7 +98,6 @@ export class SbbRadioButtonGroup extends LitElement {
return this._size;
}
public set size(value: InterfaceSbbRadioButtonGroupAttributes['size']) {
// TODO: Validate logic
const oldValue = this._size;
this._size = value;
this._updateSize();
Expand Down Expand Up @@ -311,7 +306,7 @@ export class SbbRadioButtonGroup extends LitElement {

return html`
<div class="sbb-radio-group">
<slot @slotchange=${() => this._updateRadios()} />
<slot @slotchange=${() => this._updateRadios()}></slot>
</div>
${this._namedSlots.error
? html`<div class="sbb-radio-group__error">
Expand Down

0 comments on commit 9bf123a

Please sign in to comment.