Landscape phone 844×390: Reply reach in the conversation (rest of #478)
メンテナーはふだん 1 日以内に返信
評価
この issue はまだ評価されていません。
説明
Rest of #478. Its header repair (Open navigation, PR #485) is separate; this issue carries the one acceptance criterion that PR cannot meet. Owner: claude-maurycy (#151 adaptive layouts). Coordinate with #341 / PR #414 and #340 / PR #372, which change the phone header, tab bar and composer.
Evidence (current main 7babb7ef, merged with the #485 head; Chromium)
./scripts/check_ui.sh test_adaptive_matrix gives Ran 20 tests, FAILED (failures=1): test_01_the_same_journey_works_at_every_fixture_width at 844×390 only.
a message's Reply at 844×390 is not clipped or covered: {'inside': True, 'hit': False, 'covered': {'top': 'button.ui-btn.ui-btn--quiet', 'bottom': 'div.composer.project-convo__composer'}, ... 'box': [731, 179, 46, 44]}
a message's Reply at 844×390 offers a 44 px touch target: {'size': [46, 44], 'across': 46.5, 'down': 34.5, 'stops': {... 'up': 'button.ui-btn.ui-btn--quiet', 'down': 'div.composer.project-convo__composer'}, 'overlay': {... 'up': None, 'down': '.project-convo__composer'}}
Reviewer PelikanFix16 reproduced the same failure on an unmodified main (bb63), so it is not a regression of #485.
Cause
At 844×390 (the tablet shell: sidebar, a 131 px project header, the 45 px views row, a 164 px composer) the conversation stream is 34 px tall, so a 44 px Reply cannot fit in it. scrollIntoView leaves the button 5 px above the stream's top edge, where the views row (its "What matters" button, .views__recap) is hit instead. The recorded known gap for this size names .ui-tabs__bar and the composer as the covering elements (KNOWN_GAPS, docs/design/adaptive-layout-rules.md, waiting for #414); the "What matters" button is neither, so the gap no longer excuses it. A z-order change cannot help: the stream is a clipping scroller shorter than the target. The same stream is 10 px tall at 640×360 and is still excused by the recorded gap.
Acceptance criteria (unchanged from #478)
test_adaptive_matrixpasses in Chromium: all 20 tests, with the Reply guard andKNOWN_GAPSrules not weakened and no gap exemption added. A gap that the work closes is removed fromKNOWN_GAPSand from the table indocs/design/adaptive-layout-rules.md, as the test's own message requires.
(Acceptance 1, Open navigation at 44×44 px light and dark, and 3, the negative control, are met by #485.)
Refs #478, #151, #341, #340, #485
- 主要言語
- TypeScript
- スター
- 3
- フォーク
- 0
- 平均マージ
- 19時間 44分
- マージ済み PR（30日）
- 367
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ColdPhase/flux のほかの issue
-
Task, result and decision cards in the thread panel show only the first characters of their titleオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
Unit tests for project doc text helpers in core: sourceLine, dateLine, sectionSource, describeChangeオープンgood first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 93/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
awesome-dsh-plugin/awesome-dsh-plugin#9184 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
farion1231/cc-switch#8829 ·
メンテナーはふだん 1 日以内に返信
-
[core-client] Remove unused hand-maintained transport helpers after the definition-table cutoverオープンdoc-audit enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100