DOM の課題の追加#519
Merged
Merged
Conversation
Contributor
Author
|
と appendChild、createElementの追加 |
Deploying with
|
| Latest commit: |
d4a18e7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://b6d0792f.utcode-learn.pages.dev |
| Branch Preview URL: | https://improve-dom-exercise.utcode-learn.pages.dev |
chelproc
reviewed
Oct 28, 2023
Contributor
chelproc
left a comment
chelproc
left a comment
There was a problem hiding this comment.
普通にレビューしてみたのですがそもそもクラスもやっていない状況でここまでやる意味ありますかね？
Contributor
Author
クラスの課題を別のものにすることになって、クラス以外にToDoまでにDOMのメソッドの説明を入れるところがここくらいしかなかったです |
Contributor
|
DOMは非常に脱落者が多いところなのでこれ以上難しくしたくはないですね。とりあえず、おみくじまでは全員が到達して欲しいので。後は、クラスをやる前に説明すると後方参照が増えるのでなるべく避けたいという気持ちもあります。クラスのところも量を増やしたくはないけど、DOMよりはマシかなという感じだと思います。 |
Contributor
Author
|
クラスの後に｢DOM の提供するメソッド｣みたいなセクションを作りますか? |
Contributor
|
メモ: 10/30 のミーティングで Node#appendChild と document.createElement は DOM 側に統合することになった |
chelproc
reviewed
Oct 31, 2023
chvmvd
reviewed
Nov 1, 2023
chvmvd
reviewed
Nov 1, 2023
chvmvd
reviewed
Nov 1, 2023
chelproc
reviewed
Nov 1, 2023
chelproc
reviewed
Nov 2, 2023
Comment on lines
+75
to
+77
| const newElement = document.createElement("div"); | ||
| newElement.textContent = "Hello World!"; | ||
| parent.appendChild(newElement); |
Contributor
There was a problem hiding this comment.
newElement はさっき出てきたのでここで書く必要はないのでは？
chvmvd
reviewed
Nov 2, 2023
span 初出だと思うので、button のほうが良ければ revert してください。
chelproc
reviewed
Nov 2, 2023
Comment on lines
+64
to
+69
| そして、`要素1.appendChild(要素2)` とすることで、要素1の子要素に要素2を追加し、画面に表示することができます。 | ||
| 今回は、`div` 要素の子要素にしてみましょう。 | ||
|
|
||
| ```html title="index.html" | ||
| <div id="parent-element"></div> | ||
| ``` |
Contributor
There was a problem hiding this comment.
五月雨式で申し訳ないのですが、この部分このセクションの冒頭にあった方が良くないですかね？
この部分は HTML なのですでに知っている情報なので、この HTML を最初に提示した上で、どうやって子要素を追加するのか、というストーリーになるのがいいんじゃないでしょうか。
Contributor
Author
There was a problem hiding this comment.
書き方が難しいですが頑張ってみます
Contributor
Author
There was a problem hiding this comment.
ストーリーの目的は、div 要素の子要素に追加することではなく新しい要素を作ることなので、最初に空の div 箱を作っておくよりここで作るほうが自然な感じがしました。
なのでここのほうがいいと思います
Contributor
Author
|
良さげならマージします！ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.