Caffeine-rohit
Caffeine-rohit
@bogminic @bkimminich First PR up: #2933 Started with navbar - Various spacing values migrated (2px, 4px, 8px, 10px). Tested across desktop/tablet/mobile, no visual regressions. Following the approach we discussed -...
> Bonus: Ideally, this would be even an auto-fixable rule where the nearest predefined size is chosen. Agreed - definitely need tooling to prevent regression after this migration. Here's what...
> The basic rule is totally fine for the time being! 👍 Great, sounds good. I’ll wire up the basic rule in the final migration PR then. We can revisit...
> Great we have a plan! 🎉 @bogminic - got the navbar and contact component PR up. If the pattern looks good to you, I'll use the same approach for...
@bkimminich Good catch on the naming inconsistency. Let me address your questions directly: **On the current naming (`ms`, `xls`):** You're right to be confused - there's no clear logic. `ms`...
@bkimminich @bogminic Got it - t-shirt sizing it is. Makes sense to keep the semantic naming, especially with `space-m` as the intuitive base. Given the feedback, here's what I'm thinking...
> This works for me as well. Thank you, @caffeine-rohit, for the suggestion! Thanks for this, @bogminic ! Glad to hear it works for you as well. **This approach looks...
> Let's roll with it for now. It's already on `develop` branch, so all PRs can be updated to reflect the new values. Perfect! I’ll align upcoming PRs with these...
@bkimminich @bogminic ## Frontend Modernization - Progress Update Fixed input field focus visibility by switching to accent color theme variables. Labels and indicator lines now properly match the theme palette...