local-motion / product

Apache License 2.0
2 stars 1 forks source link

Implement previous / next card functionality #344

Closed maxvisser closed 5 years ago

maxvisser commented 5 years ago

Description

In the redesign there is a element that is used to have the CTA and two arrows to go to next and previous card.

image

Definition of Done

dimitriirtimid commented 5 years ago

@maxvisser Can you explain the intended working of the "ik heb deze stap volbracht!" button? For me this button seems to contradict with concept that people go through the steps together as opposed to individually. I am curious on how you see this.

maxvisser commented 5 years ago

I am still not really sure on the implementation; But this is how I see it: Each step has a CTA: Flyers verspreiden -> CTA: 'BESTEL FLYERS!' Betrek beheerder -> CTA: 'STUUR UITNODIGING!' Wij worden rookvrij! -> CTA: 'JA, WIJ WORDEN ROOKVRIJ!'

Some steps though have multiple checkboxes that need to be checked; I find these steps from the user point of view abit difficult; a news article, or website adjustment can already be done by a team member and does not need to be worked on again. as in they need to check each checkbox to get a checkmark / succeed in the step. In these kind of steps, with multiple checkboxes, the CTA can be 'Ik heb deze stap volbracht'; as sometimes the user does not need to check each checkbox; (as a checkmark can already be set by someone else).

maxvisser commented 5 years ago

lets move discussion to different issue https://github.com/local-motion/product/issues/357

KarafiziArtur commented 5 years ago

Done previous and next button's functionality https://github.com/local-motion/onboarding-web/pull/395