Skip to content

Conversation

@marcomq
Copy link

@marcomq marcomq commented Aug 25, 2021

The original webview project was updated, functions were removed and a lot changed. The new Webview isn't using C99 but C++ as core languange.
Wails (https://github.com/wailsapp/wails) is still using an "old" C99 version of webview and they also seem to maintain the library in case of bugs or functionality that is otherwise missing. The last change is 2 months ago. Therefore, I updated to wails webview, which still seems to be mostly compatible with the old version.

@apahl
Copy link

apahl commented Sep 28, 2021

Wow, thanks a lot for taking the effort to update this lib. I just tried your fork and everything is working again, great job!

If there is no response from the original author of this repo, we should maybe try to have the entry in the nimble package list replaced with your fork.
This is at all not to disregard the efforts of the original author and we all know there are many reasons why an open-source project might get abandoned, but it would be nice if the nimble package list contained a working version of this lib.

Kind regards,
Axel

@marcomq
Copy link
Author

marcomq commented Sep 28, 2021

Hey Alex!
I'm quite new to all this open source stuff. I just wanted nim webview to run on the latest ubuntu version. Glad that it is working for you 🙂

Yeah - I'm not sure what would be best...
I was hoping that this somehow gets merged after some time. But maybe we actually need to create a PR for the nimble list too, if this is the way to do it.

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.

2 participants