improve a11y

docs/resources/changelog.md

@@ -8,7 +8,10 @@ _During the beta period, these restrictions may be relaxed in the event of a mis
 
 ## Next
 
+- Fixed a bug in `sl-tooltip` where events weren't properly cleaned up on disconnect
+- Fixed a bug in `sl-tooltip` where they wouldn't display after toggling `disabled` off and on again [#391](https://github.com/shoelace-style/shoelace/issues/391)
 - Improved a11y for disabled buttons that are rendered as links
+- Improved a11y for `sl-button-group`
 
 ## 2.0.0-beta.36
 
@@ -19,8 +22,6 @@ _During the beta period, these restrictions may be relaxed in the event of a mis
 - Fixed a bug where `sl-color-picker` could be opened when disabled
 - Fixed a bug in `sl-color-picker` that caused erratic slider behaviors [#388](https://github.com/shoelace-style/shoelace/issues/388) [#389](https://github.com/shoelace-style/shoelace/issues/389)
 - Fixed a bug where `sl-details` wouldn't always render the correct height when open initially [#357](https://github.com/shoelace-style/shoelace/issues/357)
-- Fixed a bug in `sl-tooltip` where events weren't properly cleaned up on disconnect
-- Fixed a bug in `sl-tooltip` where they wouldn't display after toggling `disabled` off and on again [#391](https://github.com/shoelace-style/shoelace/issues/391)
 - Renamed `components.json` to `metadata.json`
 - Updated to the prerelease versions of LitElement and lit-html
 - Updated to Bootstrap Icons 1.4.1

src/components/button-group/button-group.ts

@@ -15,7 +15,7 @@ export default class SlButtonGroup extends LitElement {
   static styles = unsafeCSS(styles);
 
   /** A label to use for the button group's `aria-label` attribute. */
-  @property() label: string;
+  @property() label = '';
 
   handleFocus(event: CustomEvent) {
     const button = event.target as HTMLElement;
@@ -32,6 +32,7 @@ export default class SlButtonGroup extends LitElement {
       <div
         part="base"
         class="button-group"
+        role="group"
         aria-label=${this.label}
         @focusout=${this.handleBlur}
         @focusin=${this.handleFocus}