Skip to content

Bring back the workaround for #1126#1460

Closed
Difegue wants to merge 1 commit intolepoco:mainfrom
Difegue:touch-maximize-workaround-2
Closed

Bring back the workaround for #1126#1460
Difegue wants to merge 1 commit intolepoco:mainfrom
Difegue:touch-maximize-workaround-2

Conversation

@Difegue
Copy link
Copy Markdown
Contributor

@Difegue Difegue commented Jun 24, 2025

This brings back the workaround for touch input triggering the maximize button twice that got removed by #1366 .

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

Using a touchscreen to interact with the maximize button in an app using WPFUI will trigger it twice, maximizing and restoring the window in quick succession.

Issue Number: #1126

What is the new behavior?

Touch only maximizes or restores the app once.

Other information

@Difegue Difegue requested a review from pomianowski as a code owner June 24, 2025 16:39
@github-actions github-actions Bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request release titlebar Titlebar updates labels Jun 24, 2025
@Difegue
Copy link
Copy Markdown
Contributor Author

Difegue commented Jun 24, 2025

Nevermind, seems like this doesn't actually work anymore due to changes to the hwndhook. Will have to dig deeper...

@Difegue Difegue closed this Jun 24, 2025
@ctmusall
Copy link
Copy Markdown

ctmusall commented Jul 3, 2025

Nevermind, seems like this doesn't actually work anymore due to changes to the hwndhook. Will have to dig deeper...

Any updates on fixes for this issue?

@Difegue
Copy link
Copy Markdown
Contributor Author

Difegue commented Jul 3, 2025

nope! help welcome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

controls Changes to the appearance or logic of custom controls. PR Pull request release styles Topic is related to styles titlebar Titlebar updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants