fix: rename brand text split across markup - #687
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe default “taskito” brand label changes to “FlexiQ” in the dashboard sidebar and documentation site navigation. Custom sidebar branding remains unchanged, and existing accent styling is preserved. ChangesBranding update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized brand-text update changes only the displayed wordmark in two files, with no actionable merge-blocking risk remaining after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
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 |
0d2232d to
4dd0feb
Compare
Two brand strings still read "taskito" because the name was split across an element boundary, so no search for
taskitomatched them:dashboard/src/components/layout/sidebar.tsx—task<span>ito</span>docs/app/components/ui/site-nav.tsx—task<span class="bto">ito</span>Both now render
Flexi<span>Q</span>, keeping the accent on the final glyph exactly as before and matching the wordmark treatment from #686 and thesite.namedefault, which is alreadyFlexiQ.Same class of miss as the wordmark SVG in #686. A repo-wide grep for the split pattern turns up no further instances.
Biome and typecheck clean on both packages.
Summary by CodeRabbit