react-albus icon indicating copy to clipboard operation
react-albus copied to clipboard

onNext is called on initialization

Open RareSecond opened this issue 7 years ago • 9 comments

Wizard's onNext prop is called on the component's mount and not only when firing a next event.

RareSecond avatar Oct 17 '18 11:10 RareSecond

It shouldn't!

zaguiini avatar Dec 14 '18 17:12 zaguiini

This issue is stale because it has been open 30 days with no activity. Remove no-issue-activity label or comment or this will be closed in 5 days.

github-actions[bot] avatar Apr 08 '20 00:04 github-actions[bot]

Looks like a dead repo to me. Even the bot is months too late.

RareSecond avatar Apr 08 '20 18:04 RareSecond

This issue is stale because it has been open 30 days with no activity.

github-actions[bot] avatar May 09 '20 00:05 github-actions[bot]

Wizard's onNext prop is called on the component's mount and not only when firing a next event.

in my case, <WithWizard/> is a UI Component . u can get a ref from Button and trigger it Manually, even u can set css display: none to hidden UI ,

<WithWizard render={({ next, previous, step, steps }) => ( <div className="buttons_back"> {steps.indexOf(step) > 0 && ( <Button ref={buttonRef} className="btn-fluid btn-secondary" onClick={previous}> back </Button> )} )} />

set function :() => buttonRef.current.props.onClick() to call

wowtrxqn avatar Aug 12 '20 07:08 wowtrxqn

This issue is stale because it has been open 30 days with no activity.

github-actions[bot] avatar Sep 12 '20 00:09 github-actions[bot]

onNext probably shouldn't be called on initial render. That'd solve for https://github.com/americanexpress/react-albus/issues/33.

ryhinchey avatar Mar 04 '21 04:03 ryhinchey

this will be solved in the v3 release. work is being done on the 3.x branch

ryhinchey avatar Mar 09 '21 01:03 ryhinchey

This issue is stale because it has been open 30 days with no activity.

github-actions[bot] avatar Aug 05 '23 00:08 github-actions[bot]