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

Support for 2.5 #10

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Support for 2.5 #10

wants to merge 7 commits into from

Conversation

camperking
Copy link

Hi there,

I just wanted to add some features and then it went somehow out of control and I rewrote everything. 😆

This PR adds:

  • automatic download of webui lib binaries
  • supports all webui 2.5 features
  • Window and Event structs for easy library usage
  • a bindgen script to regenerate bindings for future versions
  • and with it proper enums for all the webui types
  • updated examples

Unfortunately this will break the official webui docs but I don't see them as a good place for wrapper documentation anyways. They should introduce a new user with examples and use cases for each wrapper and then link to their generated docs.

Of course documentation and testing is needed but I have compiled and used this library on windows and linux.

What do you think?

@AlbertShown
Copy link
Contributor

This is a great PR, Thank you for your contribution 👍
All seems correct, but just a question, did you remove call_js_from_rust.rs and call_rust_from_js.rs ?

@neroist neroist linked an issue Oct 7, 2024 that may be closed by this pull request
@neroist
Copy link
Member

neroist commented Oct 7, 2024

Please resolve the clippy and rustfmt errors please! Otherwise it looks good. The CI / macos / build_release job is broken unfortunately, so we don't know if it passes on mac or not.

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

Successfully merging this pull request may close these issues.

Not fully support original webui?
3 participants