Skip to content
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

updateSearchInput ? #52

Closed
steve-liang opened this issue Feb 6, 2018 · 3 comments
Closed

updateSearchInput ? #52

steve-liang opened this issue Feb 6, 2018 · 3 comments

Comments

@steve-liang
Copy link

@steve-liang steve-liang commented Feb 6, 2018

Hi,

I am wondering if you can add updateSearchInput just like other input UI. I understand there's a button involved so it may be a bit tricky. I personally attempted to do that using session$sendInputMessage but it won't simulate button click to trigger input value change. Any idea?

Steve

@pvictor
Copy link
Member

@pvictor pvictor commented Feb 6, 2018

Hi,
You're right, this function is missing, I added it in the last commit. Please re-install package from Github and see ?updateSearchInput example.
There's an argument trigger to decide if only ui value must be updated and let user click button or both ui and server value must be updated at the same time.
If it works for you, I let you close the issue.

Victor

@steve-liang
Copy link
Author

@steve-liang steve-liang commented Feb 6, 2018

Thanks for your quick response with perfect solution. Looking forward to it being on CRAN later.
I need to study your code to see how you did it with the trigger.

@steve-liang steve-liang closed this Feb 6, 2018
@pvictor
Copy link
Member

@pvictor pvictor commented Feb 6, 2018

You're welcome :)
Maybe this week-end on CRAN but I can't promise anything.
The trigger is a JavaScript click on the button, I haven't found a better way... Code is here in receiveMessage method if you're interested.

Victor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.