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

ヒントアイコンの導入 #47 #83

Merged
merged 5 commits into from
Jan 13, 2020
Merged

ヒントアイコンの導入 #47 #83

merged 5 commits into from
Jan 13, 2020

Conversation

yume-yu
Copy link
Collaborator

@yume-yu yume-yu commented Jan 11, 2020

#47 で提案された、inputのヒントをinputに対するマウスオーバーではなくアイコンを使った表示に変更するものです。

@blhsrwznrghfzpr blhsrwznrghfzpr added the enhancement New feature or request label Jan 11, 2020
@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 12, 2020

このPRではとりあえず #47 だけひとまず対応して、その後 #81 のスタイルを調整してからgh-pagesにリリースを考えています。

@yume-yu yume-yu marked this pull request as ready for review January 12, 2020 02:21
@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 12, 2020

以下の環境でスタイル崩れがないことを確認しました
in mac OS 10.14.6

  • Vivaldi 2.10.1745.26
  • GoogleChrome 79.0.3945.117
  • firefox 72.0.1
  • Safari 12.1.2

Windows環境の方、GoogleChromeとfirefox、できればOperaで表示の確認お願いします。

また油断してDraftはずしたら

  • Safari in iPhone SE
  • Chrome in Google Pixel 3

でバルーンがはみ出すのを確認しました。修正します。

@yumetodo
Copy link
Contributor

Chrome
image
Firefox
image

どうもChromeとFirefoxで幅が小さいときの振る舞いが違うよう。なんで?

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 12, 2020

@yumetodo
PCのfirefoxでのモバイルサイズの検証を忘れた...! firefoxではwidthでfit-contentが無効みたいですね

@yumetodo
Copy link
Contributor

@yume-yu あれっ、これFirefoxのほうが意図した挙動だと思ってたけど逆・・・?

見えないのは困るなぁ・・・。

@yumetodo
Copy link
Contributor

そもそも

width: fit-content;

って使い方あってない

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 12, 2020

firefoxでは同名の関数がある?のでプレフィックスが必要なようです
https://caniuse.com/#search=fit-content

@yumetodo
Copy link
Contributor

https://developer.mozilla.org/ja/docs/Web/CSS/fit-content
あれ、この関数呼ぶのではだめ・・・?

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 12, 2020

fit-content()はgridレイアウトで使うものみたいなので、とりあえず今回はwidthの方で調整してみようと思います。

@yumetodo
Copy link
Contributor

(地道にcalcvwと組み合わせてやったほうが安定しそう?(小声))

@KEINOS
Copy link
Member

KEINOS commented Jan 13, 2020

Win10 Pro 64bit 借りて Chrome, Edge, Opera, Firefox で ❓の ツールチップ表示確認しました。(コミット 83e4e87 現在)個人的には LGTM なんだども.

スクリーンショット 2020-01-13 13 05 03スクリーンショット 2020-01-13 13 08 15スクリーンショット 2020-01-13 13 14 36スクリーンショット 2020-01-13 13 20 02

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

@yumetodo @KEINOS Win環境での検証ありがとうございます!
mergeはモバイル環境での修正と検証が終わるまで待ちでお願いします 🙇‍♀
今後はDraftの解除を慎重にしなくては...

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

この変更で
Vivaldi 2.10.1745.26 in mac OS 10.14.6
GoogleChrome 79.0.3945.117 in mac OS 10.14.6
firefox 72.0.1 in mac OS 10.14.6
Safari in iPhone SE
Chrome in Google Pixel 3

で正常なバルーン表示が確認できました! 確認お願いします 🙇

@yumetodo
Copy link
Contributor

yumetodo commented Jan 13, 2020

ハズキルーペがほしいレベルになった。「文字が小(ry
image

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

あんまり小さい幅のはモバイル前提のフォントサイズなのでPCだと見にくいかもです...デバイスでもあんまりに感じられたらもう少し大きくしてみようかなと思います

@yumetodo
Copy link
Contributor

yumetodo commented Jan 13, 2020

そもそもこのツールチップ出すために、?をタップしたら入力枠にフォーカスが移って、上が見切れるというもっとあれな問題

3F6E8A3F-255E-41F6-BE1D-A98BD4C4A825

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

ちなみにPixel3だとこんなかんじのサイズ感です
75bdf6c86cbf9e14

font-size: .6rem;
}
}

@media screen and (max-width: 669px) {
/* width for mobile view */
.error {
.hint {
right: -8vw ;
font-size: 2vw;
Copy link
Contributor

Choose a reason for hiding this comment

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

文字が小さくなるのはこれのせいか。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

これですね。改行が発生するよりはfont-sizeで調整して埋めようという考えです

@yumetodo
Copy link
Contributor

yumetodo commented Jan 13, 2020

?の判定がシビアすぎる。文字列選択になったり、ツールチップが出たり、入力枠にフォーカスしたり。

というかこの判定は何・・・?
6011886A-3303-476D-B5AB-1A3AF4CB5D30

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

現状ではアイコンがlabelに収まっているので、タッチした後に該当inputにフォーカスが行くみたいですね。iOSだと勝手にヨリになるので見えている範囲からバルーンがはみ出る可能性がある...?

@yumetodo
Copy link
Contributor

input枠にフォーカスがある間は一度出したら消えないでほしい感ある(もうこのPRの範疇超えていますが)

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

ちなみに #83 (comment) の判定はlabel要素の判定です。クリック/タッチされると該当するinputにフォーカスします

@yumetodo
Copy link
Contributor

iosだと勝手にヨリになるので

どういうことでしょう?

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

iosだと勝手にヨリになるので

どういうことでしょう?

iOS Safariはinput type=textとか textareaにフォーカスすると、その幅に合わせて勝手にズームする時があるのです

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

とりあえず @yumetodo さんのいうとおりバルーンの出現を維持する点については別対応でも大丈夫な気がするのでもう少し待って表示に異常が出なければまーじします!

@yume-yu
Copy link
Collaborator Author

yume-yu commented Jan 13, 2020

修正後問題のあった環境で改善が確認されたのでmergeします

@yume-yu yume-yu merged commit 5c69869 into master Jan 13, 2020
@yumetodo
Copy link
Contributor

@yume-yu branch消し忘れていそう

@yume-yu yume-yu deleted the add_hinticon branch January 14, 2020 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants