feat: message-box support appendTarget#65
Merged
levy9527 merged 3 commits intoFEMessage:devfrom Aug 10, 2020
Merged
Conversation
Member
|
不如看看能不能改成支持 shadowDom ？这样应该就可以随行所欲的写样式了 |
|
Deploy preview for femessage-element ready! Built with commit 4453ad3 |
|
😂抱歉现在才来处理。补充个文档吧，让其他人也能学习使用。 |
Author
|
|
0f4a2f1 to
4453ad3
Compare
Author
|
其实如果要采用这个方案的话，那么包括 dialog drawer tooltip 之类的组件都要新增这个api，目前这里只改动了 message-box |
没事，有用到再说。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Please make sure these boxes are checked before submitting your PR, thank you!
devbranch.由于 message-box 目前只能是 append to body，子应用如果需要修改覆盖其默认样式，只能通过 custom-class 写入带 name scoped 的全局 class 名
故拓展该组件，支持传入 插入文档的目标位置