Skip to content

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Sep 4, 2022

prepend=queue 时,第一个 style 会添加到最前端,之后 style 会依次添加。

ref ant-design/ant-design#36742

@vercel
Copy link

vercel bot commented Sep 4, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
util ✅ Ready (Inspect) Visit Preview Sep 5, 2022 at 1:20PM (UTC)

@codecov
Copy link

codecov bot commented Sep 4, 2022

Codecov Report

Merging #337 (b8bc2aa) into master (780a217) will increase coverage by 0.46%.
The diff coverage is 100.00%.

❗ Current head b8bc2aa differs from pull request most recent head 804af2e. Consider uploading reports for the commit 804af2e to get more accurate results

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
+ Coverage   67.43%   67.90%   +0.46%     
==========================================
  Files          47       47              
  Lines        1035     1050      +15     
  Branches      343      347       +4     
==========================================
+ Hits          698      713      +15     
  Misses        298      298              
  Partials       39       39              
Impacted Files Coverage Δ
src/Dom/dynamicCSS.ts 96.92% <100.00%> (+0.92%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@MadCcc
Copy link
Member

MadCcc commented Sep 4, 2022

🐂的

const existStyle = findStyles(container).filter(node =>
['prepend', 'prependQueue'].includes(node[APPEND_ORDER]),
);
if (existStyle.length) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

所以这里第一个 style 还是 prepend?qiankun 没劫持 prepend,会透出去,后面 exitStyle 找不到还是会继续 prepend?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prepend 让 qiankun 处理就好了,否则 rc-util 就要做框架检测了,这样就变成针对 qiankun 的耦合了。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants