Skip to content

saved is not readonly anymore #59

Open
christophe-g wants to merge 11 commits into
Zecat:masterfrom
PolymerEl:allow-setting-saved
Open

saved is not readonly anymore #59
christophe-g wants to merge 11 commits into
Zecat:masterfrom
PolymerEl:allow-setting-saved

Conversation

@christophe-g

@christophe-g christophe-g commented Dec 19, 2016

Copy link
Copy Markdown
  • Saved is not readonly anymore to allow to set it from outside paper-stepper (e.g. bind it to database value)
  • saved and edit icons are now properties
  • a step can be unsaved. In this case, it fires a paper-step-unsave and the stepper will select the last unsaved step
  • paper-step-save and paper-step-unsave events are fired from within a new _savedObserver observer method

…tepper (e.g. bind it to database value) - saved and edit icons are now properties
@Zecat Zecat requested review from Zecat and removed request for Zecat December 20, 2016 00:17
@Zecat

Zecat commented Dec 20, 2016

Copy link
Copy Markdown
Owner

Nice, could you remove the two // comments ?

1 similar comment
@Zecat

Zecat commented Dec 20, 2016

Copy link
Copy Markdown
Owner

Nice, could you remove the two // comments ?

@christophe-g

Copy link
Copy Markdown
Author

Agree, not the most elegant - here you go!

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