-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add song preview to contest entry for submitted beatmap #11225
base: master
Are you sure you want to change the base?
Add song preview to contest entry for submitted beatmap #11225
Conversation
This removes the icons for the individual gamemode. If it's required to show an icon for the gamemodes for visibility I'm not sure where to put it... |
winnerVotes: number; | ||
} | ||
|
||
export const Entry = (props: Props) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this function is too long and contains too many different things
(the original wasn't readable either and this conversion isn't helping at all if not making it a bit more verbose)
<TrackPreview | ||
track={{ | ||
coverUrl: `https://b.ppy.sh/thumb/${props.entry.preview}.jpg`, | ||
preview: props.entry.preview, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is very wrong
Resolves: #11208
Added track preview with cover image in contest entry.
Refactored to typescript