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

[Minor] Bottom bar looks odd on iPhones with rounded borders (iPhones X, 11-13) #8311

Open
NovaAndrom3da opened this issue Feb 12, 2022 · 16 comments
Labels
packages/frontend Client side specific issue/PR

Comments

@NovaAndrom3da
Copy link

Summary

The current bottom padding amount looks odd on the iPhone X, 11, 12, and 13. This is not a major issue.

@NovaAndrom3da NovaAndrom3da added the ✨Feature This adds/improves/enhances a feature label Feb 12, 2022
@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

Can someone provide a screenshot?

@syuilo syuilo added packages/frontend Client side specific issue/PR and removed ✨Feature This adds/improves/enhances a feature labels Feb 12, 2022
@NovaAndrom3da
Copy link
Author

NovaAndrom3da commented Feb 12, 2022

@syuilo Sadly iOS screenshots do not show the screen border radius so it yields the same screenshot a device with a rectangular screen would provide. I can take a picture of my phone separately though.

Also, the emoji picker (both the one for a reaction and for when composing a note) are too cut off by the border radius.

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

@Froggo8311 do you mean like this?

imageimageimage

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

セーフエリア考慮したCSS書いてるから↑のようにはならないはずなんだけどな
(ただ実機で動作確認したことは無し)

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

(Misskeyのバージョンが古いとかかしら)

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

misskey.io って古いんだっけ、最新のやつ走ってるとこある?

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

https://sushi.ski/ が最新かつ誰でも登録できる

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

sushi.skiだとこんな感じ (あんま変わんない気がする)
imageimage

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

あー動いてなさそう
ありがとう🙏

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

セーフエリア考慮したCSS書いてる箇所は #8311 (comment) の三枚目のUIだけだった

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

それも結局だめそう
image

@syuilo
Copy link
Member

syuilo commented Feb 12, 2022

あれー
こういう書き方じゃ効かないのかしら

padding: 12px 0 calc(env(safe-area-inset-bottom, 0px) + 12px) 0;

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

https://developer.mozilla.org/ja/docs/Web/CSS/env()

ブラウザーに画面上の利用可能な範囲全体を使用するように指示し、 env() 変数を使用できるようにするには、新しいビューポートのメタ値を追加する必要があります。

これかな

@rinsuki
Copy link
Contributor

rinsuki commented Feb 12, 2022

↑これ追加したらまあまあな感じになった
image

syuilo added a commit that referenced this issue Feb 12, 2022
@NovaAndrom3da
Copy link
Author

The new tab bar position looks better, but the emoji picker is still very close to the screen border.
F7D182A0-B700-458B-A21F-119D12C7752A

@NovaAndrom3da
Copy link
Author

The settings > privacy screen now isnt long enough so the "keep content warnings" checkbox is unaccessible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

No branches or pull requests

3 participants