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.
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: ソング:音素タイミングの表示エリアを追加 #2472
feat: ソング:音素タイミングの表示エリアを追加 #2472
Changes from 10 commits
f15e54c
7470bd8
2e5b636
7a18ff6
c219876
4bdab0e
4f668e3
914bcfb
4756f9b
4f1c341
6c942e9
9f2977f
382b8e7
9a10b1b
6d1fe84
07172e5
43923d6
ab02d28
247f059
584791d
1436c64
672fc6e
adad0e2
7662a2f
8e05527
2342a8b
519d9b1
b64096b
fce1c16
39400f8
0e896dd
8495413
b58632d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@romot-co
できるだけ以前と同じ挙動になるようにしましたが、正しく実装できているか自信が無いので、スタイル周り、ご確認いただけると助かります…!
(
contain: layout;
など、効いていなさそうなプロパティを消しましたが、必要なものも消しているかもしれません…)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sigprogramming
こちら確認いたします!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
メモ書きです…!
contain: layout
はfixed
以外で実装しようとした時の名残なので消しても大丈夫そうです(スクロール時のパフォーマンス上がることがあるので残しておいても問題ないはず)position: sticky
+子にmax-width
指定が一番楽そうほか気になっているやつ:
https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_anchor_positioning
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sigprogramming
こちら以下で修正いたしました…!挙動他問題や影響ありましたらおしらせください!
sigprogramming@6c942e9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@romot-co
ありがとうございます!!とても助かります…!
挙動を確認してみたのですが、今のmainブランチではシーケンサーのheightがある程度大きくなると立ち絵も拡大されるようになっていて、 6c942e9 では拡大されなくなっているかもです。ここの挙動、どうしましょう…
また、
position: sticky
ありとposition: sticky
なしの挙動の差があまり分かりませんでした…調べてみたところ、
position: sticky
になっている要素がスティッキーアイテムで、その親がスティッキーコンテナになるようです。https://coliss.com/articles/build-websites/operation/css/css-position-sticky-how-it-really-works.html
なので、 6c942e9 の実装だと
character-portrait-wrap
がスティッキーアイテム、score-sequencer
がスティッキーコンテナになりますが、character-portrait-wrap
はグリッドアイテムでもあるので、もしかしたら上手く機能していないかも…?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ちょっと自分も試してみたんですが、原理は全く分かりませんがなんかこれでうまいことできた気がします!!
PRも作ってみました! sigprogramming#4
ちなみにChatGPTに聞きました。o1君、結構CSSの強い味方かもしれない。
https://chatgpt.com/share/678d25ad-68f0-8008-b197-0c1b7da2b59b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Hiroshiba
こちら確認しました!
簡潔な手法でだいぶベストに思えます…!
o1すごい…!
+たぶん
ポートレート幅 > 親のQSplitter幅の場合に
にスクロールバーが二重に出るかもなので
みたいにすれば完了に思えます!
あとで修正PR送ります…!