[Webkit-unassigned] [Bug 217447] Web Inspector: update styles to use CSS properties with neutral directionality
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Oct 21 17:42:42 PDT 2020
https://bugs.webkit.org/show_bug.cgi?id=217447
Devin Rousso <drousso at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #412052|review? |review-
Flags| |
--- Comment #4 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 412052
--> https://bugs.webkit.org/attachment.cgi?id=412052
Update styles to use CSS properties with neutral directionality
View in context: https://bugs.webkit.org/attachment.cgi?id=412052&action=review
r- for the reasons below
I think you're missing a lot of other `body[dir=*]` rules :(
I'd also suggest updating `-webkit-margin-*`/`-webkit-padding-*` to use `margin-inline-*`/`padding-inline-*` instead
> Source/WebInspectorUI/UserInterface/Debug/UncaughtExceptionReporter.css:80
> + float: inline-start;
Safari doesn't support `inline-*` for `float`
ditto in other files
> Source/WebInspectorUI/UserInterface/Views/CPUTimelineView.css:119
> border-right: var(--cpu-timeline-view-overview-divider-border-end);
Why are we keeping this?
> Source/WebInspectorUI/UserInterface/Views/CPUTimelineView.css:120
> + border-block-end: var(--cpu-timeline-view-overview-divider-border-end);
This should be `-inline-` since it's left/right (we only support a horizontal writing mode, so `-inline-` will always be left/right and `-block-` will always be top/bottom).
Also, please remove the variable and inline its value now that it's only used in one place.
ditto in other files
> Source/WebInspectorUI/UserInterface/Views/FindBanner.css:100
> + border-start-start-radius: var(--find-banner-search-box-border-radius-start);
> + border-end-start-radius: var(--find-banner-search-box-border-radius-start);
> + border-start-end-radius: var(--find-banner-search-box-border-radius-end);
> + border-end-end-radius: var(--find-banner-search-box-border-radius-end);
??? these are not CSS properties
ditto in other files
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20201022/5cbebdfe/attachment.htm>
More information about the webkit-unassigned
mailing list