whatwg/html
Small clarity improvements to canvas transform methods
Ouverte
#2 848 ouverte le 19 juil. 2017
clarificationgood first issuetopic: canvas
Métriques du dépôt
- Stars
- (7 654 étoiles)
- Métriques de merge PR
- (Merge moyen 42j 18h) (22 PRs mergées en 30 j)
Description
While in the area for #2845, I noticed the following small tweaks that we could make to improve that section of the spec:
- Replace "abort these steps" with "return" everywhere. (The latter is a more modern convention, per Infra.)
- Link "multiplying" in the definition of
transform(a, b, c, d, e, f)to https://drafts.fxtf.org/geometry/#matrix-multiply - Re-do
setTransform(a, b, c, d, e, f)steps 2-3 to reset the current transformation matrix to the appropriate matrix (a c e / b d f / 0 0 1), instead of resetting the matrix to the identity matrix then invoking the transform() method. It's bad practice to invoke methods from other methods.
This is a good first bug :)