Unable to dock properly

[architkshk]

when i open new tab in my application having chromium web browser it open but it don't dock properly.

[jornh]

Well that's not a lot of information to start helping you from. Please use the example applications we provide both in this repo and CefSharp.MinimalExample and see if you can reproduce ... Also please see https://github.com/cefsharp/CefSharp/wiki/Trouble-Shooting#still-having-problems

[architkshk]

i've attached the problem....

[sylvain-hamel]

We have the same intermittent problem here and we are currently investigating it. Re-sizing the container form fixes it. We don't have the steps to reproduce yet.

[architkshk]

ya, resizing fixes it.......

[jornh]

OK, @sylvain-hamel since you submitted #536 I guess you are on the JsBinding_WIP branch, right? Thanks for that one 👍 btw!

And I guess from the "yeah, resizing fixes it" that you are both on WinForms ...?

If so please note that the fixes in master between release 33.0.0 and 33.0.2 have not been applied to JsBinding_WIP nor the -pre NuGet yet.

[sylvain-hamel]

@jornh, yes on JsBinding_WIP (you are welcome btw), yes on Windows Forms. We'll try to reproduce it on master but I don't think we'll be able to do so because we rely heavily on js binding. I don't think our app will load correctly.

Are there recent fixes in master that you think can fix this? Can you tell me which ones, we could cherry pick those to test if they fix the issue.

@asarchit65 are you on JsBinding_WIP too?

[jornh]

It's #512, #513 and the few commits in #521 that are not just updates to the Example code as far as I can see. Then you of course might need to unload the Example project.

[jornh]

Nope, sorry apparently the good @amaitland already merged those! Forget my last comment.

[amaitland]

Is the issue present in the 'WinForms Example'? I was see that issue, was hopeful it's fixed.

[jornh]

@amaitland I think there is still hope that the fixes in the three PRs I mentioned above fixes it. Your fixes were merged into JsBinding_WIP only two days ago with 52db1e2 right?

This must mean that if @sylvain-hamel has seen issues with code from before that merge-point new testing has to be done before we can conclude anything new.

@asarchit65 a screenshot in itself doesn't really help us help you, we need some steps to reproduce. Only thing we can see is that you are most likely building a tabbed browser UI. For that I suggest you take a look at latest CefSharp.Example.WinForms and work from there.

[amaitland]

@amaitland I think there is still hope that the fixes in the three PRs I mentioned above fixes it. Your fixes were merged into JsBinding_WIP only two days ago with 52db1e2 right?

@jornh That is a very valid point, it was very recently that I merged those changes from master, guess we'll see what feedback comes from it.

[sylvain-hamel]

Hi guys, I updated my branch with the new code and I still have the bug, but things have improved.

As a workaround, I can now call Refresh() in webBrowser_IsBrowserInitializedChanged and it fixes the problem. This fix did not work without the new code. I'll keep this workaround until this gets fixed for real.

By the way, I'm building a tabbed interface too. The tab component by Janus. The bug might be cause by the fact that the control is created while it is not actually visible on the screen.

[sylvain-hamel]

@jornh , @amaitland did you see my last comment? Any thoughts on what could be the cause? Do you think you can work on this issue even without a proper repro project?

[amaitland]

@sylvain-hamel Without an easy way to reproduce the issue it's quite difficult to investigate this further. Thoughts?

[amaitland]

There's also the option of trying a newer version of Cef, the latest version that includes JS Binding is here https://github.com/cefsharp/CefSharp/tree/cef/2062-jsb-wip

There's also an even newer version of Cef, though JS binding is absent from that branch https://github.com/cefsharp/CefSharp/tree/cef/2171

Also it might be worth having a look at #522

[sylvain-hamel]

@amaitland Ok. I'll give it a try later this week. For now my workaround is viable.