Bug 233281
| Summary: | Web Inspector: Alignment swatch should handle multi-token values better | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> |
| Component: | Web Inspector | Assignee: | Nikita Vasilyev <nvasilyev> |
| Status: | ASSIGNED | ||
| Severity: | Normal | CC: | inspector-bugzilla-changes, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | All | ||
| OS: | All | ||
Nikita Vasilyev
Bug 230065: Web Inspector: Add a swatch for align-content
For this case:
align-content: unsafe center
The alignment inline swatch doesn't show that "center" is selected.
We could detect that properly.
Doing that, however, introduces complications. (This bug is rather an open discussion rather than a set of steps to do.)
What if you click on the "space-between" icon? It doesn't support "unsafe". `align-content: unsafe space-between`
isn't valid.
Let's look at another case:
align-content: first baseline
Currently, we display the inline swatch even though `first baseline` (or even `baseline`) isn't present in the alignment editor popover.
I think it's good. If we change multi-token behavior, I'd like to continue displaying the alignment swatch for this case.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/85733260>