search

nickslick03 / messiah-meal-planner

1 stars 0 forks source link

Tutorial #41

Closed cdrice26 closed 3 months ago

cdrice26 commented 3 months ago

Added the tutorial modals. Implementation is pretty straightforward, I just created a TutorialModal component and a static file to hold the markup for each section. Implements #35.

nickslick03 commented 3 months ago

looks good, I added some height and width constraints so it presents nicely on a desktop display. Totally mergable bro.

nickslick03 commented 3 months ago

Also, the tutorial doesn't pop up for new users, is that intentional? I remember we originally planned on having it pop up.

cdrice26 commented 3 months ago

Oops, yeah that was my bad, I forgot about making it appear on first launch and I also should have known better than to create this PR at the same time as the charts one since that had so many extensive modifications...but anyway, it now shows up on first launch. Let me know what you think, I'm good to merge at this point whenever you are.

nickslick03 commented 3 months ago

Looks good! Just wondering, is there a reason you started on the available meals section instead of the meal info section?

cdrice26 commented 3 months ago

Oops, yeah no, that wasn't right. It starts on Meal Plan Info now:)