fix: rename artistBadge ref to hostBadge in ContestCommentsTile#14260
Merged
dylanjeffers merged 2 commits intomainfrom May 7, 2026
Merged
fix: rename artistBadge ref to hostBadge in ContestCommentsTile#14260dylanjeffers merged 2 commits intomainfrom
dylanjeffers merged 2 commits intomainfrom
Conversation
…pre-fill
UI feedback from Julian on the remix-contest surface, batched into one
PR. Each fix below is the minimum change to unblock the report.
WEB:
- Comments overflow kebab moved out of the header row and into the
action bar next to Reply, matching the track-page CommentBlock
layout. The header was wrapping when the user link + timestamp +
badge were wide. (ContestCommentsTile.tsx)
- Comment artist flair restyled to match the track-page CommentBadge
(IconStar + accent text via Flex), and the label changed from
"Artist" to "Host" — contest hosts aren't necessarily the comment-
thread artist, and "Host" reads correctly here.
(ContestCommentsTile.tsx)
- Stems & Downloads card now collapses by default when there are
more than 5 stems and stays expanded when there are fewer — long
lists were pushing the followers + comments tiles way down the
page. The user's explicit toggle (override) wins over the
heuristic, so clicking the caret never gets stomped.
(ContestStemsCard.tsx, applied to both web and native variants)
- Pick-winners "Remixes (N)" heading renamed to "Submissions (N)".
Added a dedicated `pickWinnersSubmissionsTitle` message rather
than reusing `submissionsTitle` (which is the longer "Remix
Contest Submissions" used on the regular remixes page).
(messages/remixes.ts, PickWinnersPage.tsx)
- Submit-to-contest entry points now pre-fill the upload form with
the source track's artwork (fetched as a File for the artwork
input) + genre + remix_of, matching the desktop track-page
RemixContestSection that already did this. Factored into a shared
`useEnterContest(trackId)` hook used by:
* Desktop contest page (was: navigate('/upload?remix_of=…')
— query string ignored by upload page)
* Mobile contest page (same bug)
* Mobile track-page contest details tab (had remix_of but no
artwork/genre)
MOBILE (native):
- Discovery contest carousel: when the CONTESTS flag is on the
section title is "Contests", but the data was still the curated
`featuredRemixContests` subset. Swap to `useAllRemixContests` so
the data matches the label (Julian: "the full list of contests
isn't showing on the mobile discovery page").
(FeaturedRemixContests.tsx)
- Profile Contests tab: bumped page size 50 → 100 so deep-list
hosts (ended contests, smaller accounts — @DimensionX report) are
reached by the auto-pagination effect that was already in place.
No backend change needed; the discovery endpoint doesn't support
host=… filtering, so client-side filter against the global list
remains the only option.
(ContestsTab.tsx)
OUT OF SCOPE (deliberately not touched):
- Native ContestScreen `handleEnterContest` is a stub
(`console.info('Enter contest — native upload flow not yet
wired')`). Wiring up the native upload from the dedicated contest
screen is a separate piece of work — UploadRemixFooter (track
screen) already has the artwork/genre pre-fill and serves as the
reference implementation.
VERIFICATION:
- `tsc --noEmit` clean across `packages/web` and `packages/mobile`
for every file in this PR. The remaining typecheck errors visible
in CI are pre-existing in unrelated files (jupiter/api,
proxy-memoize, type-fest exports, etc.) and unchanged by this PR.
- `eslint` clean on every changed file (mobile shows pre-existing
`react-native` import-resolution warnings that affect untouched
files too).
- Browser preview verification deferred to the apps-3 worktree per
the user's stored preference.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Contributor
🌐 Web preview readyPreview URL: https://audius-web-preview-pr-14260.audius.workers.dev Unique preview for this PR (deployed from this branch). |
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
1 participant
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.
One-line typecheck fix missed when renaming Artist to Host in #14253.
messages.artistBadgeon line 818 should bemessages.hostBadge.