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

fix: drop JavaScript function tipsy() in favor of bootstrap tooltip #610

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

DeepDiver1975
Copy link
Member

@DeepDiver1975 DeepDiver1975 commented Nov 14, 2023

refs owncloud/core#41099

can be released any time - but before 10.14 goes out ....

@@ -7,6 +7,4 @@ $(document).ready(function() {
}
OC.AppConfig.setValue('customgroups', $(this).attr('name'), value);
});

$('.section .icon-info').tipsy({gravity: 'w'});
Copy link
Member

Choose a reason for hiding this comment

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

I'd expect a $('.....').tooltip() or something similar given the description. I assume this is intended.

Copy link
Member Author

Choose a reason for hiding this comment

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

$('.section .icon-info').tooltip() is already called in core - it was a duplicate form the first day

Copy link

sonarcloud bot commented Nov 15, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@DeepDiver1975 DeepDiver1975 merged commit c176e5b into master Nov 15, 2023
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/drop-tipsy branch November 15, 2023 11:01
@DeepDiver1975 DeepDiver1975 changed the title fix: drop JavaScript function tipsy() in favor of botstrap tooltip fix: drop JavaScript function tipsy() in favor of bootstrap tooltip Nov 16, 2023
This was referenced Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants