Change Details

From T343591#9117901 [] filter menu button should follow the toggle button pattern defined in the WAI APG by adding role="button" and :aria-pressed="settings.controlMenuOpen" [] Add &:hover { cursor: @cursor-base--hover; ) to interactive elements, especially in the filters popup. CdxRadios already have this (pointer cursor when hovering over the input or the label), so unifying that behavior across the inputs would be nice. [] ListFilterMenu has a couple uses of the deprecated type prop of CdxButton, recommend changing to weight (although I'd also recommend not having two primary buttons)

DST = Design Systems Team (https://www.mediawiki.org/wiki/Design_Systems_Team) From T343591#9117901 [] filter menu button should follow the toggle button pattern defined in the WAI APG by adding role="button" and :aria-pressed="settings.controlMenuOpen" [] Add &:hover { cursor: @cursor-base--hover; ) to interactive elements, especially in the filters popup. CdxRadios already have this (pointer cursor when hovering over the input or the label), so unifying that behavior across the inputs would be nice. [] ListFilterMenu has a couple uses of the deprecated type prop of CdxButton, recommend changing to weight (although I'd also recommend not having two primary buttons)

DST = Design Systems Team (https://www.mediawiki.org/wiki/Design_Systems_Team) From T343591#9117901 [] filter menu button should follow the toggle button pattern defined in the WAI APG by adding role="button" and :aria-pressed="settings.controlMenuOpen" [] Add &:hover { cursor: @cursor-base--hover; ) to interactive elements, especially in the filters popup. CdxRadios already have this (pointer cursor when hovering over the input or the label), so unifying that behavior across the inputs would be nice. [] ListFilterMenu has a couple uses of the deprecated type prop of CdxButton, recommend changing to weight (although I'd also recommend not having two primary buttons)