Skip to content

Conversation

@christophe-g
Copy link

  • Continue fires and event (paper-stepper-continue). This allow to implement one's own logic inside paper-step and have other elements fireing the same event (and triggering the stepper to continue)
  • Continue button can be hidden with hideContinueButton set to true

@Zecat
Copy link
Owner

Zecat commented Dec 21, 2016

Thanks, I like the idea of relying on custom controls but in this case, instead of just hiding the continue button, the full #buttonsWrapper of paper-stepper should be display: none; so the "buttons block" is no longer part of the layout, maybe with a no-button attribute.

In consequence, the slideshow viewport of paper-step should be resized when no-button changes and in paper-step, _updateSlideshowViewportTop should be updated to manage this.$$('#slideshowViewport').style.bottom which is for now hard coded to 84px (the clientHeight of #buttonsWrapper).

This is just for the horizontal layout, the vertical one should also be managed.

@christophe-g
Copy link
Author

Yes, that makes more sense and is a bit more elegant.
Then in this case, we would also need to implement a paper-stepper-back and paper-stepper-progress event so as to be able to cover those cases with custom control as well.
What do you think ?

christophe-g and others added 2 commits January 14, 2017 11:17
…y (vertical and horizontal). Added paper-stepper-continue, paper-stepper-back and paper-stepper-progress event
@christophe-g
Copy link
Author

Implemented via no-button attribute.
We now have paper-stepper-back and paper-stepper-progress events (in addition to paper-stepper-continue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants