Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

Removed CanvasProxy, setContext, removed direct-* in context mode #424

Merged
merged 3 commits into from
May 26, 2016

Conversation

plehegar
Copy link
Member

Fix #374

@plehegar plehegar added this to the HTML5.1 WD (June 2016) milestone May 25, 2016
@@ -1421,7 +1418,7 @@ o............A....e

When authors use the <{canvas}> element, they must also provide content that, when
presented to the user, conveys essentially the same function or purpose as the
<code>canvas</code>' bitmap. This content may be placed as content of the <code>canvas</code>
<code>canvas</code>'s bitmap. This content may be placed as content of the <code>canvas</code>
Copy link
Member

Choose a reason for hiding this comment

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

Not critical, but worth keeping in mind for long-term spec hygiene: Whenever possible, let's try to eliminate <code></code> markup (unless it's specifically denoting some code). These are hold-overs from the old WHATWG source, where <code> is used for linking (in Anolis). With Bikeshed, we use a variety of syntaxes for linking. Here, <code>canvas</code> is a reference to the element, so we can use the element syntax: <{canvas}> instead. It's good to improve linking wherever we can as we make changes :-)

@travisleithead
Copy link
Member

A few nits in comments. Assigning back to you in case you want to make the changes, otherwise, let me know, and it's merge-ready.

@chaals
Copy link
Collaborator

chaals commented May 26, 2016

@arronei so you notice the editing conventions mentioned, when you're working on "how to edit for bikeshed".

I should also mention the mapping of the source files which should probably go into the gh-pages branch…

@travisleithead
Copy link
Member

Super. LGTM.

@travisleithead travisleithead merged commit 5e1a823 into master May 26, 2016
W3C-HTML-Bot pushed a commit that referenced this pull request May 26, 2016
arronei pushed a commit to arronei/html that referenced this pull request Apr 17, 2017
…c#424)

* Removed CanvasProxy, setContext, removed direct-* in context mode

* Use bikeshed linking. Avoid break in dfn and a

* Another indent
@LJWatson LJWatson deleted the drop-canvas-proxy branch August 8, 2017 16:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants