Skip to content

fix-typo-enableOnHover #2021

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

Merged

Conversation

heesungjang
Copy link

@heesungjang heesungjang commented Jul 3, 2025

Q A
Bug fix? [ O ]
New feature? [ ]
New sample? [ ]
Related issues? fixes #2020

What's in this Pull Request?

Expected Behavior
When enableOnHover is set to true in the event object in Calendar control, it should enable hover effects and display a popover with event details.

Observed Behavior
Setting enableOnHover to true does not enable the hover effect, and no popover appears.

This appears to be due to a typo in the actual codebase. The property is currently named enableOnHouver instead of enableOnHover.

The documentation correctly refers to it as enableOnHover, so I assume the typo is in the codebase, not the documentation.

Guidance

  • You can delete this section when you are submitting the pull request.
  • Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
  • Please target your PR to dev branch.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for the first contribution to this project.

@joaojmendes joaojmendes self-assigned this Jul 13, 2025
@joaojmendes
Copy link
Collaborator

Hey @heesungjang , Thank you for the fix!

@joaojmendes joaojmendes merged commit 05b5f8f into pnp:dev Jul 13, 2025
1 check passed
@joaojmendes joaojmendes added the status:fixed Issue fixed in current or prior release. label Jul 13, 2025
@joaojmendes joaojmendes added this to the 3.22.0 milestone Jul 13, 2025
@PopWarner
Copy link
Member

🎉✨ Woohoo! ✨🎉

Thank you @heesungjang for your awesome contribution to the community with this merged PR! 🚀 You've earned a Credly badge through the Microsoft 365 & Power Platform Recognition Program. 🏅 To receive your badge, please take a moment to opt in to the program. You can learn more and sign up here: https://aka.ms/community/recognition.

Can't wait to see you in the program! 🎊

cc: @joaojmendes

Cheers,
David Warner 🧡💜

@heesungjang
Copy link
Author

Done! Joined the program.

Thank you @PopWarner .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed Issue fixed in current or prior release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants