xdg-shell: make acked serial part of the pending state

[mahkoh]

No description provided.

[Copilot]

Pull request overview

This pull request refactors the xdg-shell implementation to make the acknowledged configure serial part of the pending state. The key change is tracking serials as u64 internally while still using u32 for the wire protocol, with mapping logic to handle the conversion.

Key Changes:

• Converted internal serial tracking from u32 to u64 with mapping functions to handle protocol u32 serials
• Made acked serial part of the surface's pending state for proper synchronization
• Replaced have_initial_commit boolean with an InitialCommitState enum for better state tracking
• Changed initial_configure from returning a Result to returning nothing (removing XdgSurfaceError from the signature)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
src/ifs/wl_surface/xdg_surface.rs	Core changes: u64 serial tracking, serial mapping logic, new InitialCommitState enum, ack_configure validation, and pending state integration
src/ifs/wl_surface/xdg_surface/xdg_toplevel.rs	Updated to remove XdgSurfaceError import and changed initial_configure signature to match new trait definition
src/ifs/wl_surface/xdg_surface/xdg_popup.rs	Updated to remove XdgSurfaceError import and changed initial_configure signature to match new trait definition
src/ifs/wl_surface.rs	Added serial field to PendingState struct for tracking acked serials through the commit process

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

@mahkoh I've opened a new pull request, #690, to work on those changes. Once the pull request is ready, I'll request review from you.