RESOLVED FIXED 234675
Web Inspector: `shape-outside` description doesn't make sense 
https://bugs.webkit.org/show_bug.cgi?id=234675
Summary Web Inspector: `shape-outside` description doesn't make sense
Tim Horton
Reported 2021-12-24 15:27:53 PST
CSSDocumentation.js says: "shape-outside": { "description": "Specifies an orthogonal rotation to be applied to an image before it is laid out.", "syntax": "none | [ <shape-box> || <basic-shape> ] | <image>", "url": "https://developer.mozilla.org/docs/Web/CSS/shape-outside" }, I don't think that's a valid description of shape-outside?
Attachments
[fast-cq] Patch 1.0 (4.87 KB, patch)
2022-01-05 10:34 PST, Razvan Caliman 		no flags
DetailsFormatted DiffDiff
[fast-cq] Patch 1.0 (4.92 KB, patch)
2022-01-06 05:38 PST, Razvan Caliman 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-12-31 15:28:18 PST
<rdar://problem/87036546>
Razvan Caliman
Comment 2 2022-01-05 10:34:38 PST
Created attachment 448402 [details] [fast-cq] Patch 1.0 Add the ability to overwrite individual CSS property documentation fields with data from local JSON file.
Devin Rousso
Comment 3 2022-01-05 12:27:30 PST
Comment on attachment 448402 [details] [fast-cq] Patch 1.0 r=me, nice solution :)
Simon Fraser (smfr)
Comment 4 2022-01-05 15:25:52 PST
Are we going to get the VSCode description fixed?
Simon Fraser (smfr)
Comment 5 2022-01-05 15:27:02 PST
Seems like we should file an issue at https://github.com/microsoft/vscode-custom-data/issues
EWS
Comment 6 2022-01-06 05:13:21 PST
/Volumes/Data/worker/Commit-Queue/build/Source/WebInspectorUI/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).
Razvan Caliman
Comment 7 2022-01-06 05:38:29 PST
Created attachment 448490 [details] [fast-cq] Patch 1.0 Carry over R+; fix missing reviewer from Changelog
EWS
Comment 8 2022-01-06 05:42:13 PST
Committed r287682 (245778@main): <https://commits.webkit.org/245778@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 448490 [details].
Note You need to log in before you can comment on or make changes to this bug.