-
Notifications
You must be signed in to change notification settings - Fork 8
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
Icons rendering inconsistently #11
Comments
Hello @MatthewKapteyn , Thank you for reporting the issue. I will carefully consider and investigate to find the best solution for this problem. If anyone else has additional ideas, feel free to share them. Best regards, |
I'm also experiencing the same. Any workaround? |
I have now checked the “SetCalendarIcon” method so that it is not async. Could you please try again? If possible can you send me a sample code that doesn't work? |
I tried it out but the issue persists. What i found was that loading more than handful of these controls on a page leads to a huge drain on resources. A demo page will start to lag trying to create more than 5-10 of them. When this happens the 100ms delay is not enough and any non-default icon will fail to load. Unfortunately this is because the Maui Image control for the icon has the same performance problems as Xamarin. I've replaced the Image control with an FFImage control. This allows for using the original 100ms delay and you can load as many NullableDateTimePicker controls on the page as you want. I've mocked up a torture test here |
Thank you very much for your suggestion and example. I will try and find the most suitable solution. Best Regards |
hello everyone I also added selection based on 12 hour format. Thanks again for this idea @MatthewKapteyn :) Best Regards |
Awesome! Hey thanks for porting in that 12hour feature, better looking than what i did. In regards to performance it's still a bit laggy with the 50+ scrolling torture test but much improved. Using custom icons that are small enough are ok too. I'll still have to test it out in a different project that loads a lot more fields into a page to see if the icon issue comes back but i think this will work great for most people. |
Thank you for your testing and ideas, if you have any other ideas or requests, you can share them. |
Hello @NijishaN , |
Thankyou for your response @sebarslan. Experiencing the issue with my own icons as well as the standard icons. |
Hi @NijishaN |
Hello @sebarslan, Still experiencing the issue. |
Hello @NijishaN Thank you very much for the test. |
Hi @sebarslan , I downloaded Sebarslan.Maui.NullableDateTime pakage from nuget package manager. Before selecting the date the icon was in expected size After selecting the date the icon size was increased. Expected behavior : The icon should be the same as it was before selecting the date or It would be better if you add IconSize property for this control. |
Hi @NijishaN |
Hello @sebarslan |
Hi @NijishaN Thanks |
If the inputs are being generated dynamically or part of a larger page, icons can appear to be massive or be missing entirely. The touch events still work but they aren't visible. Happens when providing custom icons too.
looking at the code i can see why, the post load 100ms delay to wait until things are hopefully visible before swapping icons doesn't seem to work consistently. Hoping to find a better method.
Maui.NullableDateTimePicker/Maui.NullableDateTimePicker/NullableDateTimePicker.cs
Line 111 in 554ab51
Will investigate further and provide screenshots. For now i've just disabled icons in my fork and moved the touch event to the Entry textbox.
The text was updated successfully, but these errors were encountered: