Skip to content

Conversation

@Arieldelossantos
Copy link
Contributor

GitHub Issue: #1188

PR Type

What kind of change does this PR introduce?

  • Bugfix

Description

Wasm

Before After
WASM-button-before WASM-button-after

Android

Before After
AND-button-before AND-button-after

GTK

Before After
GTK-button-before GTK-button-after

iOS

Before After
iOS-button-before iOS-button-iOS

PR Checklist

Please check if your PR fulfills the following requirements:

@Arieldelossantos Arieldelossantos self-assigned this Sep 5, 2023
@Arieldelossantos Arieldelossantos linked an issue Sep 18, 2023 that may be closed by this pull request
6 tasks
Copy link
Member

@Youssef1313 Youssef1313 left a comment

Choose a reason for hiding this comment

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

The issue mentions that things are working properly on Windows already, so that would mean it's bug in core uno.

Can you please confirm if this still reproduces on Skia with latest Uno dev? Recently, there were improvements to the measure/arrange phases for Skia and Wasm.

@kazo0
Copy link
Collaborator

kazo0 commented Oct 11, 2023

The issue mentions that things are working properly on Windows already, so that would mean it's bug in core uno.

Can you please confirm if this still reproduces on Skia with latest Uno dev? Recently, there were improvements to the measure/arrange phases for Skia and Wasm.

@Arieldelossantos can you confirm if this is still an issue without this fix?

@Arieldelossantos
Copy link
Contributor Author

Confirmed that issue still occurs with Uno 5.0.0-dev.3431 on platforms Android, GTK, Wasm and iOS on master without the fix
cc.: @kazo0 @Youssef1313

@Arieldelossantos Arieldelossantos force-pushed the dev/arde/fix-button-visibility-when-grid-cell-is-zero branch from ab3d0f9 to 6218f02 Compare October 11, 2023 20:56
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.

Button partially visible even in a zero width&height container

5 participants