Skip to content

[css-ui-4] add the window-drag text#13801

Merged
tabatkins merged 12 commits intow3c:mainfrom
kyerebo:window-drag
May 6, 2026
Merged

[css-ui-4] add the window-drag text#13801
tabatkins merged 12 commits intow3c:mainfrom
kyerebo:window-drag

Conversation

@kyerebo
Copy link
Copy Markdown
Contributor

@kyerebo kyerebo commented Apr 13, 2026

This change builds off of @diekus' PR to add the window-drag (formerly app-region) CSS property to the css-ui-4 spec. It makes requested edits from the previous PR, as well as adds a note mentioning app-region as an alias as a result of the new resolution at #13102.

This comes out of resolutions #13102 and
#13101 taken on 21/01/26.

diekus and others added 5 commits December 8, 2025 17:16
- 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
@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 13, 2026

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?

Copy link
Copy Markdown
Collaborator

@emilio emilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 14, 2026

I've added a new text blurb that clarifies event dispatch behavior, as well as clarifying draggable behavior. Please let me know if the changes look good to you!

Copy link
Copy Markdown
Collaborator

@emilio emilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me, but an editor of this spec should also review it :)

@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 14, 2026

@tabatkins @frivoal Hi Editors! Could I ask one of you to take a look at this spec text for window-drag? Happy to make any changes, thank you for your time.

Comment thread css-ui-4/Overview.bs Outdated
@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 14, 2026

@fantasai Noticed you put the following in the older PR:

This probably needs a bit more clarification on what part of the element can be used as a drag target (by tying it to some other definitions, I'm not sure which ones).

Do you think specifying something like-

Specifies that the element's border box can be used as a drag area.

-would be clear enough?

@fantasai
Copy link
Copy Markdown
Collaborator

fantasai commented Apr 16, 2026

@kyerebo Technically it's the hit-testing area, whatever that is. I don't know if we define this concept anywhere though? Maybe just use [=principal box=] for now. I think that'll probably get the point across.

@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 17, 2026

Clarified [=principal box=] as the specific drag target for the element, thanks!

@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented Apr 22, 2026

Hi @tabatkins @frivoal ! I think the change is in a pretty solid state now with reviews, do you think one of you could take a final look, and help me with getting it merged?

Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Comment thread css-ui-4/Overview.bs Outdated
Co-authored-by: Tab Atkins Jr. <jackalmage@gmail.com>
Comment thread css-ui-4/Overview.bs
@tabatkins
Copy link
Copy Markdown
Member

Besides my one question, LGTM

@kyerebo
Copy link
Copy Markdown
Contributor Author

kyerebo commented May 5, 2026

@tabatkins Thank you for the review and writing changes!
If my response to the question is satisfactory, do you think you could leave an approval and help me with merging it in? (I don't have permissions unfortunately.)
Otherwise, I'm happy to make any other changes or continue to discuss.

@tabatkins tabatkins merged commit 93636fb into w3c:main May 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants