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

Update cef3lcl.pas #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update cef3lcl.pas #70

wants to merge 2 commits into from

Conversation

nandofw
Copy link

@nandofw nandofw commented Aug 23, 2018

small patch for gtk2

small patch for gtk2
@nandofw
Copy link
Author

nandofw commented Aug 26, 2018

In linux gtk2 the render frame open in a new windows.
Similar problem as this #38
In old version i not have this problem.

@dliw
Copy link
Owner

dliw commented Aug 27, 2018

This happens whenever a new CEF browser is started and the parent widget has not yet been initialized.

gtk_widget_realize is a possible workaround, but it shouldn't be necessary to change the parent.

I will try my best to find a reliable solution for the next release version.

@nandofw
Copy link
Author

nandofw commented Aug 27, 2018

I have tested here only call gtk_widget_realize whitout change the parent and work fine.

@dliw
Copy link
Owner

dliw commented Aug 28, 2018

Thanks for your effort. I will merge as soon as I have access to my computer.

@nandofw
Copy link
Author

nandofw commented Aug 31, 2018

Hi, i have a big update related to the last cef3 release 3.3440, so if you can please create a dev brach so i can upload these modification without modify the master brach.

@dliw
Copy link
Owner

dliw commented Aug 31, 2018

That sounds great! I just created a new "development" branch. Feel free to issue a pull request.

Please note, that I won't have access to my computer two more weeks and it probably will take me some time to merge your changes into my WIP branch.

@dliw
Copy link
Owner

dliw commented Dec 20, 2018

I just pushed an update to CEF 3.3538 into the development branch. Unfortunately the LCL integration is broken and there are other issues. The cause of the problems and possible solutions have yet to be found.

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.

None yet

2 participants