search

nickslick03 / messiah-meal-planner

1 stars 0 forks source link

Added sorting logic and associated unit test #24

Closed cdrice26 closed 1 month ago

cdrice26 commented 1 month ago

Added sorting logic. Not entirely sure how it should behave though. As of now, the sorting preferences do not persist due to the current scope of the sorting variables as state in the MealContainer (which will not work for this given the way localStorage works, we would need to refactor if we want this). The user also must sort each table individually, not sure if this control is the best or if it would be better if all the tables sorted automatically the same way based on a global setting, or maybe if there would be a "sort all tables" checkbox in the modal. Thoughts on these issues?

nickslick03 commented 1 month ago

honestly, I think not saving the sorting preferences and having individual sorting preferences are both fine. In a future sprint, I want to implement a search bar for quicker meal access alongside sorting.