Redesign of nutritional plan widget #1032
Conversation
| I'm back and have some time to finally look at all your PRs |
| width: 48, | ||
| height: 48, | ||
| padding: const EdgeInsets.all(12), | ||
| child: Image.asset( |
There was a problem hiding this comment.
I would leave the original icons we were using so the style is more consistent.
There was a problem hiding this comment.
I was planning to refine all the icons with upcoming PRs. But I don't mind first doing the full redesign and then creating a separate PR where all the icons are being redefined at once
There was a problem hiding this comment.
creating a separate PR where all the icons are being redefined at once
that would be my suggestion. I will also write you on discord about the icons, because there's a handful new ones we will need
| child: Container( | ||
| padding: const EdgeInsets.symmetric(horizontal: 5, vertical: 15), | ||
| child: FlNutritionalPlanGoalWidget(nutritionalPlan: _plan!), | ||
| return Padding( |
There was a problem hiding this comment.
I would leave the card with the same color as the others, so it doesn't stand out (we can of course talk of changing this, but then we'd need to do it for all of them)
2 participants
Proposed Changes
I know that this PR itself doesn't appear like a well-rounded design language yet. However, it is part of broader redesign efforts that I will perform throughout the following weeks. I want to keep the PRs small and digestible for a streamlined review process.
Related Issue(s)
Not applicable
Screenshots