-
-
Notifications
You must be signed in to change notification settings - Fork 51k
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
site: update docs styles #39348
site: update docs styles #39348
Conversation
typeof record.value === 'string' && | ||
(record.value.startsWith('#') || record.value.startsWith('rgb')); | ||
if (isColor) { | ||
return <Badge color={record.value} text={record.value} />; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我们现在貌似没有方形色块这样的组件。 @MadCcc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Badge 有点小,用上面那个 span 吧。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好像 Badge 也行,我还以为是个小圆点。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好,那我看看如何去修改。
可能我们真的需要一个方形色块组件 @MadCcc
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #39348 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 546 546
Lines 9366 9363 -3
Branches 2654 2652 -2
=========================================
- Hits 9366 9363 -3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
有演示页面看看么? |
可以加个边框和阴影:https://ngx-colors.web.app/ |
那还是保持现状吧,不做修改 |
不是有宽度缩掉的问题么 |
哈哈,我是觉得改个这个感觉就没必要 PR 了。那我改一下,重新打开 :) |
d6e9ef6
to
2f4bb65
Compare
This reverts commit 4a80396. # Conflicts: # .dumi/theme/builtins/ColorChunk/index.tsx # .dumi/theme/builtins/TokenTable/index.tsx # docs/react/customize-theme.en-US.md # docs/react/customize-theme.zh-CN.md
* chore: update blog style * Revert "chore: update blog style" This reverts commit afc0a1d. * Revert "site: update docs styles (#39348)" This reverts commit 4a80396. # Conflicts: # .dumi/theme/builtins/ColorChunk/index.tsx # .dumi/theme/builtins/TokenTable/index.tsx # docs/react/customize-theme.en-US.md # docs/react/customize-theme.zh-CN.md * chore: revert * chore:revert * chore: rename props * chore: update * chore(deps): add `dumi-plugin-color-chunk` plugin * chore: update docs * chore: update * Update package.json Co-authored-by: afc163 <[email protected]> Signed-off-by: 红 <[email protected]> * docs: pick #44945 Co-authored-by: RedJue <[email protected]> --------- Signed-off-by: 红 <[email protected]> Signed-off-by: 红 <[email protected]> Co-authored-by: MadCcc <[email protected]> Co-authored-by: afc163 <[email protected]> Co-authored-by: RedJue <[email protected]>
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
ref: #38463 (comment)
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge