Skip to content

Conversation

thinkasany
Copy link
Member

@thinkasany thinkasany commented Mar 3, 2025

🤔 This is a ...

  • 🆕 New feature

🔗 Related Issues

wait for react-component/image#378

📝 Change Log

Language Changelog
🇺🇸 English feat: ConfigProvider support classNames and styles for image
🇨🇳 Chinese feat: ConfigProvider support classNames and styles for image

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link
Contributor

github-actions bot commented Mar 3, 2025

👁 Visual Regression Report for PR #53028 Passed ✅

🎯 Target branch: next (f0ed87f)
📖 View Full Report ↗︎
📖 Alternative Report ↗︎

🎊 Congrats! No visual-regression diff found.

Copy link
Contributor

github-actions bot commented Mar 3, 2025

Preview is ready

Copy link

Walkthrough

This PR introduces a new feature to the ConfigProvider, adding support for classNames and styles for images. This enhancement allows for more customizable styling options for image components, particularly in preview configurations.

Changes

Files Summary
components/config-provider/context.ts Added classNames and styles to ImageConfig interface for image preview customization.
components/image/demo/_semantic.tsx Added a new demo file showcasing semantic DOM usage with image previews.
components/image/index.en-US.md, components/image/index.zh-CN.md Updated documentation to include semantic DOM example.
components/image/index.tsx Enhanced image preview functionality to support classNames and styles from context.

Copy link
Contributor

github-actions bot commented Mar 3, 2025

size-limit report 📦

Path Size
dist/antd.min.js 506.75 KB (-367 B 🔽)
dist/antd-with-locales.min.js 599.24 KB (-600 B 🔽)

Copy link

cloudflare-workers-and-pages bot commented Mar 3, 2025

Deploying ant-design with  Cloudflare Pages  Cloudflare Pages

Latest commit: b2e1c25
Status: ✅  Deploy successful!
Preview URL: https://8ce8076d.ant-design.pages.dev
Branch Preview URL: https://cp-image.ant-design.pages.dev

View logs

Copy link

codecov bot commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (55e075f) to head (b2e1c25).
Report is 13 commits behind head on next.

Additional details and impacted files
@@            Coverage Diff            @@
##              next    #53028   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          773       778    +5     
  Lines        13929     13999   +70     
  Branches      3652      3672   +20     
=========================================
+ Hits         13929     13999   +70     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

socket-security bot commented Mar 5, 2025

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@rc-component/image@1.0.01.2.5 None 0 135 kB zombiej

View full report↗︎

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

pkg-pr-new bot commented Mar 5, 2025

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

codecov bot commented Mar 5, 2025

Bundle Report

Changes will increase total bundle size by 2.27kB (0.06%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
antd.min-array-push 3.9MB 2.27kB (0.06%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: antd.min-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
antd-with-locales.min.js 1.15kB 2.14MB 0.05%
antd.min.js 1.12kB 1.76MB 0.06%

Files in antd-with-locales.min.js:

  • ./components/config-provider/context.ts → Total Size: 1.51kB

  • ./components/image/style/index.ts → Total Size: 8.33kB

Files in antd.min.js:

  • ./components/image/style/index.ts → Total Size: 8.33kB

  • ./components/config-provider/context.ts → Total Size: 1.51kB

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

@thinkasany thinkasany force-pushed the cp-image branch 3 times, most recently from 4c09ef1 to 9788053 Compare March 10, 2025 03:49
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

@thinkasany
Copy link
Member Author

LGTM

@zombieJ zombieJ merged commit 23b8710 into next Mar 19, 2025
42 checks passed
@zombieJ zombieJ deleted the cp-image branch March 19, 2025 02:53
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