Add new category flow

[syedbarimanjan]

A video showing the changes in action:

simplescreenrecorder-2025-02-13_19.34.10.online-video-cutter.com.mp4

Fixes #1734
/claim #1734

[syedbarimanjan]

@zachgoll can you please approve the workflow and review.

[zachgoll]

@goldenstein64 would you mind opening a new issue for this? I agree with your suggestion, but since it applies to all of our modals / drawers in the app, it's probably worth a standalone issue.

[syedbarimanjan]

@zachgoll

simplescreenrecorder-2025-02-18_05.07.03.mp4

[syedbarimanjan]

@zachgoll

simplescreenrecorder-2025-02-19_22.05.57.mp4

[zachgoll]

Overall looking a lot better! Core flow works great.

Before we merge, I think there are a few general "cleanup" items to address (several mentioned in comments):

Model validation / migration

Currently, the Category class allows nil for a category. Per @justinfar comments about the shapes icon being our default, we'll need to:

• Add a default value for lucide_icon of shapes on the Category table
• Add a NOT NULL constraint on lucide_icon

Edge Cases in UI

• When you edit a subcategory, users can change the color to be different than the parent. This is because we have before_create :inherit_color_from_parent in the model, but really, we need this to be before_commit to cover both create + update flows + hide the color option in the UI if it is editing a subcategory.
• When a user selects a preset color, then choose a custom color, the preset color remains selected in the UI (see video below):

CleanShot.2025-02-19.at.15.01.50.mp4

[syedbarimanjan]

@zachgoll

[zachgoll]

After all comments are addressed and merge conflicts resolved, we'll run the tests and get this merged.

Functionality looks good!

[zachgoll]

Code looks good! Looks like there is a failing check, so once resolved we're good to merge.

[syedbarimanjan]

@zachgoll resolved the failing check and merged with main to resolve conflicts.

[zachgoll]

@syedbarimanjan great, nice work! Let me know if you have any issues with the bounty payment.