Skip to content

Handle disabled case when computing menu item background color #4614

Handle disabled case when computing menu item background color

Handle disabled case when computing menu item background color #4614

Triggered via pull request November 4, 2024 20:27
Status Failure
Total duration 54s
Artifacts

lint-changed.yml

on: pull_request
Changed files ESLint check
46s
Changed files ESLint check
Fit to window
Zoom out
Zoom in

Annotations

4 errors
Changed files ESLint check: src/components/PopoverMenu.tsx#L267
Replace `!!item.isSelected,·focusedIndex·===·menuIndex,·item.disabled·??·false,·theme.activeComponentBG,·theme.hoverComponentBG` with `⏎························!!item.isSelected,⏎························focusedIndex·===·menuIndex,⏎························item.disabled·??·false,⏎························theme.activeComponentBG,⏎························theme.hoverComponentBG,⏎····················`
Changed files ESLint check: src/components/SelectionList/BaseListItem.tsx#L101
Replace `pressableStyle,·isFocused·&&·StyleUtils.getItemBackgroundColorStyle(!!item.isSelected,·!!isFocused,·item.isDisabled,·theme.activeComponentBG,·theme.hoverComponentBG)` with `⏎····················pressableStyle,⏎····················isFocused·&&·StyleUtils.getItemBackgroundColorStyle(!!item.isSelected,·!!isFocused,·item.isDisabled,·theme.activeComponentBG,·theme.hoverComponentBG),⏎················`
Changed files ESLint check: src/components/SelectionList/BaseListItem.tsx#L107
Replace `·style={[wrapperStyle,·isFocused·&&·StyleUtils.getItemBackgroundColorStyle(!!item.isSelected,·!!isFocused,·item.isDisabled,·theme.activeComponentBG,·theme.hoverComponentBG)]}` with `⏎····················style={[⏎························wrapperStyle,⏎························isFocused·&&·StyleUtils.getItemBackgroundColorStyle(!!item.isSelected,·!!isFocused,·item.isDisabled,·theme.activeComponentBG,·theme.hoverComponentBG),⏎····················]}⏎················`
Changed files ESLint check
Process completed with exit code 1.