Skip to content

Commit 1d5cac8

Browse files
committed
uncomment aria-* attribute in JsxDOM
1 parent 2b8442b commit 1d5cac8

File tree

6 files changed

+34
-25
lines changed

6 files changed

+34
-25
lines changed

CHANGELOG.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@
1212
1313
# 10.1.0-rc.1
1414

15+
#### :boom: Breaking Change
16+
17+
- Change the DOM element attribute `method` to `method_`
18+
1519
## :rocket: New Feature
1620

1721
- Add extra variants for output filename suffixes in `bsconfig.json`: `.bs.mjs` and `.bs.cjs` are allowed https://github.com/rescript-lang/rescript-compiler/pull/5631
@@ -22,10 +26,6 @@
2226
- Fix location issue in make function in JSX V4 that breaks dead code elimination https://github.com/rescript-lang/syntax/pull/660
2327
- Fix parsing (hence pretty printing) of expressions with underscore `_` and comments.
2428

25-
#### :nail_care: Polish
26-
27-
- Remove type `JsxDOM.props`
28-
2929
# 10.1.0-alpha.2
3030

3131
#### :rocket: New Feature

jscomp/main/builtin_cmi_datasets.ml

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

jscomp/others/jsxDOM.ml

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,26 +39,31 @@ type domProps = {
3939
(* accessibility *)
4040
(* https://www.w3.org/TR/wai-aria-1.1/ *)
4141
(* https://accessibilityresources.org/<aria-tag> is a great resource for these *)
42-
(* [@bs.optional] [@bs.as "aria-current"] ariaCurrent: page|step|location|date|time|true|false, *)
42+
ariaCurrent: [`page | `step | `location | `date | `time | `_true | `_false]
43+
[@ns.optional] [@bs.as "aria-current"];
4344
ariaDetails: string
4445
[@ns.optional] [@bs.as "aria-details"];
4546
ariaDisabled: bool
4647
[@ns.optional] [@bs.as "aria-disabled"];
4748
ariaHidden: bool
4849
[@ns.optional] [@bs.as "aria-hidden"];
49-
(* [@ns.optional] [@bs.as "aria-invalid"] ariaInvalid: grammar|false|spelling|true, *)
50+
ariaInvalid: [`grammar | `spelling | `_true| `_false]
51+
[@ns.optional] [@bs.as "aria-invalid"];
5052
ariaKeyshortcuts: string
5153
[@ns.optional] [@bs.as "aria-keyshortcuts"];
5254
ariaLabel: string
5355
[@ns.optional] [@bs.as "aria-label"];
5456
ariaRoledescription: string
5557
[@ns.optional] [@bs.as "aria-roledescription"];
5658
(* Widget Attributes *)
57-
(* [@ns.optional] [@bs.as "aria-autocomplete"] ariaAutocomplete: inline|list|both|none, *)
58-
(* [@ns.optional] [@bs.as "aria-checked"] ariaChecked: true|false|mixed, https://www.w3.org/TR/wai-aria-1.1/#valuetype_tristate *)
59+
ariaAutocomplete: [`inline | `list | `both | `none]
60+
[@ns.optional] [@bs.as "aria-autocomplete"];
61+
ariaChecked: [`_true | `_false | `mixed] (* https://www.w3.org/TR/wai-aria-1.1/#valuetype_tristate *)
62+
[@ns.optional] [@bs.as "aria-checked"];
5963
ariaExpanded: bool
6064
[@ns.optional] [@bs.as "aria-expanded"];
61-
(* [@ns.optional] [@bs.as "aria-haspopup"] ariaHaspopup: false|true|menu|listbox|tree|grid|dialog, *)
65+
ariaHaspopup: [`menu | `listbox | `tree | `grid | `dialog | `_true | `_false]
66+
[@ns.optional] [@bs.as "aria-haspopup"];
6267
ariaLevel: int
6368
[@ns.optional] [@bs.as "aria-lavel"];
6469
ariaModal: bool
@@ -67,10 +72,12 @@ type domProps = {
6772
[@ns.optional] [@bs.as "aria-multiline"];
6873
ariaMultiselectable: bool
6974
[@ns.optional] [@bs.as "aria-multiselectable"];
70-
(* [@ns.optional] [@bs.as "aria-orientation"] ariaOrientation: horizontal|vertical|undefined, *)
75+
ariaOrientation: [`horizontal | `vertical | `undefined]
76+
[@ns.optional] [@bs.as "aria-orientation"];
7177
ariaPlaceholder: string
7278
[@ns.optional] [@bs.as "aria-placeholder"];
73-
(* [@ns.optional] [@bs.as "aria-pressed"] ariaPressed: true|false|mixed, https://www.w3.org/TR/wai-aria-1.1/#valuetype_tristate *)
79+
ariaPressed: [`_true | `_false| `mixed] (* https://www.w3.org/TR/wai-aria-1.1/#valuetype_tristate *)
80+
[@ns.optional] [@bs.as "aria-pressed"];
7481
ariaReadonly: bool
7582
[@ns.optional] [@bs.as "aria-readonly"];
7683
ariaRequired: bool
@@ -92,11 +99,13 @@ type domProps = {
9299
[@ns.optional] [@bs.as "aria-atomic"];
93100
ariaBusy: bool
94101
[@ns.optional] [@bs.as "aria-busy"];
95-
(* [@ns.optional] [@bs.as "aria-live"] ariaLive: off|polite|assertive|rude *)
102+
ariaLive: [`off | `polite | `assertive | `rude]
103+
[@ns.optional] [@bs.as "aria-live"];
96104
ariaRelevant: string
97105
[@ns.optional] [@bs.as "aria-relevant"];
98106
(* Drag-and-Drop Attributes *)
99-
(* [@ns.optional] [@bs.as "aria-dropeffect"] ariaDropeffect: copy|move|link|execute|popup|none *)
107+
ariaDropeffect: [`copy | `move | `link | `execute | `popup | `none]
108+
[@ns.optional] [@bs.as "aria-dropeffect"];
100109
ariaGrabbed: bool
101110
[@ns.optional] [@bs.as "aria-grabbed"];
102111
(* Relationship Attributes *)

lib/4.06.1/unstable/js_compiler.ml

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

lib/4.06.1/unstable/js_playground_compiler.ml

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

lib/4.06.1/whole_compiler.ml

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)