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

feat!: accordion要素をdetails~summary要素へ #292

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Hidetaro7
Copy link
Contributor

No description provided.

@Hidetaro7 Hidetaro7 linked an issue Oct 13, 2022 that may be closed by this pull request
@Hidetaro7 Hidetaro7 self-assigned this Oct 13, 2022
@Hidetaro7
Copy link
Contributor Author

とはいえ、マーカーのスタイルを変更したい場合どうしたらよいのか、もう少し考えたほうが良さそうな気がする。

Copy link
Member

@knokmki612 knokmki612 left a comment

Choose a reason for hiding this comment

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

見た目が変わっていますが意図どおりですかね

変更前
image

変更後
image

@Hidetaro7
Copy link
Contributor Author

見た目が変わっていますが意図どおりですかね

はい、極力独自のスタイルを提供するのは避けたい、ブラウザデフォルトとさほど変わらないくらいのシンプルさがコンセプトに近いです。

冒頭にその方針や意図を書くことが大事かもしれないですね。

Figmaでは反映しておきました

@knokmki612
Copy link
Member

knokmki612 commented May 10, 2023

マーカーのスタイルを変更したい場合どうしたらよいのか

svgtodatauri を使っている箇所は同じことが言えるかもしれませんね
また、マーカーの位置をサマリーテキストに対して左にするのか右にするのか、というのもケースバイケースで変わってくるかもしれませんね

@knokmki612
Copy link
Member

擬似要素を使う代わりに、 [aria="presentation"] 要素をマーカーとして取り扱い、任意の形状のアイコンを使えるようにしてもいいかもしれませんね

@Hidetaro7
Copy link
Contributor Author

[aria="presentation"]

属性を使ったサンプルを見つけたので参考までに貼っておきます。

https://codepen.io/diomed/pen/gQyapY

@Hidetaro7
Copy link
Contributor Author

属性を使ったサンプルを見つけたので参考までに貼っておきます。

見間違いだったのかもしれません。

@knokmki612 knokmki612 changed the title feat: accordion要素をdetails~summary要素へ feat!: accordion要素をdetails~summary要素へ Jun 28, 2023
@knokmki612 knokmki612 added the breaking-change This change doesn't have backward compatibility label Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change This change doesn't have backward compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accordion は detail要素 〜 summary要素でマークアップ
2 participants