Some Polishing#63
Conversation
Removed AI Title Helper and Profile Editor from tools list.
|
@Coder-soft is attempting to deploy a commit to the yamura3's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis PR adds a new GreenVoidPopup dialog component wired into ResourcesHub, removes hover-to-play behavior and its localStorage-backed state/UI from ResourceCard and Account, adds a minecraft font stylesheet link and Tailwind font-family entry, and rewrites README.md into a condensed format. ChangesGreen Void Popup and hover-to-play removal
Minecraft font and README rewrite
Estimated code review effort: 2 (Simple) | ~12 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR polishes the resources experience and project metadata. The main changes are:
Confidence Score: 5/5This looks mergeable after small cleanup to the popup sequencing and animation preview preference behavior.
src/components/resources/GreenVoidPopup.tsx, src/components/resources/ResourceCard.tsx
|
| Filename | Overview |
|---|---|
| src/components/resources/GreenVoidPopup.tsx | Adds the Green Void announcement dialog, but its localStorage gate can defer the popup until a later remount. |
| src/components/resources/ResourceCard.tsx | Changes animation cards to render autoplaying previews once in view and removes the hover preference gate. |
| src/pages/Account.tsx | Removes the Hover to Play Videos preference UI and related state. |
| src/pages/ResourcesHub.tsx | Mounts the new Green Void popup alongside the existing Minecraft changelog popup. |
| index.html | Adds the external Minecraft font stylesheet. |
| tailwind.config.ts | Adds the minecraft font family key used by the new popup. |
Comments Outside Diff (1)
-
src/components/resources/ResourceCard.tsx, line 205-218 (link)Stored Playback Opt-Out Ignored
Animation cards now mount an autoplaying muted video as soon as they enter view, even for users who previously saved
hoverToPlay=false. Since the Account toggle was removed too, those users lose the only opt-out and can get unexpected motion and video fetches while scrolling the resource grid.
Reviews (1): Last reviewed commit: "Update tool descriptions in README.md" | Re-trigger Greptile
| if (!greenVoidDismissed && mcPlaylistDismissed) { | ||
| setOpen(true); | ||
| } |
There was a problem hiding this comment.
Prerequisite Dismissal Is Stale
For a first-time ResourcesHub visitor, the Minecraft popup opens while mcPlaylistDismissed is still absent, so this condition keeps Green Void closed. Dismissing the Minecraft popup in the same session updates localStorage, but this effect does not run again, so the new Green Void announcement is skipped until a later remount.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation