Skip to content

ImGuiIntegration: implement updating application cursor - #56

Merged
mosra merged 1 commit into
masterfrom
imgui-cursor
Nov 6, 2019
Merged

mosra merged 1 commit into
masterfrom
imgui-cursor

Conversation

@mosra

@mosra mosra commented Oct 19, 2019

Copy link
Copy Markdown
Owner

Depends on mosra/magnum#383. Putting into a PR because I'm not sure if I can wrap up the other one in time for a release.

Cc: @Melix19

@mosra mosra added this to the 2019.0c milestone Oct 19, 2019
Comment thread src/Magnum/ImGuiIntegration/Context.hpp
GlfwApplication supports only a subset of SDL2 cursors, so there has to
be some ugly SFINAE to make the things work.
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #56 into master will decrease coverage by 14.69%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   74.77%   60.08%   -14.7%     
==========================================
  Files          21       14       -7     
  Lines         880      486     -394     
==========================================
- Hits          658      292     -366     
+ Misses        222      194      -28
Impacted Files Coverage Δ
src/Magnum/ImGuiIntegration/Context.h 100% <ø> (ø) ⬆️
src/Magnum/ImGuiIntegration/Context.cpp 3.12% <0%> (-79.92%) ⬇️
src/Magnum/BulletIntegration/DebugDraw.cpp 12.82% <0%> (-3.46%) ⬇️
src/Magnum/BulletIntegration/MotionState.cpp 84.21% <0%> (-1.51%) ⬇️
src/Magnum/ImGuiIntegration/Integration.h 100% <0%> (ø) ⬆️
src/Magnum/ImGuiIntegration/Widgets.h
src/Magnum/DartIntegration/ConvertShapeNode.cpp
src/Magnum/DartIntegration/Object.h
src/Magnum/DartIntegration/World.cpp
src/Magnum/DartIntegration/Object.cpp
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88bd1a2...dd5f030. Read the comment docs.

@mosra
mosra merged commit dd5f030 into master Nov 6, 2019
@mosra
mosra deleted the imgui-cursor branch November 6, 2019 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants