Skip to content

Conversation

tim-rex
Copy link
Contributor

@tim-rex tim-rex commented May 27, 2018

Added support for

 SetItemDefaultFocus
 SetKeyboardFocusHere
 StyleColorsDark
 StyleColorsClassic
 StyleColorsLight
 AddRectFilled
 GetVersion
 GetCursorStartPos
 GetCursorScreenPos
 GetTextLineHeight
 GetTextLineHeightWithSpacing
 GetFrameHeight
 GetFrameHeightWithSpacing
 GetStyleColorName
 GetColorU32

Regarding GetColorU32.. These are overloaded functions in ImGui, and I was unable to find any way of supporting single/multiple dispatch with Cython.
That means we're left with 3 distinct cpdefs in order to support the 3 function signatures.

 get_color_u32
 get_color_u32_idx
 get_color_u32_rgba

tim-rex added 6 commits May 25, 2018 18:09
  AddRectFilled
  GetVersion
  GetCursorStartPos
  GetCursorScreenPos
  GetTextLineHeight
  GetTextLineHeightWithSpacing
  GetFrameHeight
  GetFrameHeightWithSpacing
  GetStyleColorName

Also added (but plan to refactor into a single-dispatch mechanism):
  GetColorU32
    StyleColorsDark
    StyleColorsClassic
    StyleColorsLight
    SetItemDefaultFocus
    SetKeyboardFocusHere
        SetItemDefaultFocus
        SetKeyboardFocusHere
        SetItemDefaultFocus
        SetKeyboardFocusHere
@coveralls
Copy link

coveralls commented May 27, 2018

Coverage Status

Coverage increased (+0.2%) to 46.929% when pulling 4a5d452 on tim-rex:additional_funcs_20180527 into 2d07e22 on swistakm:master.

Increases coverage to include add_draw_rect and get_color_u32
@swistakm
Copy link
Member

Yeah, I will try to think what to do with these functions. Maybe this week as I have a free weekend this time. Great work anyway.

By the way, I will try to do promised prerelease today. I have already configured sphinx build in RTD to show proper versions after this lands on master to avoid confusion.

@swistakm swistakm merged commit e2d5e2a into pyimgui:master May 30, 2018
@swistakm swistakm added the release pending Merged but still needs official release label May 30, 2018
@swistakm
Copy link
Member

I have released these changes as a prerrelese on PyPI. More details here. Unfortunately wheels for Windows are still missing because AppVeyor build got stuck. When they are ready I will upload them to PyPI.

@swistakm
Copy link
Member

Released as 1.0.0

@swistakm swistakm removed the release pending Merged but still needs official release label May 19, 2019
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.

3 participants