feat: except for the TinyRobot page, other pages have hidden playground buttons#94
feat: except for the TinyRobot page, other pages have hidden playground buttons#94wuyiping0628 wants to merge 2 commits intodevfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughRenames and types the internal medium-zoom instance, adds route-aware logic that toggles the Changes
Sequence Diagram(s)sequenceDiagram
participant Router
participant ThemeInit as Theme Init
participant MediumZoom as medium-zoom
participant DOM as document.body
Router->>ThemeInit: route change / initial run
ThemeInit->>MediumZoom: if existing -> cleanup zoom instance
ThemeInit->>MediumZoom: create new zoom instance (assign to `zoom`)
ThemeInit->>DOM: if path includes "tiny-robot" -> remove `hide-code-playground`
ThemeInit->>DOM: else -> add `hide-code-playground`
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Summary by CodeRabbit
Style
Bug Fixes