CMU-CREATE-Lab / flutter-app-android

5 stars 3 forks source link

Create Wizard for Buzzer #237

Closed tasota closed 6 years ago

tasota commented 6 years ago

Breaking up #197 into smaller issues.

tasota commented 6 years ago
jenncross commented 6 years ago

I think we need to discuss a bit how to handle the dual volume-pitch nature of buzzer. Since the Wizard model doesn't lend itself to the current toggling method.

Volume Dialog: https://app.zeplin.io/project/57f43918d127f7023581eaab/screen/57f4398abcb58ed62c8ab8f0

Pitch Dialog: https://app.zeplin.io/project/57f43918d127f7023581eaab/screen/57f439882aa9cdb466f7ac18

jenncross commented 6 years ago

@ehamner @lczito @tasota @pdille --- Can we discuss we want to approach the wizard for the buzzer? (This might be a good in-person discussion as it is a bit complex.)

Do we want to: A) Have separate wizards for pitch and volume (and when we make a pitch link, it automatically creates a volume link and vice versa) B) Have a single wizard where you first make a choice for whether the relationship you are creating is for pitch, volume or both C) Have a single wizard where we always create two links (pitch and volume) during the process, so sensors are selected twice, relationship type is selected twice, etc. D) Other?

tasota commented 6 years ago

Just wanted to ping everyone about this again because it will be coming up again soon. Thoughts?

jenncross commented 6 years ago

I lean towards option B above:

Have a single wizard where you first make a choice for whether the relationship you are creating is for pitch, volume or both

That first screen (the pitch/volume/both choice) of option B is currently missing from the Zeplin, but I think is simple enough to envision?

tasota commented 6 years ago

Sounds like we will have an option for which output you want to link (volume, pitch, or both). For both option, be sure to include a page that says "now we are going to set the VOLUME/PITCH" so the user understands what they are about to set the relationship/sensor/min/max for.

Jenn's scans from the discussion:

img_20180718_152443

img_20180718_152448