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

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

Merged
merged 4 commits into from
Feb 11, 2020
Merged

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

merged 4 commits into from
Feb 11, 2020

Conversation

ryamaguchi0220
Copy link
Contributor

@ryamaguchi0220 ryamaguchi0220 commented Feb 10, 2020

変更内容: Summary

おめシス人気投票へのリンクを一時的に追加しました。
スクリーンショット 2020-02-11 4 31 34

確認事項: Check point

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

補足: Other Information

@hirune4791dev
Copy link
Contributor

hirune4791dev commented Feb 11, 2020

@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
Copy link
Contributor Author

ryamaguchi0220 commented Feb 11, 2020

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

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

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

スクリーンショット 2020-02-11 13 50 45

@hirune4791dev
Copy link
Contributor

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

@omegasisters omegasisters merged commit 61921de into omegasisters:master Feb 11, 2020
@omegasisters
Copy link
Owner

👍

@ryamaguchi0220 ryamaguchi0220 deleted the add-popularity-pole-link branch February 11, 2020 08:44
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.

None yet

3 participants