Skip to content

Conversation

@jturcotte
Copy link
Contributor

  • If the change modifies a visible behavior, it changes the documentation accordingly
  • If possible, the change is auto-tested
  • If the changes fixes or close an existing issue, the commit message reference the issue with Fixes #xxx or Closes #xxx
  • If the change is noteworthy, the commit message should contain ChangeLog: ...

Copy link
Member

@tronical tronical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - looks good to me! Just one question within about the resizing handling.

}

fn resize_event(&self, size: PhysicalWindowSize) -> Result<(), PlatformError> {
if size.width == 0 || size.height == 0 {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I‘m surprised that this is needed in light of 0db8650 . Is there another call site for this function?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I needed this commit on top of 1.14.1 and didn't see during rebase that it was fixed in master in that other function already.

I removed the commit.

The surface capabilities don't only list formats that are usable for
render attachments. Since those formats should be available most of the
time, pick either BGRA or RGBA from the list of supported formats.

Fixes slint-ui#10233
@tronical tronical merged commit 11b33fb into slint-ui:master Jan 8, 2026
38 checks passed
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

Successfully merging this pull request may close these issues.

2 participants