youngcw

Nice. I feel like the background color should be darker but everything else looks good

The color of the upcoming schedule pill seems too light. I think the other pill colors are the same as dark mode except for that one. Is that on purpose?

> I can forsee that the theme list could get crazy long. We'll need a clever way to manage a multitude of themes to avoid that in the future. I...

Notes: * Cashflow report has a hard coded black 0 line. That should be changed probably * Custom Report options toggles are missing a border, and look weird when off....

> Honestly, I don't see a need for the icon to be on the title bar. Most users don't have a need to change the theme on a regular basis....

You need to change the timestamp on that migration. Importing doesn't work since the simplefin migrations have a newer time

It doesn't seem like the reports are saving. If I create a test file then try to make a report it doesn't seem like anything happens other than the name...

the saved reports have a white border around their dashboard cards where the other cards dont The table card is really tight and only fits one category of data. Maybe...

I don't dislike the border, it just makes the space for data really tight. I would be ok if the cards were bigger and maintained the border.

> Okay, the tableGraph cards should be more visible. LMK what you think! Nice. That looks pretty good. I tested with a whole year of data. It cant show everything...