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

setXXXChangeFunction and setXXXSubmitFunction not working for FileEntry widgets #226

Closed
BlacKCaT27 opened this Issue Aug 24, 2017 · 9 comments

Comments

Projects
None yet
2 participants
@BlacKCaT27

BlacKCaT27 commented Aug 24, 2017

According to the documentation, if I wan't to set an event handler for a widget, I follow the syntax from this issues title. But when I try:

app.setFileEntrySubmitFunction("myFileEntryWidgetTitle", myfunc)

or

app.setFileEntryChangeFunction("myFileEntryWidgetTitle", myfunc)

I get a warning at start-up saying the functions don't exist.

Are FileEntry/DirEntry types not supported?

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 24, 2017

Owner

Ah yes, this is a bug.

These are new widgets introduced in 0.07 - but they haven't been added to the WIDGETS dictionary, so none of the auto-generated functions have been set for those widgets.

You can add a trace manually for the time being:

Get the entry var: app.n_entryVars["myFileEntryWidgetTitle"]
Then call trace() passing in the details.

Owner

jarvisteach commented Aug 24, 2017

Ah yes, this is a bug.

These are new widgets introduced in 0.07 - but they haven't been added to the WIDGETS dictionary, so none of the auto-generated functions have been set for those widgets.

You can add a trace manually for the time being:

Get the entry var: app.n_entryVars["myFileEntryWidgetTitle"]
Then call trace() passing in the details.

@jarvisteach jarvisteach added the bug label Aug 24, 2017

@jarvisteach jarvisteach added this to the 0.08 milestone Aug 24, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 24, 2017

Owner

Actually @BlacKCaT27 - these are still just Entry widgets, so this should work:

app.setEntrySubmitFunction("myFileEntryWidgetTitle", myfunc)
Owner

jarvisteach commented Aug 24, 2017

Actually @BlacKCaT27 - these are still just Entry widgets, so this should work:

app.setEntrySubmitFunction("myFileEntryWidgetTitle", myfunc)
@BlacKCaT27

This comment has been minimized.

Show comment
Hide comment
@BlacKCaT27

BlacKCaT27 Aug 24, 2017

BlacKCaT27 commented Aug 24, 2017

@BlacKCaT27

This comment has been minimized.

Show comment
Hide comment
@BlacKCaT27

BlacKCaT27 Aug 24, 2017

BlacKCaT27 commented Aug 24, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 24, 2017

Owner

I"ve tested the setEntrySubmitFunction() - that seems to work here, if the Entry is in focus and I press Enter it calls my function.

For the double function call - the first is happening when the is removed, that shouldn't happen, so I'll dig a bit more...

Owner

jarvisteach commented Aug 24, 2017

I"ve tested the setEntrySubmitFunction() - that seems to work here, if the Entry is in focus and I press Enter it calls my function.

For the double function call - the first is happening when the is removed, that shouldn't happen, so I'll dig a bit more...

@BlacKCaT27

This comment has been minimized.

Show comment
Hide comment
@BlacKCaT27

BlacKCaT27 Aug 24, 2017

BlacKCaT27 commented Aug 24, 2017

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

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

Added File/Directory Entry to WIDGETS list #226
Added a CallPauseFunction when removing default text from entries -
required changing how Entry stored cmd/cmd_id

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

Reverted cmd/cmd_id references
Changed back to previous style to keep consistent - #226 #206 #220
@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 25, 2017

Owner

Resolved in build 0.08

Owner

jarvisteach commented Aug 25, 2017

Resolved in build 0.08

@BlacKCaT27

This comment has been minimized.

Show comment
Hide comment
@BlacKCaT27

BlacKCaT27 Aug 25, 2017

Great, thanks! any eta on when 0.08 will be published out to pip/pypi/etc?

BlacKCaT27 commented Aug 25, 2017

Great, thanks! any eta on when 0.08 will be published out to pip/pypi/etc?

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Aug 26, 2017

Owner

Hopefully in the next couple of days

Owner

jarvisteach commented Aug 26, 2017

Hopefully in the next couple of days

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