Repository navigation
feat(trending): use the /skills shell with a range sidebar - #433
Merged
Merged
Conversation
Harlan asked for the trending board to run full width like /skills, with the week, month, and all time switch in a left sidebar and no right aside. The sidebar carries the range links, the weekly CTA, and the skilld install on wide screens. Narrow screens keep the range row above the board and the CTA after the fifth row. Rows now switch on their own width, so the posts sit beside the Skill only when both fit, and the carousel shows two posts where it has room. "Earlier on this board" stays under the board. It lists Skills that left the ranking, not earlier weeks, and it is the crawl path to every Skill the trending sitemap lists.
harlan-zw
added
the
harlan-agent-review
Collaborator
Author
|
Agent check of the rendered page. Before is production; After is the local dev server with seeded D1 data, headless Chrome. No horizontal scroll at 390, 1100, or 1440, and html-validate passed for all three ranges.
|
harlan-github-agent
Bot
added
harlan-agent-running
Contributor
🤖 MERGED
GitHub merged this pull request. No material findings were recorded. The pull request closed. |
harlan-github-agent
Bot
added
harlan-agent-running
2 of 6 tasks
harlan-github-agent
Bot
removed
the
harlan-agent-review-required
2 of 6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Linked issue
Related to #431
❓ Type of change
📚 Description
Harlan asked for
/skills/trendingto run full width like/skills, with a left sidebar to switch between week and month and no right-hand aside. The page now uses the/skillsshell. The heading lines up with the results column, a sticky 14rem sidebar sits on the left, and the board takes the rest of the width.Last 7 days, and the current one carriesaria-current. Narrow screens show the same links as a compact row above the board.Teach your agent skilldinstall sit under the range links. Narrow screens keep the CTA after the fifth row and the install under the heading. The sidebar install reports copies astrending-sidebar; the narrow one keepstrending-header.TrendingWeeklyCtadrops its unusedrowlayout. In a container narrower than 16rem it tightens its copy, so the sidebar button keeps its whole label.TrendingBoardItemswitches on its own width with a container query. Posts sit beside the Skill once the row is 52rem wide, and the posts column grows with the board. Descriptions cap at 75ch, so rows with no posts stay readable at full width.TrendingPostCarouselshows two posts at a time once it is 36rem wide. When exactly two posts fit side by side, the step controls hide.DESIGN.mdgains a 2026-10-06 decision. It supersedes the 2026-10-01 call that dropped the 9/3 sidebar.Page question: what are devs talking about this week or month? Primary action: copy the run command. Owning concern: Loop 1.