-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very (very) first short sweep just one private keyword that isn't supposed to be there. I'll have another in-depth look tomorrow.
Thank you so much for your efforts Andrei! Especially this time of year!
It's a "control" (like ListView or CollectionView) or is a View that you can use inside ListView or CollectionView (like SwipeView...) |
I've played around with it and it seems pretty solid. The only thing I can't seem to figure out is how to use the Command on this. I've added the Command binding and tried to let all other Expanders close when one opens, I think that is something people would typically would want as well. But whatever I do, I can't seem to get the Command to fire. Is there something I am doing wrong? |
Comman is invoked each time when user taps it) |
You can observe IsExpanded property |
@jfversluis Hey Gerald, need you more explanation? :) Command and Tapped are invoked once a user taps over a header. Also, we change IsExpanded to the opposite value. |
But that is just the thing, it wasn't invoked when I tried it :) Did you try it? Does it work? If so, please show me how. Maybe you can add it to the sample? |
@jfversluis yes, I can extend the sample. |
7dcddb7
to
f3aee53
Compare
@jfversluis check it please |
Yes! Awesome, thanks! Not sure why, the command just didn't seem to fire for me. Just retried on both iOS and Android and seems to work fine. Thank you for that! Any specific reason you didn't add any tests? |
What kind of test do you mean? Unit or UI? |
Ideally both! But I guess UI tests would make most sense for this :) |
@jfversluis yes. Frankly speaking, I do not understand what can be covered by unit tests in this class. As for UI tests, I tried to run them at least a half year ago but didn't work out it. (Tried on Mac). Also, I have no experience in writing UI tests |
@jfversluis I will appreciate your assistance in writing tests ;) |
Sorry, was still on my todo list! Could you reach out to gerald.versluis [weird email sign] microsoft.com. Let's see if we can get the tests to run on your machine short term. If not, we might add something or do it later. Thanks! |
Co-Authored-By: Gerald Versluis <github@geraldversluis.nl>
@AndreiMisiukevich by "100 percent cross-platform" does this mean it works on the macOS, UWP, WPF, and GTK targets of Xamarin.Forms as well? |
Well, basically it should work, but I don’t promise that
On Mon, Apr 13, 2020 at 10:09 Matthew Joughin ***@***.***> wrote:
@AndreiMisiukevich <https://github.com/AndreiMisiukevich> by "100 percent
cross-platform" does this mean it works on the macOS, UWP, WPF, and GTK
targets of Xamarin.Forms as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#9044 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACNH4DUWROC5HZWWPMQSEWTRMK3ELANCNFSM4KBCQFZA>
.
--
*Andrei Misiukevich*
*Lead Software Engineer at ISsoft Solutions*
*mobile: *+375 29 <+375%2017%20389%200100>303 40 07 Email:
andrei.misiukevich@gmail.com Skype: csi.andreymisyukevich
*Minsk,* *Belarus *(GMT+3)
|
That is why this control is under experimental feature-flag |
If I generate a list of expanders and add it to a StakeLayout one by one, iOS works most of the time, but Android crashes on first or second time I tap the expander header. Here are the stack traces for both iOS and Android:
iOS:
Android:
|
@benazir46 |
Hey @AndreiMisiukevich jumping in to say congrats on getting this component into the Xamarin Forms project! 🥇 |
Thanks, mate) |
Description of Change
Hi folks,
Currently, there is no expander control in the XF box.
Perhaps this control will be useful for many developers.
I tried to align API according to Windows Expander:
https://docs.microsoft.com/en-us/windows/communitytoolkit/controls/expander
API Changes
Added:
So there are two approaches how to use Expander
Platforms Affected
Since this implementation is 100 percent cross-platform, the changes affect Core part only.
Testing Procedure