Merging attributes page into Manipulating Elements#713
Merging attributes page into Manipulating Elements#713connorcartwright wants to merge 5 commits intojquery:mainfrom
Conversation
…om the attributes page
|
Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA). 📝 Please visit http://contribute.jquery.org/CLA/ to sign. After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know. If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check. |
|
@connorcartwright Thank for the PR :-) Looks good to me. @AurelioDeRosa Does this look good to you? |
There was a problem hiding this comment.
Probably this line and the next can be combined with the previous one. I'm not sure we need break lines in between.
|
I've added a few comments to improve the PR. Overall I'm excited to see that we have a new contributor. Thank you @connorcartwright! |
|
Hey @AurelioDeRosa No problem, thank you for the feedback! I've removed the line spacing between those two sentences and added a link to the Would it be best to add the
Or is there a simpler way I'm missing? |
|
I think we might have a process in place for these situations but I'd like to have our expert opinion @gnarf |
Hey guys @AurelioDeRosa @agcolom
Following the comments made in issue #642 by devs4u I’ve merged the attributes page into the manipulating elements page.
I've added some line spacing and moved around the text a bit in the Manipulating Attributes section. I also added in a sentence from the old attributes page and a getter code example.
Let me know if you have an feedback or if there are any problems and I'll get on it asap