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

アコーディオンを開いたときに他のアコーディオンを閉じる設定を追加 #204

Open
inc2734 opened this issue Oct 1, 2019 · 7 comments

Comments

@inc2734
Copy link
Owner

inc2734 commented Oct 1, 2019

No description provided.

@kmix-39
Copy link
Contributor

kmix-39 commented Oct 4, 2019

現在のアコーディオンにこの設定を追加するより、

  • 項目をradioボタンで設定した排他的なアコーディオンブロックを別に作る
  • ブロックのスタイルで、タブスタイルにも変えられるようにもCSSでスタイリングする
  • 現在のアコーディオンとトランスフォームで切り替え出来る(親ブロックを選択時)

の方が将来的にもスタイル当てやすくて使い勝手が良くなるかとー。
アコーディオンの親ブロックで「開いている項目以外の項目を閉じる」って作っても分かりにくい問題もあると思います。

どうでしょう?

@inc2734
Copy link
Owner Author

inc2734 commented Oct 4, 2019

そうですね、それが良い気がします。
そもそもの問題はチェックボックスでやっちゃったことですね。アクセシビリティ考えると素直に JS でやったほうが良かった気がしています…。

@inc2734
Copy link
Owner Author

inc2734 commented Oct 4, 2019

ブロックのスタイルで、タブスタイルにも変えられるようにもCSSでスタイリングする

タブはフォーラムで要望もらうこともあるのですが、エディタ上での表現が難しい気がしていて特に作業していないんですよね。アコーディオンはエディタ上では全部開きっぱなしでも違和感がありませんが、タブは全部開きっぱなしが難しいなと。

@kmix-39
Copy link
Contributor

kmix-39 commented Oct 4, 2019

タブは選択した項目だけ開いて編集させりゃ良いかと。

開いている項目ってオプション作って、「すべて閉じた状態で表示」「選択している項目を表示」とするのが良いと思ってもいます。子が選択された際に親に選択された子のindexをattributeに保持させるのは出来るはず。

@inc2734
Copy link
Owner Author

inc2734 commented Oct 4, 2019

タブは選択した項目だけ開いて編集させりゃ良いかと。

お、エディタ上でタブ操作して、ってことですよね。なるほど。なんか勝手にタブがタブとして動くのは実際の画面だけと考えていました。

開いている項目ってオプション作って、「すべて閉じた状態で表示」「選択している項目を表示」とするのが良いと思ってもいます。

このオプションあるとタブスタイルとの兼ね合いがめんどくさくないです? それならタブは別でつくったほうがシンプルかなーと。

@kmix-39
Copy link
Contributor

kmix-39 commented Oct 5, 2019

このオプションあるとタブスタイルとの兼ね合いがめんどくさくないです? それならタブは別でつくったほうがシンプルかなーと。

プルリクしたのは、一度オプション無しにしました。
選択しないと中身が見れないと言うのも何か微妙だったし、全部開いて出してます。
選択している項目…と言うのも微妙だったし、そこのエディター時UIと設定はもう少し考えないとですね。

何となく、タブを別にするなら今のアコーディオンを排他的にするオプションでやっても良い気もしてきました。
rootIdをマイグレーションで取得して云々は、項目の方にrootBlockのattributesを取ってそのタイプが「排他的」に選択されていれば…radioにするとすれば出来るなーと。
その時に「開いた状態で表示」のオプションをインスペクターから外せば良いし。

今のアコーディオンを排他的アコーディオンにtransformさせるのが思ったより面倒なんで、その形が良いかな…と、悩んでます。インナーブロックの子の項目を上手く変換するのが何か上手くいかない。

作るのは全然こちらで出来るんでどっちにしましょうかねー…うーん。

@kmix-39
Copy link
Contributor

kmix-39 commented Oct 5, 2019

タブブロックは別のブロックで作りますかね。まあまあ良い感じのが思い付いたんで。

アコーディオンは別ブロックに分けなくても、radioなので開いてる項目は閉じれないって問題だけあるけど、それ以外は何とかなりそう。transformsも思ったより使いにくいし、再度、現状のアコーディオンブロックでまとめてみます。
と言うことで、プルリクは一度閉じました。

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

No branches or pull requests

2 participants