Skip to content

feat(tooltip): add tooltip samples (vnext) #971

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 24, 2025
Merged

feat(tooltip): add tooltip samples (vnext) #971

merged 3 commits into from
Apr 24, 2025

Conversation

adrianptrv
Copy link
Contributor

Same changes as master PR: #970

@RivaIvanova RivaIvanova added status: in-test PR ready for testing and removed status: in-test PR ready for testing labels Apr 23, 2025
@RivaIvanova
Copy link
Member

The samples are great! In order for the PR to build, all samples should be updated to [email protected] - Updating Version of IgniteUI Packages.
Also, once the version is updated, run all samples so that they are automatically included in the index.html file and appear in the navigation.
image

@RivaIvanova RivaIvanova mentioned this pull request Apr 23, 2025
Closed
@RivaIvanova
Copy link
Member

@adrianptrv There is a separate task for updating the igniteui-webcomponents version for all existing samples #972. Once #973 is merged, update the version in the current samples to 5.4.1 and run all samples so they appear in the navigation, and the README files will be updated accordingly.

@RivaIvanova RivaIvanova added status: in-test PR ready for testing status: verified The PR is tested and ready for a merge and removed status: in-test PR ready for testing labels Apr 24, 2025
@RivaIvanova RivaIvanova linked an issue Apr 24, 2025 that may be closed by this pull request
Create samples for Tooltip component #962
Closed
@kacheshmarova kacheshmarova merged commit 0296604 into vnext Apr 24, 2025
5 checks passed
@kacheshmarova kacheshmarova deleted the apetrov/add-tooltip-samples-vnext branch April 24, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kacheshmarova kacheshmarova kacheshmarova approved these changes

@RivaIvanova RivaIvanova RivaIvanova approved these changes

@simeonoff simeonoff Awaiting requested review from simeonoff

Assignees
No one assigned
Labels
control: tooltip status: verified The PR is tested and ready for a merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create samples for Tooltip component
3 participants
@adrianptrv @RivaIvanova @kacheshmarova