dcramer / peated

https://peated.com
Apache License 2.0
64 stars 13 forks source link

Improve Flights #77

Open dcramer opened 1 year ago

dcramer commented 1 year ago

Built a quick MVP, but we want some more details:

  1. Should keep you within the flight details as best as possible, that is it should be easy to record a tasting/etc and return to the flight.
  2. Should capture the flight as part of the tasting details.
  3. Should show other people who have participated in the flight, maybe some stats?
  4. Maybe include a way to rate a flight?
dcramer commented 1 year ago

https://peated.app/flights/8xuhjsjpgie7

Sentry hosted an event in SF for employees and did a tasting, so something like this would have been nice. Especially if we enrich the display and make it more fluid.

ghost commented 1 year ago

Hi @dcramer,

I've thoroughly analyzed Issue #77 regarding the enhancements to the flight tasting MVP. The requirements are clear, and I can see the potential these improvements offer, especially in light of the Sentry event in SF.

To tackle this, I've come up with two possible approaches:

Approach 1:

Approach 2:

Does this align with your vision for the improvements? If these suggestions resonate, I'm ready to delve into this issue. Could I kindly request assignment, and if there are more specifics or details you'd want, please share?

Thanks for your consideration! Muditya Raghav

dcramer commented 1 year ago

I haven't given it much thought, and have mostly been incremental with changes given its more or less just beta right now.

One note is that a lot of he UI challenges come from the need to support both mobile and desktop flows, and those aer quite often very different. I haven't gotten a good system down that keeps it stable when making changes, and even right now I think there are some mobile bugs.

dcramer commented 11 months ago

Few updated thoughts:

  1. Flight details should showcase which flights you've already captuerd tasting notes on.
  2. Clicking a bottle should pop a modal/slideout (we dont have UI for this yet, just rough thinking) - keeps you on the page more eaisly.
  3. Slideout+table should probably show a "Record Tasting", and disable that once you've recorded it for the given row.
dcramer commented 11 months ago

I added the slide out + tracking of flights with tastings. Still need to have it showcase which ones you've recorded already, and pull in some interest stats about flights. Almost feels usable though!

image

Oh, and def needs some mobile improvements on the slideout pattern. A lot of work I need to do on component standardization yet so its easier to make stable UI.