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.

Sign up for GitHub

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

Jump to bottom

Document adding a double click listener to the table plugin example #12

Closed
dsmmcken opened this issue Jan 16, 2023 · 1 comment
Closed

Document adding a double click listener to the table plugin example #12

dsmmcken opened this issue Jan 16, 2023 · 1 comment
Assignees
@mofojed
Labels
enhancement New feature or request

Comments

@dsmmcken
Copy link

Kevin S
This probably sounds odd. I want to get the row data associated with a click (client side webui), then use that data to interact with the client OS. For example, a row is clicked, and some OS api is invoked using the data from a specific row of that column. I realize this is way outside the scope of a web app, but curious if anyone has thought or attempted to do this with a Chrome plugin, selenium, or even wrapping deephaven in an electron app? In my case I "simply" want to control a separate charting/plotting app with clicks from deephaven, but I can imagine a number of scenarios where this could be useful.
Alternatively, if a row click registers on the server, perhaps there's some way of having a non-browser application connect to a websocket and receive those events, which could then interact with the OS. Just wanted to check if anyone has any thoughts on this.

Document adding a double click handler that has the row data to the plugin, and if not presently available add API support first.
@dsmmcken dsmmcken added the enhancement New feature or request label Jan 16, 2023
@mofojed mofojed self-assigned this Jun 19, 2023
@mofojed mofojed mentioned this issue Jul 10, 2023
Merged
mofojed added a commit to deephaven/web-client-ui that referenced this issue Jul 11, 2023
@mofojed
refactor: Update types for TablePlugins (#1176)
792faa1 
- We were just defining a bunch of unknowns, which is bad practice
- Instead define actual types, so makers of TablePlugins can use those
types effectively
- Add some details to the AuthPlugins readme
- Did this cleanup while working on
deephaven/deephaven-js-plugins#12
- Tested using the `table-example` plugin:
https://github.com/deephaven/deephaven-js-plugins/tree/main/plugins/table-example
  - Used the following snippet:
```
from deephaven import empty_table
from deephaven.jcompat import j_hashmap
t = empty_table(5).update(["x=i"])
t2 = t.j_table.withAttributes(j_hashmap({ "PluginName": "table-example" }))
```
- Also tested error message is shown if unable to load the table plugin:

![image](https://github.com/deephaven/web-client-ui/assets/4505624/b8fe8b96-4bf7-441b-bff2-5ab4f06ff8ac)
@mofojed mofojed mentioned this issue Jul 13, 2023
Merged
mofojed added a commit to deephaven/web-client-ui that referenced this issue Aug 14, 2023
@mofojed
refactor: Plugin cleanup (#1409)
3445586 
- Clean up a bunch of the typings around plugins
- Add hooks for panel registration, like `useComponent`
  - Removes a lot of the boilerplate code currently used in plugins
- One hydrate method now is used for new panels and rehydrating
dehydrated panels alike
- Makes the examples for
deephaven/deephaven-js-plugins#12 much cleaner
@mofojed
Copy link
Member

mofojed commented Sep 5, 2023

Updated this in the template to include a data click listener: deephaven/deephaven-js-plugin-template#1

@mofojed mofojed closed this as completed Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mofojed mofojed

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dsmmcken @mofojed