Skip to content

Add meal type selection picker and dialog#386

Merged
aimok04 merged 16 commits into
aimok04:mainfrom
jonsch318:meal-type-sheet
May 1, 2026
Merged

Add meal type selection picker and dialog#386
aimok04 merged 16 commits into
aimok04:mainfrom
jonsch318:meal-type-sheet

Conversation

@jonsch318

Copy link
Copy Markdown
Contributor

This is a kinda large Pr since I needed to add 2 new Form Field types, but I think you will like it.

  1. Added a new FormField type MealTypePickerField which currently replaces the MealTypeSearchField which could not probably be removed. This opens a bottom sheet allow you to select the meal-type by just clicking.
    The rational behin this is that most of the people will reasonably only have a few meal-types

  2. Add a new MealTypeCreationAndEditDialog which allows the app to create, edit and delete new meal types. This is integrated in to the new PickerField as well as i could.

  3. The dialog needed a color chooser and a time field both of which were added

Now here is a complete show of the new stuff

screen-20260416-221927-1776370717452.mp4

@aimok04

aimok04 commented Apr 18, 2026

Copy link
Copy Markdown
Owner

This would make #373 and #372 obsolete, right?

@jonsch318

Copy link
Copy Markdown
Contributor Author

Somewhat yes if we decide to completly replace it but then I should delete the old Input file as well.
Can do in an hour

@jonsch318

Copy link
Copy Markdown
Contributor Author

Now removed the old search field. I'll close the other ones since I find this way nicer.

@aimok04 aimok04 force-pushed the meal-type-sheet branch from f748bd1 to 29df100 Compare May 1, 2026 20:05
@aimok04 aimok04 merged commit 822d06d into aimok04:main May 1, 2026
@aimok04

aimok04 commented May 1, 2026

Copy link
Copy Markdown
Owner

Thanks!

@jonsch318

Copy link
Copy Markdown
Contributor Author

SegmentedListItem is a nice change

@aimok04

aimok04 commented May 1, 2026

Copy link
Copy Markdown
Owner

@jonsch318

Thanks!

Just noticed that you already updated your fork - I did a mistake which lead to the app not compiling so I reverted the original release: 2.1.0 commit because I thought nobody noticed yet 😅

@jonsch318

Copy link
Copy Markdown
Contributor Author

@aimok04 well yes I just came back home at night and saw the merged PR and then it was the last thing before being to tired. I refetched and rebased them again :D thanks for letting me know

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants