Skip to content

Add individual story view with share portal layout and navigation#915

Merged
maebeale merged 3 commits intostory-share-portalfrom
copilot/sub-pr-902-another-one
Feb 10, 2026
Merged

Add individual story view with share portal layout and navigation#915
maebeale merged 3 commits intostory-share-portalfrom
copilot/sub-pr-902-another-one

Conversation

Copy link
Contributor

Copilot AI commented Feb 10, 2026

What is the goal of this PR and why is this important?

Individual story pages need to maintain the share portal's navigation and visual consistency when accessed from the portal context.

How did you approach the change?

New show_share_portal action and route:

  • Added member route stories/:id/show_share_portal
  • Action uses share_portal layout and fetches 4 related stories ordered by bookmark count

View structure:

  • Hero section: story image with sector badge overlay and semi-transparent title bar
  • Content section: story metadata and body
  • "What Others Are Reading": 4 related stories with sector-colored overlays
  • "Get Involved" CTA with purple buttons

Link updates:

  • All story links in share_portal.html.erb now route to show_share_portal_story_path to maintain portal context

Example usage:

<%= link_to show_share_portal_story_path(story), class: "..." do %>
  <!-- Story card content -->
<% end %>

Anything else to add?

The page reuses the same navigation header (_share_portal_navbar.html.erb) and styling patterns from the main portal page for consistency.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Feb 10, 2026
Copilot AI changed the title [WIP] Work in progress on story share portal Add individual story view with share portal layout and navigation Feb 10, 2026
Copilot AI requested a review from maebeale February 10, 2026 14:14
Copilot AI and others added 3 commits February 10, 2026 09:26
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
@maebeale maebeale force-pushed the copilot/sub-pr-902-another-one branch from 19959d3 to 1c0548a Compare February 10, 2026 14:27
@maebeale maebeale marked this pull request as ready for review February 10, 2026 14:27
@maebeale maebeale merged commit f89ba21 into story-share-portal Feb 10, 2026
3 checks passed
@maebeale maebeale deleted the copilot/sub-pr-902-another-one branch February 10, 2026 14:27
maebeale added a commit that referenced this pull request Feb 11, 2026
* Initial plan

* Add show_share_portal page with matching layout and navigation

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Improve popular stories query and sector fallback

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
maebeale added a commit that referenced this pull request Feb 11, 2026
* Initial plan

* Add show_share_portal page with matching layout and navigation

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Improve popular stories query and sector fallback

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
maebeale added a commit that referenced this pull request Feb 12, 2026
* Initial plan

* Add show_share_portal page with matching layout and navigation

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Improve popular stories query and sector fallback

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
maebeale added a commit that referenced this pull request Feb 12, 2026
* WIP: story share portal

* WIP: show page for story

* Isolate Story Share Portal with dedicated purple-themed layout (#903)

* Initial plan

* Redesign share_portal page with Tailwind CSS to match design

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Add full-width layout support for share_portal page

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Address code review feedback: add accessibility labels and use dynamic story links

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Create separate layout for story share portal with purple navigation

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Address code review feedback: remove redundant nav wrapper and duplicate link

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Redesign Story Share Portal to match visual specifications (#904)

* Initial plan

* Add Telefon font, redesign navbar with 3 rows, update sector styling and overlays

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Refine font declarations and remove unused CSS variables

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Fix Tailwind class interpolation and improve scroll performance

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Use actual story sectors for category badges in What Others Are Reading

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Add new logo image and merge conflict

* Redesign Story Share Portal to match visual specifications (#905)

* Initial plan

* Update story share portal navigation and layout to match design

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Force first 5 stories for stories_by_focus for now

* Refine Story Share Portal layout and visual hierarchy (#911)

* Initial plan

* Update Story Share Portal layout per feedback

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Visual tweaks

* WIP: messing w hero height

* Fixes

* Refine What Others Are Reading section colors and alignment (#912)

* Initial plan

* Update What Others Are Reading section colors and alignment

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Constrain hero carousel height and stretch to full width on story share portal (#913)

* Initial plan

* Limit hero image height to prevent vertical growth

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Allow extra classes to be sent in

* Don't hard-code the size

* Stretch hero image to full width horizontally

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Hard-code nav item colors

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Co-authored-by: maebeale <maebeale@users.noreply.github.com>

* Add individual story view with share portal layout and navigation (#915)

* Initial plan

* Add show_share_portal page with matching layout and navigation

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Improve popular stories query and sector fallback

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>

* Fix test

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
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.

2 participants