State Restore collection inherits other collections sub-buttons
State Restore collection inherits other collections sub-buttons
Benjamin.Smith@OptimalAnswers.com
Posts: 15Questions: 2Answers: 0
in Bug reports
Link to test case: https://jsfiddle.net/benjamin_smith_optimal/ctvbsuz0/20/
Debugger code (debug.datatables.net): Couldn't work it with fiddle
Error messages shown: None
Description of problem: In the buttons.buttons of savedStates button, the collection "imports" the collection of other buttons instead of its own.
In the fiddle, see the Sub button appears in the savedStates collection (from the other button's collection) instead of the createState button.
Replies
Right enough! Thanks for the test case for this. I'll post back when I've had a chance to look at it (should be later this afternoon).
Allan
Fix committed here. The nightly will be updated with the change soon.
Thanks again for flagging this up!
Allan
Fixes the corruption issue! Thank you.
I'm losing the buttons.buttons prepend though (adding a createState). Is it part of the same area? I'll get a fiddle up once I the nightly is available. For now, I manually entered the commit change into local file.
To repro, the Create button is in the collection if no states are stored. Once a state is added I lose the button. Also (possibly unrelated), I lose the text override as well (reverts to Saved States). A page refresh is sometimes required after state adding/removing to get the behavior.
Urgh - yes, sorry. The nightly is up to date, and right enough there is still an error when saving the state. I think it is exactly the same thing, just in a slightly different area of the code. I'll check that when I'm in the office tomorrow morning.
Allan
Fix for the same sort of error committed here.
Regarding the text issue, this is happening because of the ability to update the text to show the number of saved states.
What to do is set the button text through the
buttons.savedStates
language option - e.g.: https://jsfiddle.net/q0n2powy/ .It does seem a bit odd that - I'll have a think about how we can improve that. It might need a new API in Buttons...
Allan
Awesome! Thank you so much!