New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functions to get all & reset optionBoxes/CBs/RBs/etc #202

Closed
jarvisteach opened this Issue Aug 3, 2017 · 3 comments

Comments

Projects
None yet
1 participant
@jarvisteach
Owner

jarvisteach commented Aug 3, 2017

It would be useful to have functions to get & reset all the optionBoxes, RadioButtons, CheckBoxes, etc - just like there are for Entries

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 11, 2017

Owner

.getAllEntries() loops through all entries, calls .getEntry() and adds them to a dictionary.

We need to provide the same for:

  • .getAllTextAreas()
  • .getAllOptionBoxes()
  • .getAllSpinBoxes()
  • .getAllListBoxes()
  • .getAllScales()
  • .getAllCheckBoxes()
  • .getAllRadioButtons()
  • .getAllDatePickers()
  • .getAllProperties()

Some of these are multi-select, so they will have to provide a list of values for each key.

Owner

jarvisteach commented Aug 11, 2017

.getAllEntries() loops through all entries, calls .getEntry() and adds them to a dictionary.

We need to provide the same for:

  • .getAllTextAreas()
  • .getAllOptionBoxes()
  • .getAllSpinBoxes()
  • .getAllListBoxes()
  • .getAllScales()
  • .getAllCheckBoxes()
  • .getAllRadioButtons()
  • .getAllDatePickers()
  • .getAllProperties()

Some of these are multi-select, so they will have to provide a list of values for each key.

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 11, 2017

Owner

.clearAllEntries() loops through all entries, and empties them. It doesn't call the .clearEntry() function, although it uses the same code except .setFocus() - maybe this should be reworked, and make setFocus a parameter on .clearEntry()

The exact same list of functions as above needs to be provided for each of the widgets.

However, not all of those widgets have a clear option (in fact, the language doesn't always make sense, maybe rename to reset?

So first, clear/reset functions will need to be created for some - ensuring starting values are recorded. Then the ALL functions can be coded.

Owner

jarvisteach commented Aug 11, 2017

.clearAllEntries() loops through all entries, and empties them. It doesn't call the .clearEntry() function, although it uses the same code except .setFocus() - maybe this should be reworked, and make setFocus a parameter on .clearEntry()

The exact same list of functions as above needs to be provided for each of the widgets.

However, not all of those widgets have a clear option (in fact, the language doesn't always make sense, maybe rename to reset?

So first, clear/reset functions will need to be created for some - ensuring starting values are recorded. Then the ALL functions can be coded.

jarvisteach added a commit that referenced this issue Aug 12, 2017

Initial getAllXXX #202
Implemented a getAll for each widget.

NB, had to create an array of DatePicker names (n_dps) so we could find
them all. This should be used for verification down the line…

jarvisteach added a commit that referenced this issue Aug 12, 2017

Updated docs for #202
Docs now mention getAllXXX functions.

Also added getAllScales

n_dps now used to validate DatePickers

Changed functions on OptionBox - replaced `.getListItems()` with `.getListBox()`

jarvisteach added a commit that referenced this issue Aug 12, 2017

Added testing for getAllXXX #202
Also added deprecated messages to `.getListItemsPos()` & `.getListItems()`

jarvisteach added a commit that referenced this issue Aug 12, 2017

jarvisteach added a commit that referenced this issue Aug 12, 2017

Fix ListBoxPos bug in 3.3 #202
A bug in tkinter 1.160 returns curselection as strings, need to convert to ints

jarvisteach added a commit that referenced this issue Aug 12, 2017

clearAllXXX #202
Implemented:
* `.clearAllCheckBoxes()`
* `.clearAllScales()`
* `.clearAllSpinBoxes()`
* `.clearAllRadioButtons()`
* `.clearAllListBoxes()`
* `.clearAllTextAreas()`

Also, `.clearAllEntries()` changed to wrap `.clearEntry()`.

jarvisteach added a commit that referenced this issue Aug 12, 2017

Testing for clearAllXXX #202
Added testing

jarvisteach added a commit that referenced this issue Aug 12, 2017

jarvisteach added a commit that referenced this issue Aug 12, 2017

Fixed clearAllXXX tests #202
Mistakenly used the returned getAll array, instead of app calls.

Also adjusted clearScale - to get the starting values rather than use 0.

jarvisteach added a commit that referenced this issue Aug 12, 2017

jarvisteach added a commit that referenced this issue Aug 14, 2017

jarvisteach added a commit that referenced this issue Aug 14, 2017

Clear/Reset functions for properties (#202)
Added both clear & reset function to Properties

jarvisteach added a commit that referenced this issue Aug 14, 2017

jarvisteach added a commit that referenced this issue Aug 15, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 15, 2017

Owner

All now implemented - went with clear for now - all widgets have a clear/clearAll

This reverts all of them back to their original state, except for Properties. Properties get set to false, but also has a reset/resetAll to revert back to its original values.

Owner

jarvisteach commented Aug 15, 2017

All now implemented - went with clear for now - all widgets have a clear/clearAll

This reverts all of them back to their original state, except for Properties. Properties get set to false, but also has a reset/resetAll to revert back to its original values.

jarvisteach added a commit that referenced this issue Aug 15, 2017

Added missing clearOptionBox (#202)
Forgot to add clearOptionBox - now added.

Also, updated clear/clearAll ListBox to check callFunction

And, added a friendly error message for passing a list of names to
addButton.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment