Skip to content

Partially revert "[Windows] Update CollectionView changing ItemsLayout (#14532)#18356

Merged
PureWeen merged 1 commit into
net8.0from
dev/ema/revert-cv
Oct 26, 2023
Merged

Partially revert "[Windows] Update CollectionView changing ItemsLayout (#14532)#18356
PureWeen merged 1 commit into
net8.0from
dev/ema/revert-cv

Conversation

@emaf
Copy link
Copy Markdown
Contributor

@emaf emaf commented Oct 25, 2023

Description of Change

This partially reverts commit d569287, keeping the AdaptiveCollectionView sample.

The reverted change was switching to only use FormsGridView as backing control for CollectionView what causes issues like #16234.

This re-introduces #7747.

Issues Fixed

Fixes #16234
Fixes #18128

@emaf emaf requested a review from a team as a code owner October 25, 2023 17:33
@emaf emaf requested review from PureWeen and mattleibow October 25, 2023 17:33
@PureWeen PureWeen requested review from Redth and hartez October 25, 2023 19:55
#14532)"

This partially reverts commit d569287,
keeping the AdaptiveCollectionView sample.

The reverted change was switching to only use FormsGridView as backing
control for CollectionView what causes issues like
#16234.
@emaf emaf force-pushed the dev/ema/revert-cv branch from 1382213 to 7f99b06 Compare October 25, 2023 20:18
@emaf emaf requested a review from a team as a code owner October 25, 2023 20:18
@emaf emaf changed the base branch from main to net8.0 October 25, 2023 20:18
@emaf emaf removed the request for review from a team October 25, 2023 20:20
@mattleibow
Copy link
Copy Markdown
Member

We should re-open #7747 once this is merged.

@mattleibow
Copy link
Copy Markdown
Member

@emaf you mentioned that GridView only measures the first item. Where is this info coming from? Your tests or is this documented? Is this a bug in the WASDK maybe?

@PureWeen PureWeen merged commit ac996a0 into net8.0 Oct 26, 2023
@PureWeen PureWeen deleted the dev/ema/revert-cv branch October 26, 2023 14:37
emaf added a commit that referenced this pull request Oct 26, 2023
emaf added a commit that referenced this pull request Oct 26, 2023
PureWeen pushed a commit that referenced this pull request Oct 27, 2023
PureWeen pushed a commit that referenced this pull request Oct 27, 2023
emaf added a commit that referenced this pull request Nov 3, 2023
PureWeen pushed a commit that referenced this pull request Nov 6, 2023
emaf added a commit that referenced this pull request Nov 10, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 5, 2023
@samhouts samhouts added the fixed-in-8.0.100-rc.2.9530 Look for this fix in 8.0.100-rc.2.9530 label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

fixed-in-8.0.100-rc.2.9530 Look for this fix in 8.0.100-rc.2.9530 platform/windows

Projects

None yet

4 participants