Skip to content
Snippets Groups Projects

qt: detect Windows 11 22H2 properly in win32windoweffects module

Merged Fatih Uzunoğlu requested to merge fuzun/vlc:qt/nativebgblurwin11 into master
All threads resolved!

Windows 11 identifies as Windows 10 (NT 10), so this comparison always yields false.

QOperatingSystemVersion(QOperatingSystemVersion::Windows, 10, 0, 22621) would also correct this issue, but I opted in to directly use QOperatingSystemVersion::Windows11_22H2 now instead.

As a side note, native acrylic background for Win32 windows only gained support with the Windows 11 22H2 update, that's why there is the check here.

Note that according to the general conventions a transparent window must be frameless on Windows. This is currently not the case, because "frameless window don't support areo snapping". So, it can be considered as a known bug if native acrylic does not work as intended. This grants a requirement of change to use frameless window, and get rid of the current workaround that simulates frameless window in mainctx_win32.cpp. If aero snapping does not work, it is Windows's problem since we are using frameless windows in every other platform.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 3add97b0 - qt: detect Windows 11 22H2 properly in win32windoweffects module

    Compare with previous version

  • Fatih Uzunoğlu resolved all threads

    resolved all threads

  • Steve Lhomme changed milestone to %4.0

    changed milestone to %4.0

  • Pierre Lamot approved this merge request

    approved this merge request

  • MR Acceptance result

    :tada: This MergeRequest has been Accepted! Congratulations.

    MR acceptance checks details:

    • :white_check_mark: MR should be considered mergeable by Gitlab
    • :white_check_mark: Last pipeline should be successful
    • :white_check_mark: MergeRequest should have at least one external review and/or vote
    • :white_check_mark: All threads should be resolved, have votes and score > 0
    • :white_check_mark: MergeRequest should have no activity (threads/votes) for (72h/72h)

    This message was automatically generated by homer-bot.

  • Steve Lhomme added 200 commits

    added 200 commits

    • 3add97b0...3a4a3dea - 199 commits from branch videolan:master
    • aaba70b6 - qt: detect Windows 11 22H2 properly in win32windoweffects module

    Compare with previous version

  • Steve Lhomme enabled an automatic merge when the pipeline for aaba70b6 succeeds

    enabled an automatic merge when the pipeline for aaba70b6 succeeds

  • merged

  • Please register or sign in to reply
    Loading