Vladimír Vondruš

Actually, I just went and temporarily disabled PR builds on CircleCI, so that should preserve the scarce minutes, hopefully. AppVeyor could be enough to verify correctness, ImGui is tested there...

Hi! I'm not a "Windows native" user so I don't know how exactly the package works -- however, if I remember correctly, ImGui had various issues when built dynamically. Can...

I dug into the vcpkg package history and it seems that dynamic builds might have worked in the past, but they certainly don't anymore (unless imgui itself is doing something...

Regarding the SDL includes -- vcpkg has a patch to fix those (https://github.com/microsoft/vcpkg/blob/master/ports/magnum/001-sdl-includes.patch) but looks like it doesn't apply on current `master` anymore, which is why you get the errors....

There's no disadvantage to linking statically (apart from the usual tradeoffs with static/dynamic builds) :) You're not forced to use [Utility::Resource](https://doc.magnum.graphics/corrade/classCorrade_1_1Utility_1_1Resource.html) for anything, but if you would want to use...

The SDL-related issue should be fixed once https://github.com/microsoft/vcpkg/pull/10158 gets merged. ImGui will likely need more work, that's not done yet.

Yep, `TYPE` target property is a good idea -- does the ImGui CMake config coming from Vcpkg expose this information? For all I know, imported targets usually lack info whether...

Hi! coincidentally, we were discussing this exact issue yesterday on the [Gitter channel](https://gitter.im/mosra/magnum/archives/2019/02/21). The problem is that, for text input, SDL2 and GLFW fire both `keyPressEvent()` and `textInputEvent()` for each...

If I'm not mistaken, the above is doing the same as this: ```cpp if (m_imgui.handleKeyPressEvent(event) || isTextInputActive()) return; ``` This in practice means, if any text input widget is active,...

Yup, why not? :)