search

JakubKoralewski / lets-watch-it-together-private

Unconnected fork of LWIT. Private for the duration of SCRUM project.
https://lets-watch-it-together-scrum.herokuapp.com
MIT License
1 stars 0 forks source link

feature(welcome): add stepper component #14

Closed JakubKoralewski closed 3 years ago

JakubKoralewski commented 3 years ago

https://user-images.githubusercontent.com/43069023/105721658-1d018680-5f25-11eb-8fde-952203bee9cd.mp4

both for mobile and desktop

Closes #3

Yurii17K commented 3 years ago

change 3 to "choose a friend to meet" and 4 to "create a meeting"

JakubKoralewski commented 3 years ago

change 3 to "choose a friend to meet" and 4 to "create a meeting"

Wait, cause 4 is "you're finished": once you are in 3, and click next it takes you to redirect. So you must mean sth else? Meeting is created in 3, the label "youre finished" for 4 actually can be deleted i wasnt sure if we should even have it shown.

Yurii17K commented 3 years ago

change 3 to "choose a friend to meet" and 4 to "create a meeting"

Wait, cause 4 is "you're finished": once you are in 3, and click next it takes you to redirect. So you must mean sth else? Meeting is created in 3, the label "youre finished" for 4 actually can be deleted i wasnt sure if we should even have it shown.

wait, but when u r in 3 u have this image

JakubKoralewski commented 3 years ago

wait, but when u r in 3 u have this

change 3 to "choose a friend to meet"

yeah changing 3 is fine what u suggested, I only mean number 4, 4 is when you are on 3 and you click next 😄, then it takes you to a dummy "you are finished, pls wait we are implementing" etc. page

Yurii17K commented 3 years ago

wait, but when u r in 3 u have this

change 3 to "choose a friend to meet"

yeah changing 3 is fine what u suggested, I only mean number 4, 4 is when you are on 3 and you click next 😄, then it takes you to a dummy "you are finished, pls wait we are implementing" etc. page

lets assume pagse r like in main app there my suggestion makes sense

JakubKoralewski commented 3 years ago

lets assume pagse r like in main app there my suggestion makes sense

ok so I should remove the 4th one from being displayed?

Yurii17K commented 3 years ago

lets assume pagse r like in main app there my suggestion makes sense

ok so I should remove the 4th one from being displayed?

no sign them as i suggested in the 1 comment

JakubKoralewski commented 3 years ago

Ok I did what u asked (39b2121), but I don't think the label for the 4th one makes sense tho

JakubKoralewski commented 3 years ago

you need to accept so i can merge