Skip to content
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

支持多层popup #1592

Open
wants to merge 2 commits into
base: v2
Choose a base branch
from
Open

支持多层popup #1592

wants to merge 2 commits into from

Conversation

WangLin2011
Copy link

@WangLin2011 WangLin2011 commented Jun 21, 2017

Please makes sure the items are checked before submitting your PR, thank you!

  • Rebase before creating a PR to keep commit history clear.
  • Only One commit
  • No eslint errors

@airyland
Copy link
Owner

我怎么记得现在是已经可以多层 popup 的

@WangLin2011
Copy link
Author

masker不行,从全屏popup中弹出半屏popup,masker是共用的一个,所以后面弹出的popup看不到masker,所以没法点击masker关闭后面弹出的popup

@WangLin2011
Copy link
Author

popup

如上图显示,表单在一个全屏的popup中,点击月营收入,希望出现有遮罩的PopupPicker,但是由于所有的popup都使用的同一个遮罩,并且这个遮罩的z-index是500,popup里的内容的z-index为501,所以在月营收入的picker弹出后看不到遮罩,希望能解决这个问题。
ps: 通过修改遮罩的z-index,可以出现如下图想要的效果,但是点击遮罩后,所有的popup都关闭了,这也是不希望的效果,所以修改时,使用每个popup一个遮罩简单粗暴的方法。
popup2

@airyland
Copy link
Owner

能否在 popup demo 里添加一个稍微复杂的多层 popup?

@WangLin2011
Copy link
Author

您好,目前文档中本来就有多层popup的例子,只不过都是半屏幕的,都能看见遮罩层,但是存在的问题就是点击遮罩后,两个popup都同时关闭了。需要我再增加一个全屏加半屏popup的demo么?

@airyland
Copy link
Owner

对,麻烦再添加一个吧 😄

@WangLin2011
Copy link
Author

已增加一个稍微复杂的demo,在此demo中,如果放在原来没有修改的组件的环境中跑,那么在点击“颜色选择”时,将看不到弹出的全屏popup,因为之前的多层popup依赖于popup书写的先后顺序。望审一下代码,谢谢。

@WangLin2011
Copy link
Author

@airyland 知道阁下业务繁忙,可能遗忘了此小小的pr,望阁下抽出一丢丢宝贵的时间,审核代码,如有不足,望指导之,他日必以贡献更多的开源代码回报之,拜谢~

@airyland
Copy link
Owner

airyland commented Jul 7, 2017

这两天就处理。

@hedanto
Copy link

hedanto commented Sep 13, 2017

@airyland 我遇到同样的问题,不知道,有没有处理了这个pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants