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

Windows HorizontalList CollectionView has some weird space if cells are smaller than 88 dp #15241

Closed
unombun opened this issue May 24, 2023 · 6 comments · Fixed by #12811
Closed
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView fixed-in-7.0.96 Look for this fix in 7.0.96 SR8! fixed-in-7.0.100 fixed-in-7.0.101 fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! layout-grid p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/windows 🪟 s/try-latest-version Please try to reproduce the potential issue on the latest public version t/bug Something isn't working

Comments

@unombun
Copy link

unombun commented May 24, 2023

Description

On Windows just create a horizontal collection view. If the cell has a width smaller then ~88dp a space will appear until the next cell

image

Steps to Reproduce

  1. Create a MAUI App for Windows
  2. Add a horizontal list CollectionView with items that are smaller than 88 dp.
  3. The content of the cell will become smaller, but then a space until the next cell will appear. Somehow like there is a minimum cell width.

Link to public reproduction project repository

https://github.com/flesarradu/WindowsHorizontalCollectionViewSpacingIssue

Version with bug

7.0.86

Last version that worked well

Unknown/Other

Affected platforms

Windows

Affected platform versions

Did you find any workaround?

No

Relevant log output

No response

@unombun unombun added the t/bug Something isn't working label May 24, 2023
@unombun unombun changed the title Windows HorizontalList CollectionView has some weird space if cells are smaller than 90 dp Windows HorizontalList CollectionView has some weird space if cells are smaller than 88 dp May 24, 2023
@mikeparker104 mikeparker104 added the partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with label May 24, 2023
@samhouts samhouts added the p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint label May 24, 2023
@mattleibow mattleibow added this to the Backlog milestone May 24, 2023
@ghost
Copy link

ghost commented May 24, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@samhouts samhouts modified the milestones: Backlog, .NET 8 May 24, 2023
@Eilon Eilon added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label May 25, 2023
@bwoebi
Copy link
Contributor

bwoebi commented May 30, 2023

This affects CollectionViews in both directions equally - on my end it seems like Windows forces a Height of at least 40 for each entry in the CollectionView.

@jsuarezruiz
Copy link
Contributor

Tested in main branch and cannot reproduce the issue:
image

Could you try again with the latest version?

@samhouts samhouts added the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Jun 15, 2023
@ghost
Copy link

ghost commented Jun 15, 2023

Hi @unombun. We have added the "s/try-latest-version" label to this issue, which indicates that we'd like you to try and reproduce this issue on the latest available public version. This can happen because we think that this issue was fixed in a version that has just been released, or the information provided by you indicates that you might be working with an older version.

You can install the latest version by installing the latest Visual Studio (Preview) with the .NET MAUI workload installed. If the issue still persists, please let us know with any additional details and ideally a reproduction project provided through a GitHub repository.

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@ghost ghost closed this as completed Jun 23, 2023
@samhouts samhouts added s/try-latest-version Please try to reproduce the potential issue on the latest public version and removed s/try-latest-version Please try to reproduce the potential issue on the latest public version labels Jul 18, 2023
@dotnet dotnet deleted a comment Jul 18, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! label Aug 15, 2023
@mrderon
Copy link

mrderon commented Sep 2, 2023

@samhouts Can you please confirm that this fix is working? I'm using 8.0.0-preview.7.8842 with the sample app given in #12495 and it still doesn't seem to work (result still has the red spacing as initially reported).

@ghost
Copy link

ghost commented Sep 2, 2023

Hello lovely human, thank you for your comment on this issue. Because this issue has been closed for a period of time, please strongly consider opening a new issue linking to this issue instead to ensure better visibility of your comment. Thank you!

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView fixed-in-7.0.96 Look for this fix in 7.0.96 SR8! fixed-in-7.0.100 fixed-in-7.0.101 fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! layout-grid p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/windows 🪟 s/try-latest-version Please try to reproduce the potential issue on the latest public version t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants