おめシス人気投票へのリンクを一時的に追加

ryamaguchi0220 commented 4 years ago

変更内容: Summary

おめシス人気投票へのリンクを一時的に追加しました。

確認事項: Check point

[x] PR を作成する前に、 https://github.com/omegasisters/homepage の最新の master を取り込み済みである。
- Conflict や他の方の変更で自分の変更が動かなくなる可能性を防ぎます。
- 最新の master を取り込む方法
- upstream に fork 元リポジトリを追加
  - git remote add upstream git@github.com:omegasisters/homepage.git
- 現在のブランチに upstream の master を取り込む
  - $ git pull --rebase upstream master
- おまけ
- rebase 後に再度 push する場合、 --force-with-lease オプションをつける
  - git push --force-with-lease origin <ブランチ名>
[x] 動作確認済みである。
- 何らかの理由で本番に取り込まれるまで確認できない場合はその旨を補足に記載する。
[x] prettier によるコード整形を行った、もしくは画面に関係ない変更である。
- 可能な方のみで良いと思いますが、意図せず他の方がフォーマットするとコード差分が増えすぎるので自分の分は自分でやるのがよろしいかと思います。
[x] スマホ（狭い画角）でも表示を確認した、もしくは画面に関係ない変更である。
[x] 他の方の変更を意図せず削除・変更していないか、差分をもう一度確認した。
[x] 破壊的な変更を行った場合、影響範囲をもう一度確認した。もしくは破壊的な変更を行っていない。
[x] Pull Request に関連した issue の URL を貼り付けた

補足: Other Information

hirune4791dev commented 4 years ago

@ryamaguchi0220 画面の横幅依存で、2段になってしまうケースがあるようです。　~~また、フォント色がなぜか白いままです。~~

スクリーンショット (113)

以下で直るといえば直ります。理屈は理解できていないので、副作用の可能性ありです。少なくとも、フォントサイズの再調整は必要そうです。参考までに。

diff --git a/assets/css/index.css b/assets/css/index.css
index 36490fa..568d82a 100644
--- a/assets/css/index.css
+++ b/assets/css/index.css
@@ -34,6 +34,7 @@ header.drawer-navbar .drawer-navbar-header {
   background-color: rgba(0, 0, 0, 0);
   border-bottom: none;
   padding: 0 1.5rem;
+  height: auto !important;
 }

 header.drawer-navbar {

思ったのですが、背景色の関係で、"赤"は逆に目立たないかもしれません。さらに、ハンバーガメニューの場合、今回のように最後尾に配置してしまうと、画面の外に出てしまって、気づかないケースがありそうです。期間限定なので、先頭に持ってきてしまうのはどうでしょうか？

ryamaguchi0220 commented 4 years ago

画面の横幅依存で、2段になってしまうケースがあるようです。

画面サイズが特定値 (横幅 1359px) に達すると、以降ヘッダーメニュー (drawer-nav) の横幅が1344pxに固定されてしまい、1344pxだと font-size 1rem で2段になってしまう。という理屈のようです。これは drawer が定義している font-size 1rem のスタイルを上書いてあげると筋が良さそうです (ベストはシュリンクを捨ててこっちの方法に移行する) -> 8b43d51

文字色と位置は調整しました -> f84c56d もしより良い色があれば提案してもらえれば〜。

hirune4791dev commented 4 years ago

f84c56d049f638c93265bb6906f4ab027f39a599 確認しました。OKです。色、場所とも、よく目立って良いと思います。

omegasisters commented 4 years ago

👍

omegasisters / homepage