CommunityToolkit/WindowsCommunityToolkit

AdaptiveGridView throws argument exception

Open

#3 142 ouverte le 21 févr. 2020

Voir sur GitHub
 (11 commentaires) (3 réactions) (0 assignés)C# (1 403 forks)batch import
bug :bug:controls :control_knobs:help wantedneed more info :pushpin:

Métriques du dépôt

Stars
 (5 708 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Upon resizing the control the adaptive grid view throws argument exception.

This happens when toggling the visibility of a control using the adaptive grid view. On some cases theCalculateItemWidth method returns an invalid Item width.

[Update] var columns = CalculateColumns(containerWidth, DesiredWidth); CalculateColumns method returns an invalid value (negative value) when DesiredWidth value is 0, But still can't get enough clue why the desired width is set to 0.

StackTrace is as below System.ArgumentException: Value does not fall within the expected range. at Windows.UI.Xaml.DependencyObject.SetValue(DependencyProperty dp, Object value) at Microsoft.Toolkit.Uwp.UI.Controls.AdaptiveGridView.RecalculateLayout(Double containerWidth) at Microsoft.Toolkit.Uwp.UI.Controls.AdaptiveGridView.OnSizeChanged(Object sender, SizeChangedEventArgs e)

Guide contributeur

Direction de recherche
Tracez la méthode `CalculateColumns` pour comprendre pourquoi `DesiredWidth` devient 0. Inspectez les méthodes `RecalculateLayout` et `CalculateItemWidth` pour les cas limites lors du basculement de la visibilité.
Stack technique
csharp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C#UWP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60