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.

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

Edited the 'state state' typo and added Nereida Rondon to contributor… #8349

Merged
merged 4 commits into from
Oct 6, 2022
Merged

Edited the 'state state' typo and added Nereida Rondon to contributor… #8349

merged 4 commits into from
Oct 6, 2022

Conversation

NereidaRondon
Copy link
Contributor

@NereidaRondon NereidaRondon commented Oct 2, 2022

… list in source.
Editorial fix issue ticket - Typo in section "4.8.13 The area element" #8346

I deleted the second 'state' (see below) and added my name to list of contributors at the end of the source file.
image

  • At least two implementers are interested (and none opposed):
  • Tests are written and can be reviewed and commented upon at:
  • Implementation bugs are filed:
    • Chrome: …
    • Firefox: …
    • Safari: …
    • Deno (only for timers, structured clone, base64 utils, channel messaging, module resolution, web workers, and web storage): …
    • Node.js (only for timers, structured clone, base64 utils, channel messaging, and module resolution): …

(See WHATWG Working Mode: Changes for more details.)


/acknowledgements.html ( diff )
/image-maps.html ( diff )

source Show resolved Hide resolved
@NereidaRondon
Copy link
Contributor Author

I also signed the Participation Form. 🤗

@@ -40359,7 +40359,7 @@ interface <dfn interface>HTMLAreaElement</dfn> : <span>HTMLElement</span> {
must be the radius of the circle, again in <span data-x="'px'">CSS pixels</span>.</p>

<p>In the <dfn attr-value for="area/shape" data-x="attr-area-shape-default">default state</dfn>
state, <code>area</code> elements must not have a <code data-x="attr-area-coords">coords</code>
Copy link
Member

Choose a reason for hiding this comment

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

The comma needs to stay on the previous line so that there's not a visible space between "default state" and the comma :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😅 Sorry... fixed.

Copy link
Member

Choose a reason for hiding this comment

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

It doesn't look fixed to me, looking at https://github.com/whatwg/html/pull/8349/files ... perhaps you forgot to push?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just pushed again, I think the first time I didn't push to the correct branch maybe.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks, but now there is a trailing extra space at the end of line 40361! The build will fail if there is trailing space. Could you remove that?

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

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

Perfect, thanks very much!

@domenic domenic merged commit e696c37 into whatwg:main Oct 6, 2022
@NereidaRondon
Copy link
Contributor Author

Thank you! 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants