search

pwiecz / go-fltk

A simple wrapper around FLTK 1.4 library
MIT License
124 stars 24 forks source link

Newest version (20240101) breaks tabs #121

Closed mark-summerfield closed 10 months ago

mark-summerfield commented 10 months ago

tabs ok: github.com/pwiecz/go-fltk v0.0.0-20230924064308-597b69513d35 not ok: github.com/pwiecz/go-fltk v0.0.0-20240103212031-a509cedbac17

If you build minicalc it correctly shows tabs & they work fine. (Might be worth adding minicalc to ShowCase.)

But if you do go get -u ./... to update to the newest release of go-fltk the tabs don't appear at all & the UI is all messed up.

pwiecz commented 10 months ago

Still looking into it. Not sure what's going on. One thing I can see, that for me setting buttonHeight to value e.g. 40 makes the tabs visible. Still sizing of the controls is way off...

mark-summerfield commented 10 months ago

OK, I've solved the problem, which is that MeasureText() doesn't always return a sensible value (e.g., sometimes you get 0, 0). I'm going to remove all uses of MeasureText from my code. I've just done an update of minicalc without MeasureText and with the latest go-fltk and now it works fine.