Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"Clear regions that cover the pixels" #2097

Closed
annevk opened this issue Nov 25, 2016 · 1 comment
Closed

"Clear regions that cover the pixels" #2097

annevk opened this issue Nov 25, 2016 · 1 comment
Labels
topic: canvas

Comments

@annevk
Copy link
Member

annevk commented Nov 25, 2016 •
edited
Loading

This is a term used in clearRect() that does not go anywhere. Found while working on whatwg/wattsi#33. I'm not sure what it should reference.
annevk added a commit that referenced this issue Nov 25, 2016
@annevk
Editorial: fix internal cross-reference errors
549cccc 
Found while working on whatwg/wattsi#33.

I also took the liberty to remove erroneous title attributes.

#2097 is follow-up.
@annevk annevk mentioned this issue Nov 25, 2016
Merged
sideshowbarker added a commit that referenced this issue Nov 25, 2016
@sideshowbarker
Cut requirement left behind by hit-region removal
1bf0f3f 
This “Clear regions that cover the pixels” requirement for `clearRect()`
should’ve been cut in 9d493a3 along with the other hit-region requirements.

Closes #2097.
@sideshowbarker sideshowbarker mentioned this issue Nov 25, 2016
Merged
@sideshowbarker
Copy link
Contributor

The dfn for it was cut in 9d493a3 “Remove canvas element's hit region feature (for now)“.

And the stray remaining reference we still have to it was originally added in the same commit where the dfn was originally added in, a2febc2 “Make it easier to do hit testing on canvas”.

So I think we can just drop the whole list item that stray remaining xref is in; opened #2100 for that.

(I thought at first the change could just be added to #2098, but since that PR is editorial, and the list item we need to drop is stating a normative requirement, it seems better to land it on its own.)

annevk pushed a commit that referenced this issue Nov 25, 2016
@sideshowbarker @annevk
Remove requirement left behind by hit-region removal
a26f948 
This “Clear regions that cover the pixels” requirement for `clearRect()` should’ve been removed in 9d493a3 (PR #1942) along with the other hit-region requirements.

Closes #2097.
@sideshowbarker sideshowbarker mentioned this issue Nov 26, 2016
Merged
zcorpan pushed a commit that referenced this issue Nov 28, 2016
@annevk @zcorpan
Editorial: fix internal cross-reference errors
2e7e672 
Found while working on whatwg/wattsi#33.

I also took the liberty to remove erroneous title attributes.

#2097 is follow-up.
alice pushed a commit to alice/html that referenced this issue Jan 8, 2019
@sideshowbarker
Remove requirement left behind by hit-region removal
1e77b00 
This “Clear regions that cover the pixels” requirement for `clearRect()` should’ve been removed in 9d493a3 (PR whatwg#1942) along with the other hit-region requirements.

Closes whatwg#2097.
alice pushed a commit to alice/html that referenced this issue Jan 8, 2019
@annevk
Editorial: fix internal cross-reference errors
6bcb5d7 
Found while working on whatwg/wattsi#33.

I also took the liberty to remove erroneous title attributes.

whatwg#2097 is follow-up.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
topic: canvas
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sideshowbarker @annevk