feat: Improve add shopping list item form#7091
feat: Improve add shopping list item form#7091miah120 wants to merge 2 commits intomealie-recipes:mealie-nextfrom
Conversation
|
Looks like I made the quantity field a bit too small. I was focused on the add item form, but w the edit item form all you can see of the label is Q... I'll fix that tonight |
|
It looks good! The spacing is better managed. The mobile view is interesting and more intuitive. A quick note about the field displayed at the bottom on mobile devices: the two search/food icons can clutter up the interface, so I would choose just one on the left. |
michael-genson
left a comment
There was a problem hiding this comment.
Overall looks great, definitely a huge improvement for mobile and a good improvement for desktop (other than the broken quantity field ;) )
This might be an issue with the auto focus logic I just merged, but on mobile when I tap into the food field to add a new item the food menu appears before the drawer opens up, so it covers the dialog:

Tapping out and back in it's fine:

Similar to what @p0lycarpio said I don't think we need both the magnifying glass and the apple icon, maybe just use the apple icon where the magnifying glass is? I recognize that it's a search field but I think the food icon conveys that you're searching for a food well enough on its own.
The "Add item" text works when it's by itself at the bottom, but once the full dialog is open it's weird to see "Add item" (I am already adding an item). Could we switch it back to "Food" (or whatever) once we open it up?
Also, and I don't know if this is actually a good suggestion, but could we adjust the style of the food input field once the drawer opens up? When it's by itself at the bottom I like how it looks, but once the dialog opens up it's weird to see it to be a different shape than the rest of the fields. Would it make sense to change the shape of the field back?
ee1a3da to
f867c8e
Compare
|
Okay here's the followups enumerated:
Some additional tweaks I made while addressing these:
(after writing this I feel like this has AI tells in it, I swear I just use a lot of emojis) |


What this PR does / why we need it:
This PR updates the add item form in the shopping list with a few changes:
Before:

After:

Which issue(s) this PR fixes:
(REQUIRED)
NA
Special notes for your reviewer:
NA
Testing
Tested in Firefox and Android