Bug 233281 - Web Inspector: Alignment swatch should handle multi-token values better
Summary: Web Inspector: Alignment swatch should handle multi-token values better
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Nikita Vasilyev
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-11-17 13:48 PST by Nikita Vasilyev
Modified: 2021-11-24 13:49 PST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nikita Vasilyev 2021-11-17 13:48:42 PST 
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.
Comment 1 Radar WebKit Bug Importer 2021-11-24 13:49:19 PST 
<rdar://problem/85733260>