-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Picker Attribute "SelectedIndex" Not being respected on page load on Android? #9150
Comments
Edit: Can confirm that this happens on iOS. Will check Windows and Mac tomorrow. Investigated this for a few hours. It seems that in XamlNode.cs the properties of Picker have SelectedIndex above the Binding PickerItems. It first then iterates into SelectedIndex, eventually it comes in BindableObject.cs in SetValueCore, where property.CoerceValue is executed against the selectedIndex from the xaml (1 in my example). It is coerced to -1 because there are no items yet, and after that it is not loaded from the XamlNode again. The next property after SelectedIndex is the PickerItems Binding. I have yet to figure out what determines the order in which properties are listed, so that I can change the behavior. As seen in the screenshot other properties x:name and textColor are applied after the Binding PickerItems. |
I think this is the same issue as #7138 |
Reproduced the issue on Android, iOS, Windows and Mac Catalyst. Labels of issue can be updated. Also ideas about solution direction are welcome. @jfversluis Edit: Also tested this in Xamarin Forms, and this issue is also reproducible there. Is this perhaps intended behavior or should an issue be created in Xamarin Forms github as well? |
Same issue here on Mac Catalyst. |
Verified this on Visual Studio Enterprise 17.7.0 Preview 1.0. Repro on Windows 11, Android 13.0 and iOS 16.4 with below Project: |
Frighteningly, more than a year now and nothing, yikes. In case anyone is still interested, this is still going on. In fact, I can't even set the SelectedIndex in the page's constructor as a work-around. The default value is -1 and even after an explicit setting of the value, it does not change. |
You're right, this still isn't fixed. The workaround is to use |
(Added labels per #9150 (comment) comment) |
Is this ever getting fixed, its getting annoying how long this has been open and the sad part is this is a control which is used ALOT |
Description
I have a Picker that is attached to an enum for a "list type" property, with a "SelectedIndex=0" attribute on the picker, which I believe means that I want that index of the enum to be selected first, as a 'default'. This works great on the windows app, but when running on Android, the picker control is empty. Then, I'm able to force something into the control by manually changing the "selected index" in the .xaml from say, 0 to 1, which I think then triggers the event, and updates in the app. But obviously since I think the point of SelectedIndex is to default something in, then, well, it should default something in.
I found a workaround for this by grabbing the picker control in the constructor in the code-behind for the page, and setting it's SelectedIndex property there, which fixes it. But I presume that the xaml property should work without needing to do that.
Here is the repo for the demo app i'm playing with -
Xaml for the view with the Picker - https://github.com/Programming-With-Chris/ShoppingList/blob/master/ShoppingList/View/UserListDataInput.xaml
Code Behind showing the 'work around' - https://github.com/Programming-With-Chris/ShoppingList/blob/master/ShoppingList/View/UserListDataInput.xaml.cs
Steps to Reproduce
Version with bug
6.0.408 (current)
Last version that worked well
Unknown/Other
Affected platforms
Android
Affected platform versions
'android'">21.0
Did you find any workaround?
Yes, as described above - "I found a workaround for this by grabbing the picker control in the constructor in the code-behind for the page, and setting it's SelectedIndex property there, which fixes it. "
Relevant log output
No response
The text was updated successfully, but these errors were encountered: