Conversation
- adds `window-drag` to css-ui-4
- changes the computed value - updates line width styling
Updates the spec text with the naming resolution by the CSSWG. w3c#13102
|
Hi @emilio! I noticed you were reviewing Diego's original PR- I've applied the edits you had asked for there. Do you mind taking a look at this one as well? |
emilio
left a comment
emilio
left a comment
There was a problem hiding this comment.
Does this property have an effect on event dispatch? I'm assuming that elements with window-drag: move will not get "web" drag events.
The spec should say what the expectation is for something like <div draggable=true style="window-drag:move">.
Looks good to me with these things documented and the questions below asked.
css-ui-4/Overview.bs
Outdated
| <dl dfn-for=window-drag dfn-type=value> | ||
| <dt><dfn>auto</dfn> | ||
| <dd> | ||
| Represents a UA-chosen default behavior, generally ''move/none''. |
There was a problem hiding this comment.
Is there any time where none will not be the auto behavior? If so it should be called out. If not... Maybe the initial value should be just none?
There was a problem hiding this comment.
I actually don't think it makes sense for the auto behavior to ever not be none. I went ahead and removed auto in lieu of this and made none the initial value. (Please let me know if you think we should keep the auto value).
|
I've added a new text blurb that clarifies event dispatch behavior, as well as clarifying |
emilio
left a comment
emilio
left a comment
There was a problem hiding this comment.
Looks reasonable to me, but an editor of this spec should also review it :)
|
@tabatkins @frivoal Hi Editors! Could I ask one of you to take a look at this spec text for |
|
@fantasai Noticed you put the following in the older PR:
Do you think specifying something like-
-would be clear enough? |
5 participants
This change builds off of @diekus' PR to add the
window-drag(formerlyapp-region) CSS property to the css-ui-4 spec. It makes requested edits from the previous PR, as well as adds a note mentioningapp-regionas an alias as a result of the new resolution at #13102.This comes out of resolutions #13102 and
#13101 taken on 21/01/26.