Closed wjgogogo closed 1 year ago
Latest commit: 3be1c2a1ba524bea967a4bdd0c15a53a47769189
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
👋 @wjgogogo
💖 Thanks for opening this pull request! 💖
Please follow the contributing guidelines. And we use semantic commit messages to streamline the release process.
Examples of commit messages with semantic prefixes:
fix: don't overwrite prevent_default if default wasn't prevented
feat: add graph.scale() method
docs: graph.getShortestPath is now available
Things that will help get your PR across the finish line:
npm run lint
locally to catch formatting errors earlier.We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.
👋 @wjgogogo
Congrats on merging your first pull request! 🎉🎉🎉
Description
在以前的版本中,写死了 ColorPicker 组件的 container, 保证了全屏模式下,弹出窗不会被遮挡,但是 container 的层级比较深,在非全屏的模式下,会出现其他 z-index 更高的节点在它上面,且现在的结构下,没办法单纯通过调整 z-index 达到目的:
通过提供自定义 container 的能力,能让开发者更自由的挂载组件,避免遮挡的出现。
Motivation and Context
Types of changes
Self Check before Merge