Skip to content

Commit a994169

Browse files
committed
fix: address review feedback
1 parent 0aceb21 commit a994169

4 files changed

Lines changed: 12 additions & 8 deletions

File tree

.github/workflows/react-component-ci.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,5 @@ permissions:
55
jobs:
66
test:
77
uses: react-component/rc-test/.github/workflows/test-utoo.yml@main
8-
secrets: inherit
8+
secrets:
9+
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

README.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,21 +65,21 @@ Online preview: https://slider.react-component.vercel.app/
6565
| `ariaLabelForHandle` | string \| string[] | - | `aria-label` for handle elements. |
6666
| `ariaLabelledByForHandle` | string \| string[] | - | `aria-labelledby` for handle elements. |
6767
| `ariaRequired` | boolean | - | `aria-required` for handle elements. |
68-
| `ariaValueTextFormatterForHandle` | `(value: number) => string` \| Array function | - | Formatter for `aria-valuetext`. |
68+
| `ariaValueTextFormatterForHandle` | `((value: number) => string) \| ((value: number) => string)[]` | - | Formatter for `aria-valuetext`. |
6969
| `autoFocus` | boolean | - | Focus the slider on mount. |
7070
| `className` | string | - | Additional class name. |
71-
| `classNames` | Partial<Record<'tracks' \| 'track' \| 'rail' \| 'handle', string>> | - | Semantic class names for internal slots. |
71+
| `classNames` | `Partial<Record<'tracks' \| 'track' \| 'rail' \| 'handle', string>>` | - | Semantic class names for internal slots. |
7272
| `count` | number | - | Deprecated. Use `range.minCount` or `range.maxCount`. |
7373
| `defaultValue` | number \| number[] | - | Initial value. |
7474
| `disabled` | boolean \| boolean[] | false | Disable all handles or specific handles. |
75-
| `dots` | boolean | false | Show dots when `step` is greater than 1. |
75+
| `dots` | boolean | false | Show dots at each valid step position. |
7676
| `dotStyle` | React.CSSProperties \| `(dotValue: number) => React.CSSProperties` | - | Style for dots. |
7777
| `handleRender` | `HandlesProps['handleRender']` | - | Custom handle renderer. |
7878
| `handleStyle` | React.CSSProperties \| React.CSSProperties[] | - | Deprecated. Use `styles.handle`. |
7979
| `id` | string | - | Root id. |
8080
| `included` | boolean | true | Whether the track is rendered as a continuous selected interval. |
8181
| `keyboard` | boolean | true | Enable keyboard interaction. |
82-
| `marks` | Record<string \| number, ReactNode \| MarkObj> | - | Slider marks. |
82+
| `marks` | `Record<string \| number, ReactNode \| MarkObj>` | - | Slider marks. |
8383
| `max` | number | 100 | Maximum value. |
8484
| `min` | number | 0 | Minimum value. |
8585
| `onAfterChange` | `(value) => void` | - | Deprecated. Use `onChangeComplete`. |
@@ -96,7 +96,7 @@ Online preview: https://slider.react-component.vercel.app/
9696
| `startPoint` | number | `min` | Start point for a single-value track. |
9797
| `step` | number \| null | 1 | Step size. Use `null` to use marks as steps. |
9898
| `style` | React.CSSProperties | - | Root style. |
99-
| `styles` | Partial<Record<'tracks' \| 'track' \| 'rail' \| 'handle', React.CSSProperties>> | - | Semantic styles for internal slots. |
99+
| `styles` | `Partial<Record<'tracks' \| 'track' \| 'rail' \| 'handle', React.CSSProperties>>` | - | Semantic styles for internal slots. |
100100
| `tabIndex` | number \| number[] | 0 | Handle tab index. |
101101
| `track` | boolean | - | Whether to render track elements. |
102102
| `trackStyle` | React.CSSProperties \| React.CSSProperties[] | - | Deprecated. Use `styles.track`. |

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
"coverage": "rc-test --coverage",
3535
"docs:build": "dumi build",
3636
"docs:deploy": "gh-pages -d docs-dist",
37-
"gh-pages": "GH_PAGES=1 npm run compile && npm run docs:build && npm run docs:deploy",
37+
"gh-pages": "cross-env GH_PAGES=1 npm run compile && npm run docs:build && npm run docs:deploy",
3838
"lint": "eslint src/ --ext .ts,.tsx,.jsx,.js,.md",
3939
"prepublishOnly": "npm run compile && rc-np",
4040
"prettier": "prettier --write --ignore-unknown .",

tsconfig.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,10 @@
1010
"paths": {
1111
"@/*": ["src/*"],
1212
"@@/*": ["src/.umi/*"],
13-
"@rc-component/slider": ["src/index.tsx"]
13+
"@rc-component/slider": ["src/index.tsx"],
14+
"@rc-component/slider/es": ["src"],
15+
"@rc-component/slider/es/*": ["src/*"],
16+
"@rc-component/slider/assets/*": ["assets/*"]
1417
}
1518
},
1619
"include": [".dumirc.ts", "src", "tests", "docs/examples"],

0 commit comments

Comments
 (0)