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

tree/view: Send configure before mapping #8314

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Aug 27, 2024

  1. tree/view: Send configure before mapping

    Sending 0,0 as configure dimensions indicate that the client is free to
    pick its own dimensions. When tiling, the client needs to strictly
    adhere to the tile dimensions. Sway's handling of this has been to send
    a the appropriate dimensions in a new configure when the surface is
    mapped, leading to the first buffer most likely being incorrectly sized.
    
    Move the majority of the mapping logic to view_premap, issued on the
    initial role commit rather than when mapping the view. This allow the
    first configure to be driven by a tree transaction with the appropriate
    geometr, and allows container siblings to start preparing earlier as
    well, reducing the new window latency.
    
    Fixes: swaywm#2176
    kennylevinsen committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    fbf5887 View commit details
    Browse the repository at this point in the history