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.

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

Refreshing a region containing tooltips causes a memory leak #220

Closed
stephanrauh opened this issue Dec 28, 2015 · 3 comments
Closed

Refreshing a region containing tooltips causes a memory leak #220

stephanrauh opened this issue Dec 28, 2015 · 3 comments
Assignees
@stephanrauh
Labels
bug
Milestone
v0.8.5

Comments

@stephanrauh
Copy link
Collaborator

This problem can be observed in the AJAX+RTL demo of switch.xhtml in the showcase. Moving the mouse over the switch makes the tooltip occur. When you toggle the switch, it is replaced with a newer version of the same switch. However, the tooltip remains, and there's no way to hide it again. Looking at the end of the HTML view reveals that there are two tooltip divs. Each time you toggle the switch, another tooltip div appears.
@stephanrauh stephanrauh self-assigned this Dec 28, 2015
@stephanrauh stephanrauh added this to the v0.8.1 milestone Dec 28, 2015
@stephanrauh
Copy link
Collaborator Author

Destroying the tooltip before sending the AJAX request might help: http://stackoverflow.com/questions/28220597/destroy-all-bootstrap-tooltips-in-a-div

@stephanrauh
Copy link
Collaborator Author

Similar memory leaks should affect every component based on jQuery. Marco Rinck's JUA framework might help (after adapting it to jQuery instead of AngularJS).

@stephanrauh
Copy link
Collaborator Author

Todo: update layout/navigationAndAJAX.jsf of the showcase after fixing the bug.

@TheCoder4eu TheCoder4eu modified the milestones: v0.8.2, v0.8.1 Jan 15, 2016
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 21, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
1ab869b 
…u#323
@zhedar zhedar mentioned this issue Apr 21, 2016
Merged
@stephanrauh stephanrauh mentioned this issue Apr 21, 2016
Closed
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 22, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
0215964 
…u#323
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 22, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
98da58c 
…u#323
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 25, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
c8f8586 
…u#323
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 27, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
f6eee91 
…u#323
zhedar added a commit to zhedar/BootsFaces-OSP that referenced this issue Apr 28, 2016
@zhedar
destroy existing tooltips, fixes TheCoder4eu#220 and fixes TheCoder4e…
f3356e0 
…u#323
@zhedar zhedar mentioned this issue May 7, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stephanrauh stephanrauh

Labels
bug
Projects
None yet
Milestone
v0.8.5
Development

No branches or pull requests
2 participants
@stephanrauh @TheCoder4eu