Compatibility adjustments to ensure react-layer-stack works in React 15.5+ and a small enhancement #17
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi Alexey,
I've already provided an issue regarding this, but in the meantime I also fixed it myself in a fork, so I thought it might be helpful to provide it here.
What I did in this fork are two things:
I also experimented a bit to check if it would also be possible to assign additional classes to those wrappers when there are actively shown layers (think ".layer-stack-wrapper.has-active-layers > .layer-wrapper.is-active") but that didn't work as expected because the LayerStack does not seem to re-render itself when a change in the layer stack happens (only the LayerMountPoints change), so I threw that out again and ended up with this simplified version.
If you approve of those two additional props, I'd appreciate if you would accept this pull request so that your awesome module works once more in the current version of react.
Beste regards,
Stefan