Skip to content

[#113] Reorder widgets above story content on mobile#123

Merged
realproject7 merged 1 commit intomainfrom
task/113-mobile-widget-ordering
Mar 15, 2026
Merged

[#113] Reorder widgets above story content on mobile#123
realproject7 merged 1 commit intomainfrom
task/113-mobile-widget-ordering

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Adds order-first lg:order-none to the sidebar <aside> on the story page so engagement widgets (trading, price chart, ratings, donations) render above story content on mobile viewports
  • Desktop two-column layout is completely unchanged — the ordering classes only take effect below the lg breakpoint
  • Single-line Tailwind class change; no structural or behavioral changes

Fixes #113

Test plan

  • Open a story page on a mobile viewport (< 1024px) — widgets should appear before story text
  • Open the same page on desktop (>= 1024px) — layout should be identical to before (story left, widgets right)
  • Verify no visual regressions on the story page at various breakpoints

🤖 Generated with Claude Code

Add order-first lg:order-none to the sidebar aside element so
engagement widgets (trading, price chart, ratings, donations) appear
before story content on mobile viewports while keeping the desktop
two-column layout unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

T2b Review: APPROVE

Elegant single-line fix. Parent is a grid container so order works correctly. order-first surfaces widgets above story on mobile; lg:order-none restores DOM order on desktop (both siblings at order: 0). No new components, no JS, no desktop regression risk. Cleanest possible solution for the UX problem.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

The PR implements one of the issue #113-approved mobile fixes by reordering the widget sidebar ahead of story content below the lg breakpoint. Desktop layout remains unchanged because the order override is explicitly removed at lg and above.

Findings

  • None.

Decision

Approve because the implementation matches the acceptance criteria, stays narrowly scoped, and the lint-and-typecheck check passed.

@realproject7 realproject7 merged commit 50b75d6 into main Mar 15, 2026
1 check passed
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.

[Bug] Mobile story page — widgets buried below full story content

2 participants