No "visible" login nav menu for mobile

オープン 初心者向け
#3 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
68/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
静か
技術スタック
css
領域
frontend

調査の方向性

モバイル版のSafariまたはChromeを使用して、https://rss.chat/ のナビゲーションバーを幅445pxおよび444pxのレスポンシブビューで再現します。.navbar .nav.pull-right の float の動作を調査し、幅444pxでログインメニューが表示されたまま正しく配置され、より広いレイアウトを壊さないことを確認します。

索引モデルが issue の本文から書いたものです。

説明

Observed this with both mobile Safari & Chrome on my iPhone 17 Pro.

To replicate:

  1. Open https://rss.chat/ with Chrome Inspect.

  2. Select responsive view for dimensions.

  3. Set width to 445px.
    The login menu is visible.

  4. Drop width to 444px.
    The login menu "disappears".

At 444px the login menu has no room, so it's forced below the other nav items (why the container gets taller) and for some reason the float style gets wonky. Image

  1. Toggle off the float style (.navbar .nav.pull-right {float: right;}).
    The login menu will "reappear".
Image

A quick hack might be to override and remove the original float style, then add a media query: @media (width >= 444px) { .navbar .nav.pull-right { float: right; } }.

I hate floats, IMO CSS Flexbox is better for nav bars. Ask Claude to rework the nav to use Flexbox and see if you don't like that better Dave.

Also, again IMO, I like all major page elements to have an id, even if styling is done with classes. I think it helps when debugging.

主要言語
JavaScript
スター
37
フォーク
7
30日以内にマージされた PR はありません

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

scripting/rss.chat のほかの issue

scripting/rss.chat の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。