Repository navigation
feat(web): TrackRow on search track results behind track_rows - #14686
Merged
Merged
Conversation
|
dylanjeffers
force-pushed
the
design/track-row-feed-profile
branch
from
October 20, 2026 00:14
9a38655 to
f78973c
Compare
dylanjeffers
force-pushed
the
design/track-row-search-collections
branch
2 times, most recently
from
October 20, 2026 00:19
fb5fc2a to
dab0de6
Compare
dylanjeffers
force-pushed
the
design/track-row-feed-profile
branch
from
October 20, 2026 00:19
f78973c to
43e9d12
Compare
The profile page passes the row variant to its Tracks and Reposts lineups when the `track_rows` flag is on. `TrackRow` gains an optional `context` line above the row, which the connected row uses to label the artist pick. In row mode the lineup drops the two-column grid it uses for small tiles so rows span the full width. Feed stays on tiles on purpose: a feed item is a post with who reposted it and inline playlists, and the tile carries that better than a row. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Reposts mixes playlists into the track lineup. With rows the cards sat flush against the row above; give them a top margin in row layout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Search's track results pass the row variant to TrackLineup when the `track_rows` flag is on, and drop the tile spacing styles in row mode. Collection pages are left on their tracks table: it is already a dense 48px row with sortable columns and drag reorder in edit mode, which the row does not replace. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
dylanjeffers
force-pushed
the
design/track-row-feed-profile
branch
from
October 21, 2026 23:05
43e9d12 to
326e804
Compare
dylanjeffers
force-pushed
the
design/track-row-search-collections
branch
from
October 21, 2026 23:05
dab0de6 to
2fee8ee
Compare
dylanjeffers
added a commit
that referenced
this pull request
Oct 21, 2026
…14689) ## What Stacked on #14686. Collection pages keep their sortable, reorderable tracks table, but with `track_rows` on the table takes the TrackRow look. `Table` gets an `isDense` prop (a `.dense` modifier on the wrapper, applied on both the virtualized and plain render paths): - flat surface: no card shadow or white background; the page shows through - 56px rows (was 64), hover and focus-within fill with `surface-1` instead of a box shadow - active row title in the accent text color - header in sentence case at 12px/500 subdued, with a single divider under it - tabular numerals in cells - `TracksTable`: 40px artwork without a border, subdued artist line, flat disabled and locked rows `CollectionPage` reads `FeatureFlags.TRACK_ROWS`, passes `isDense`, and makes its outer wrapper transparent so the hover fill is visible. Sorting, filter, drag reorder, remove and edit mode are untouched. Library and dashboard tables are not changed in this PR; they can opt in with the same prop. ## Verified - eslint, stylelint and `tsc` clean for web - Dev server, flag forced on, Hot & New on Audius playlist: 13 tracks per screen (was 10), hover reveals repost, favorite and menu with the surface fill, light and dark. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
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.
What
Stacked on #14685. Search track results (the Tracks tab and the tracks section of All results) render Harmony
TrackRowwhentrack_rowsis on. Default off, no change otherwise. Mobile keeps tiles. The desktop tile spacing styles are skipped in row mode so rows stack flush.Collection pages: left as they are
The collection page uses
EditAwareTracksTable, which is already a dense row layout: 48px rows, sortable Added/Plays/Reposts columns, hover-revealed play and actions, and drag reorder plus remove in the owner's edit mode. Replacing it withTrackRowwould lose sorting and reordering for no density gain. If we want the two to match visually, the follow-up is to restyle the table's row to the TrackRow tokens, not to swap the component.Verified
tscand eslint clean for web/search/tracks?query=lofirenders 11 rows per screen at full width with artwork, badges, favorites and durations.🤖 Generated with Claude Code