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

fix: 提供中のブックの有無で領域の高さが異なり表示位置がずれる場合がある #446

Merged
merged 4 commits into from
Jun 22, 2021

Conversation

knokmki612
Copy link

@knokmki612 knokmki612 commented Jun 22, 2021

#418 (comment) を改善します

変更前

image

変更後

image

「提供中のブック」の表示領域がウィンドウの表示領域内にある状態で異なるブックを提供すると、「提供中のブック」が存在しない場合に表示されるメッセージが瞬間的に表示される場合があります。その際のメッセージとBookPreviewの画面要素のheightが異なり、そのheight差分のぶん「ガクッと」一覧されるブックの表示位置が変動します。

この変更によって、「提供中のブック」がない場合に表示されるメッセージをBookPreviewの高さ相当に調節し、「ガクッと」表示位置が変動されることを抑制します。

また、この変更の副次的な要素としてBookPreviewに動画プレイヤーがない場合等の高さが微妙に異なる場合もあらかじめ所定の高さをminHeightによって指定しているため、こちらのケースについても表示位置の変動が抑制されます。

@knokmki612 knokmki612 requested a review from kou029w June 22, 2021 06:19
@vercel
Copy link

vercel bot commented Jun 22, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ties-makimura/chibi-ch-i-lo/3za1WU7fcj9e9WKWDfxhWYifsSfd
✅ Preview: https://chibi-ch-i-lo-git-fix-linked-book-height-ties-makimura.vercel.app

Copy link

@kou029w kou029w left a comment

Choose a reason for hiding this comment

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

1点コメントについてコメントしていますが本PRの実装そのものについてはとても良さそうに見えます 👍

@ties-makimura
Copy link

提供中のブックがある場合でも,提供しているコース情報の所で高さの差異があり,クリックしたときにずれが出ます.

image

@knokmki612
Copy link
Author

コースチップを折り返し表示している関係上、BookPreviewがコースチップの折り返しぶん高さの変動が生じうる状態ですね

コースチップの折り返し表示を許容せず、溢れたぶんは非表示にするような対処が必要かと思いました

@ties-makimura
Copy link

ties-makimura commented Jun 22, 2021

コースチップの折り返し表示を許容せず、溢れたぶんは非表示にするような対処が必要かと思いました

コースチップは全て見せておかないと,コースで探せなくなるのでその方法は避けてください

@knokmki612
Copy link
Author

留意しておきますね

@knokmki612
Copy link
Author

knokmki612 commented Jun 22, 2021

あふれたぶんを非表示する以外には、あふれたぶん水平方向にスクロール領域となりスクロールしてもらうことも考えられますね

@ties-makimura
Copy link

ties-makimura commented Jun 22, 2021

あふれたぶんを非表示する以外には、あふれたぶん水平方向にスクロール領域となりスクロールしてもらうことも考えられますね

その方法ですと,コースチップが見切れる(見えなくなる)ので良くないです.

提供を選択した時,何か「押した!」というのがはっきりわかるようになれば,位置がずれても気にならなくなるかもしれません.
今は,押した時にスッと処理されてそのまま位置がずれてしまい,何が起こったのか分からなくなっているのかもしれない.

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.

3 participants