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

the args of notification.useNotification does not work. #7875

Open
1 task done
Dedicatus546 opened this issue Oct 14, 2024 · 2 comments
Open
1 task done

the args of notification.useNotification does not work. #7875

Dedicatus546 opened this issue Oct 14, 2024 · 2 comments

Comments

@Dedicatus546
Copy link

Dedicatus546 commented Oct 14, 2024

  • I have searched the issues of this repository and believe that this is not a duplicate.

Version

4.2.5

Environment

latest version

Reproduction link

https://github.com/Dedicatus546/ant-design-vue-notification-bug-report

Steps to reproduce

steps:

  • start dev server.
  • top's value is still 24px , but the source code is 80px.

What is expected?

top's value should be 80px .

What is actually happening?

top's value is still 24px .

@Dedicatus546
Copy link
Author

Dedicatus546 commented Oct 14, 2024

it seems like that Holder's props lack some properties.

image

image

the top property can be found in attrs.

image

but getStyles would check the props.top instead of attrs.top.

image

the notification's static method doesn't have this problem.

@Dedicatus546
Copy link
Author

btw, i found that the icons from notification.info and notification.useNotification are not same.

image

is it intentional?

@Dedicatus546 Dedicatus546 changed the title the args of notification.useNotification is not work. the args of notification.useNotification does not work. Oct 25, 2024
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

No branches or pull requests

1 participant