Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues with wxWidgets v3.11 #635

Closed
5 tasks done
vsonnier opened this issue Mar 20, 2018 · 12 comments
Closed
5 tasks done

Issues with wxWidgets v3.11 #635

vsonnier opened this issue Mar 20, 2018 · 12 comments
Assignees
Labels

Comments

@vsonnier
Copy link
Collaborator

vsonnier commented Mar 20, 2018

This issue gathers the various problems encountered during the migration to wxWidgets v3.11: (checked when solved)

  • @Magalex2x14 : OS X 10.13.3, wxWidgets 3.1.1 - missing the labels on the buttons below the bookmarks. Same commit but with wxWidgets 3.1.0 - all OK.
  • @parkerlreed : Syntax incompatible with wxWidgets v3.04
  • @Magalex2x14 : OS X 10.13.3, wxWidgets 3.1.1 : improper BookmarkView refresh, forced by re-focus.
  • Windows 10 : very slow, blocking refresh of BookmarkView.
  • Windows 10: Buttons labels are invisible.
@vsonnier
Copy link
Collaborator Author

vsonnier commented Mar 20, 2018

Solutions / Suggestions:

  • @parkerlreed : The minimum requirement is wxWidgets v3.1, so you have to compile it from scratch, see the Wiki. The previous wxWidgets v3.0x syntax compatibility was pure luck, and had to end one day.
  • @Magalex2x14 : Same story here on Windows 10. The custom text button color given from the current theme doesn't work well. I see if I can fix it, or else leaving the default color unchanged would do.

@Magalex2x14
Copy link
Contributor

Magalex2x14 commented Mar 20, 2018

@vsonnier I also noticed that with 3.1.1 when new modem is added, corresponding buttons do not appear (bookmark, start recording, remove active). They appears only after "refocusing" - I have to click into the bookmarks or ranges, then back to the modems.

P.S. Hmmm. On same commit, but with 3.1.0 this issue is also present...

@parkerlreed
Copy link

parkerlreed commented Mar 20, 2018

@vsonnier Yeah I got the 3.1.1 installed fine but it had to be compiled which actually took quite a while. Not an issue for me personally but I can see that maybe being an issue when there doesn't appear to be a single distro shipping 3.1+. Appimage is one solution but even that has had issues as of late.

@parkerlreed
Copy link

parkerlreed commented Mar 20, 2018

(And by a long compile time it took nearly 8 minutes on a 4 core/8 thread machine running make at -j10. A bit lengthy and would only be longer for somebody running lower hardware)

vsonnier added a commit that referenced this issue Mar 20, 2018
@vsonnier
Copy link
Collaborator Author

vsonnier commented Mar 20, 2018

@parkerlreed It is quite long indeed, but you have only to do it every 2 years given the super-slow wxWidgets release schedule.

I've pushed 1 commit on the master branch, which reverts most of the BookmarkView modifications... The control refresh is really slow here on Windows 10 with v3.1.1 vs 3.1.0, but is hopefully more correct.

Button foreground (i.e text color) has been forced to white (255,255,255) to stand out.

@vsonnier vsonnier reopened this Mar 21, 2018
tgunr added a commit to tgunr/CubicSDR that referenced this issue Mar 21, 2018
* 'master' of https://github.com/cjcliffe/CubicSDR:
  Fix cjcliffe#634 plus cjcliffe#534: bound checking in SDRDevices dialog selection
  Fixes for cjcliffe#635, round 1: BookmarkView
@gagath
Copy link

gagath commented Mar 21, 2018

wxgtk v3.1.x is shipped as binary release for Debian, Ubuntu and Fedora : http://codelite.org/LiteEditor/WxWidgets31Binaries

@parkerlreed
Copy link

Ahh thanks. Wonder why all the official repos seem to be behind (I guess due to compat with older software)

@vsonnier vsonnier added the bug label Mar 23, 2018
@vsonnier vsonnier self-assigned this Mar 23, 2018
@vsonnier vsonnier reopened this Mar 24, 2018
@vsonnier
Copy link
Collaborator Author

vsonnier commented Mar 24, 2018

Hello @Magalex2x14, @parkerlreed and all. I've commited some changes on the master, which I believe fixes the following points:

  • Invisible button labels (Windows, OS-X). I forced them to white instead of using the actual theme color.
  • Slow BookmarkView refresh (Windows). Turns out the UI events were calling themselves back and forth.
  • (Maybe ?) Improper control refresh (OS X). I've restored most of the old BookmarkView control logic, plus added other fixes.

Please tell me how it goes for you.

@vsonnier vsonnier reopened this Mar 24, 2018
@vsonnier vsonnier reopened this Mar 24, 2018
@Magalex2x14
Copy link
Contributor

OS X. Invisible button labels are still invisible (maybe labels must be forced to black? White labels becomes visible when the button is pressed...) In the rest everything seems OK.
cubicsdr v0 2 3 www cubicsdr com 2018-03-24 19-01-04

@vsonnier vsonnier reopened this Mar 24, 2018
@vsonnier
Copy link
Collaborator Author

vsonnier commented Mar 24, 2018

Thanks @Magalex2x14. Funny thing, colors are exactly inverted on Windows 10:
cropimage

So finally in the last commit I removed the buttons custom colors. Less sexy, but should be OK on all platforms.
What about your UI refreshing problems ?

@Magalex2x14
Copy link
Contributor

Everything looks good. I do not see any problems with the UI.

@vsonnier
Copy link
Collaborator Author

Great ! Thanks for your input, I think we can close this issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants