search

eve-seat / seat

SeAT 0x. [UNSUPPORTED]
https://github.com/eveseat/seat
MIT License
69 stars 37 forks source link

* fix tooltip display on character industry view #292

Closed warlof closed 9 years ago

warlof commented 9 years ago

Fix for tooltip issue on character industry

eve-seat commented 9 years ago

Is there not a way to get this type of fix across the application so that tooltips will work on all ajax responses? This is then instead of just fixing it in 1 view, but all that may suffer this problem?

warlof commented 9 years ago

well, will try with a trigger but absolutely not sure about the result. tooltip works onload

warlof commented 9 years ago

done with this commit on all characters json call. I've used the complete trigger of the tabs loader.

eve-seat commented 9 years ago

I think this fix is better suited globally, and not just within the character context like your PR has. app.js can have something like:

// in public/assets/js/app.js
$.ajaxStop(function() {
  $('[data-toggle="tooltip"]').tooltip();
});

This should have it work on all ajax application wide. Please test if this works, and if so, make a new PR with 1 commit for this fix.

eve-seat commented 9 years ago

Did the global fix with ecf4446