Skip to content

Answerタグの改善#485

Merged
aster-void merged 3 commits into
masterfrom
improve-answer-tag
Oct 19, 2023
Merged

Answerタグの改善#485
aster-void merged 3 commits into
masterfrom
improve-answer-tag

Conversation

@aster-void
Copy link
Copy Markdown
Contributor

No description provided.

@aster-void
Copy link
Copy Markdown
Contributor Author

・解答は一つではないので 「解答例」に変更
・問題のタイトルをつけれるように (任意)

@aster-void aster-void requested a review from chvmvd October 18, 2023 15:41
Copy link
Copy Markdown
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

引数は気になったけど、マージしてよいかと。

*/
export default function Answer({ children }) {
return <Details summary={<summary>解答</summary>}>{children}</Details>;
export default function Answer({ type = undefined, children }) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

typeというよりは、titleな気がする?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

この引数だけ見てもどういう引数かが、わからないんだよね...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

確かに

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

とりあえず、マージしてからでいいや。

@chvmvd
Copy link
Copy Markdown
Contributor

chvmvd commented Oct 19, 2023

マージする時は、こっちを先に!

@chvmvd
Copy link
Copy Markdown
Contributor

chvmvd commented Oct 19, 2023

そうすると、マージ先のブランチが自動で切り替わる。

@aster-void
Copy link
Copy Markdown
Contributor Author

マージします!

@aster-void aster-void merged commit f64832f into master Oct 19, 2023
@aster-void aster-void deleted the improve-answer-tag branch October 19, 2023 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants