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

Thinking of IE 🤔 #104

Merged
merged 4 commits into from
Nov 20, 2020
Merged

Thinking of IE 🤔 #104

merged 4 commits into from
Nov 20, 2020

Conversation

herablog
Copy link
Member

実プロジェクトで利用するに当たり、IE11でも使えたほうが良いということでいくつか対応をしました。

  • StorybookにIE11だけCSS custom propertiesのpolyfillを追加しました 89c9aa8
  • ブラウザサポートをドキュメント化しました 22f007e
  • IE11でalign-items: centermin-heightを併用すると中央配置されない問題を解決しました 6fc6a13

確認すること

StorybookのボタンをIE11で確認します。

Before

Screen Shot 2020-11-17 at 2 37 35 PM

After

Screen Shot 2020-11-17 at 2 07 18 PM

close #103

@@ -3,3 +3,4 @@
font-family: Meiryo, Yu Gothic Medium, system-ui, -apple-system, BlinkMacSystemFont, sans-serif;
}
</style>
<script>window.MSInputMethodContext && document.documentMode && document.write('<script src="https://cdn.jsdelivr.net/gh/nuxodin/ie11CustomProperties@4.1.0/ie11CustomProperties.min.js"><\x2fscript>');</script>
Copy link
Member Author

Choose a reason for hiding this comment

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

いくつか選択肢はあるけど、Storybookでは他のブラウザに影響を与えないこれで十分そう?なのでややお行儀の悪い方を入れてみました。

ref: #103 (comment)

Copy link
Contributor

@Dai7Igarashi Dai7Igarashi Nov 19, 2020

Choose a reason for hiding this comment

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

@media で IEでhover効いていないかもです。。。
https://caniuse.com/mdn-css_at-rules_media_hover

Copy link
Member Author

Choose a reason for hiding this comment

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

こりはわかりにくくて恐縮なのだけど、想定内の動作になるのだ・・!

#13

Copy link
Member Author

Choose a reason for hiding this comment

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

ドキュメントに記載しておきました〜 3a27ab0

.spui-Button--large,
.spui-Button--medium,
.spui-Button--small {
height: 1px;
Copy link
Member Author

Choose a reason for hiding this comment

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

高さ自体はなんでもよさげさん?
ref: philipwalton/flexbugs#231 (comment)

@github-actions
Copy link
Contributor

github-actions bot commented Nov 17, 2020

Visit the preview URL for this PR (updated for commit 3a27ab0):

https://ameba-spindle--pr104-chore-thinking-ie-inzdfo7v.web.app

(expires Sat, 19 Dec 2020 03:58:00 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@reg-suit
Copy link

reg-suit bot commented Nov 17, 2020

✨✨ That's perfect, there is no visual difference! ✨✨

Check out the report here.

@herablog herablog mentioned this pull request Nov 17, 2020
8 tasks
@syasuda90
Copy link

hover時のstyleが効いてなさそうでしたけどこの辺りは許容する感じでしょうか?
IE未対応...
https://developer.mozilla.org/ja/docs/Web/CSS/@media/hover

@syasuda90
Copy link

ie
わかりづらくて申し訳ないのですがfocus時に文字が少しズレるのが気になりました。
こちらも許容できる範囲かと思いますが一応コメントさせていただきました。

@herablog
Copy link
Member Author

@syasuda90

hover時のstyleが効いてなさそうでしたけどこの辺りは許容する感じでしょうか?
IE未対応...

ありがとうございます〜こりは頻出質問でしたので、許容のむねドキュメントに追記しました。 3a27ab0

@herablog
Copy link
Member Author

@syasuda90

focus時に文字が少しズレる

普通のボタンでもクリックすると文字がずれたのですが、これってもしやIE共通の動作でしたっけかね〜 (忘却彼方)

ie-button

@sasaplus1
Copy link
Member

#104 (comment)

IEはボタン押した時に文字がずれた記憶ありますね〜(うろ覚え)

@syasuda90
Copy link

検索してみるとbuttonタグでずれるという現象あるみたいですね。
(IEの問題みたいです)

情報の欠落などはなさそうなのでこちらはこのままでよいですかね。

Copy link

@syasuda90 syasuda90 left a comment

Choose a reason for hiding this comment

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

私からはapproveさせていただきます。

@herablog
Copy link
Member Author

ありがとうございます〜IE以外に影響がなさそうってreg-suitも言ってるので取り込んじゃいますね!

@herablog herablog merged commit 9b31ea2 into main Nov 20, 2020
@herablog herablog deleted the chore/thinking-ie branch November 20, 2020 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IE対応をやっと考える
4 participants