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

Request to merge a set of fix and improvements (done for our application) #24520

Open
JCYang opened this issue May 6, 2024 · 1 comment
Open

Comments

@JCYang
Copy link

JCYang commented May 6, 2024

Hi, there.
We've developed a commercial cross-platform app using wxWidgets framework for windows/mac/linux, all three mainstream OS are targeted.
And it's a non-trivial app that utilize many features of the framework, we may be one of the most heavy users.
So on the way, we found so many bugs.
We were on rapid development cycles and didn't have the resources to share our fixes to the upstream project on a regular basis, but now we find that we can no longer maintain the fixes, considering the framework is far from mature enough to allow us to stay on one LTS like version, we'd like to request you, the project maintainers to merge in most of our fixes/improvements.

Because there's too many, and in the foreseeable future, more fixes are expected and coming, we'd like to unify the set of works here for the maintainers to review.

Each of the fixes are now in their own branch, created for the sake of the pull request.

  1. Fix wxRibbonButtonBar's layout overall size computation.
    Coremail@a28b8db
  2. wxGTK: Fix UI refreshing delay when dragging the header border of generic wxDataViewCtrl to resize column.
    Coremail@381c608
  3. wxGTK: Fix improper wxEVT_LEAVE_WINDOW event
    Coremail@9286889
  4. MacOSX: Change the two dummy messages to use systemUpTime as timestamp to fix CEF renderer process crash.
    Coremail@af1f103
  5. MacOSX: Keep DoDragDrop() from re-entering to enable a proper working CEF.
    Coremail@e998db7
  6. wxOSX: Fix wxPopupTransientWindow's mouse event handling.
    Coremail@8d2ddec
  7. MacOSX: Make wxWebviewWebkit capture hotkeys less eagerly.
    Coremail@f4eb6c3
  8. wxOSX: Add support for wxDropSource icons.
    Coremail@dca4ca3
  9. wxOSX: Relax DoDragDrop() constraints.
    Coremail@71bf9d8

The 9 fixes/improvements mentioned above are meant to be merged into the master branch.
And the following is one optimization that seems work well in our app,

  1. wxMSW: Optimize AlphaBlt.
    Coremail@78afe18
@vadz
Copy link
Contributor

vadz commented May 6, 2024

Hello, thanks for the fixes but the usual process would be to open a single PR for each of them, so that they could be tested, discussed and applied independently and I think this would be very much preferable to this "unified" issue. Could you please open separate PRs for your changes?

For the bug fixes, it would be great if you could please explain how exactly the problem fixed by your changes can be reproduced (ideal is to add a test failing without the change and fixed by it, but I realize that this is not always easily possible).

Thanks in advance!

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

No branches or pull requests

2 participants