Re-rendering root node breaks dynamic styles

[WalterWeidner]

Re-rendering root node breaks dynamic styles

[ernestostifano]

@Bundas @kof

Hi again guys, I can confirm that our issue is the same. Applied the modifications of the PR into a patch and everything is now working as expected! 🎉

[ernestostifano]

Hi guys, I am not pretending to make anyone fix this, just want to receive proper briefing to try fixing it myself.

Why is this PR (#1642) not good? Insufficient implementation? Missing tests?

I really like JSS, have been using it since the beginning, but now my team is evaluating to migrate to other solutions (like React TSS), because this issue completely breaks our UIs.

This is a friendly comment guys, I would really like to help or at least to understand, because this repo is not having a lot of activity and that is a risk for us (the project is really big).

@WalterWeidner @Bundas @fjpedrosa @kof

[fjpedrosa]

I am interested on this. Will you open a PR?

[Bundas]

Hey guys, we have encountered the same problem in our project. Basically, each time the route changed, some styles didn't load properly. For example, if the table got rerendered, the last row would have broken styles. This indeed fixes the problem. I have created a PR

[ernestostifano]

Hi guys, we are currently facing the same (or at least similar) issue in a very big project. In our case, static styles get lost, probably because they are removed under the same conditions explained above. Dynamic styles continue to work because since they are linked to component's props, they get updated/added again.

We will try to debug this further, but everything seems to point to the remove function.

@Bundas @kof do you think the PR will be merged any soon? I will try to patch the dependency and see if our issue gets resolved.

Btw, we are using v10.10.0

[Bundas]

@ernestostifano I am glad it's working for you now. Unfortunately, the fix proposed by @WalterWeidner is not addressing the root cause. But it does the trick as a hotfix to get going.

btw for anybody wondering how to apply the patch to their project - we use https://www.npmjs.com/package/patch-package

[ernestostifano]

@Bundas oh, I understand. Is someone already working on it? Is it a priority for you? If needed, I could try to help. For us it turns to be an important issue (all the static styles get lost when it happens).

Regarding the patch, for people using modern versions of yarn, especially with PnP, we used yarn patch.

[ernestostifano]

I must add that in our case, the "root" was not being re-rendered, neither was the theme provider. It was sufficient to unmount just a portion of the app and mount it again, after unmounting/mounting some of the internal components.

[WalterWeidner]

Hey guys, we have encountered the same problem in our project. Basically, each time the route changed, some styles didn't load properly. For example, if the table got rerendered, the last row would have broken styles. This indeed fixes the problem. I have created a PR

Thanks. For some reason I wasn't notified of responses to this issue.

[ernestostifano]

Could be related: #1630