Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Bug: calcite-pick-list issues after upgrade #919

Closed
AdelheidF opened this issue Apr 6, 2020 · 7 comments
Closed

Bug: calcite-pick-list issues after upgrade #919

AdelheidF opened this issue Apr 6, 2020 · 7 comments
Assignees
Labels
4 - verified bug Something isn't working p - high

Comments

@AdelheidF
Copy link

AdelheidF commented Apr 6, 2020

Summary

Still happening in 5.7.0 and beta.22. I think it started with 5.6.0.

The very first time in my browser session when I open the pick-list with check boxes all works fine. Any time after that I don't see check boxes.

First time:

After that:

But I can click on a field and the selection works. Just doesn't show a checkbox.

If I start up my browser session with a pick-list with radio buttons it also works the first time.

After that I no longer see the radio buttons and the selection change does not work.

If I start with check boxes or radio buttons it works once and then I then do the same again or the opposite, like open with check boxes after radio buttons, it doesn't work anymore.
Even if I destroy the smartMapping component and create it again it won't work anymore.

test case:
http://devext.arcgis.com/apps/mapviewer/index.html?urls=https://sampleserver6.arcgisonline.com/arcgis/rest/services/USA/MapServer/2
Click on 'Styles' on the right tool bar, then click on '+ Field'.

Actual Behavior

Expected Behavior

Reproduction Steps

Relevant Info

@AdelheidF AdelheidF added bug Something isn't working 0 - new labels Apr 6, 2020
@jcfranco jcfranco added this to the KOO (King of Ooo) milestone Apr 6, 2020
@jcfranco
Copy link
Member

jcfranco commented Apr 7, 2020

This bug is a regression caused by Stencil's new compiler, which the default since v1.10.0 (calcite-app-components picked it up in 5.6.0). FWIW, I think this has been an issue since 1.9.0, but went unnoticed because we weren't testing with the latest compiler flag. I think I've narrowed it down to the next compiler affecting the component lifecycle slightly and some assumptions made by the MutationObserver logic in the list. I'll do a little bit more testing and will submit a PR if everything looks 👌:sunglasses:.

@AdelheidF In the meantime, I think setting icon on the item-generating code could help as a workaround.

@AdelheidF
Copy link
Author

AdelheidF commented Apr 7, 2020

I think setting icon on the item-generating code could help as a workaround.

Yes, this solves the issue with the missing icons. But pick-list with multiple=false seems to have no effect. I still can select multiple items.

jcfranco added a commit that referenced this issue Apr 7, 2020
… react to DOM mutations earlier to avoid stagnant renders (#920)

#919
@jcfranco
Copy link
Member

jcfranco commented Apr 7, 2020

@AdelheidF Thanks for testing. Luckily, I've installed a fix for this that addresses the underlying issue.

@AdelheidF
Copy link
Author

AdelheidF commented Apr 7, 2020

With newly released 5.7.1 calcite-app-components (tested locally) the issues with the pick-list are fixed, but now drag-n-drop doesn't work anymore in value-list. That worked with 5.7.0.

@AdelheidF
Copy link
Author

5.7.2 DND works again

@AdelheidF
Copy link
Author

verified all locally

@jcfranco
Copy link
Member

jcfranco commented Apr 7, 2020

@AdelheidF Thanks! 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4 - verified bug Something isn't working p - high
Projects
None yet
Development

No branches or pull requests

2 participants