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

Switching to CEF #217

Open
Nesh108 opened this issue Jul 13, 2017 · 3 comments
Open

Switching to CEF #217

Nesh108 opened this issue Jul 13, 2017 · 3 comments

Comments

@Nesh108
Copy link

Nesh108 commented Jul 13, 2017

Hey @KojiNakamaru,

Seeing some problems, incomplete platform support (no Windows and no Linux) and somewhat poor performance, I was wondering if we should start looking into CEF as a definitive answer to most (if not all) of them?

I know you said you might not have time for that but I was wondering what you think would be the steps to make the full switch. Having some sort of checklist, would make it easier for you and other people to work on this task, which I think is very important to make this project be even more useful.

Let me know :)

@KojiNakamaru
Copy link
Member

As I said before,
#122 (comment)
if you provide any experimental implementation, it should be very useful, I guess not only for me but others. Could you please start to work on it if you really need such functions and have time? As I said, I don't have time for now to investigate further details.

@Nesh108
Copy link
Author

Nesh108 commented Jul 14, 2017

Yeah, I know but I also would like to know what kind of stuff did you have to do while making this project.

I would be interested in helping out but I never worked on low-level browser stuff, so some sort of guidance would certainly help loads, especially since we have to make it work with your code 😄

@KojiNakamaru
Copy link
Member

Please start to read all code in this plugin. Once you understand the structure of details, then you can start to learn how CEF works and start to investigate how CEF can be integrated into this plugin.

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

No branches or pull requests

2 participants