Skip to content

Commit

Permalink
Don't add fb if attendee tab isn't visible.
Browse files Browse the repository at this point in the history
Fixes a .defer() loop due to the td.getWidth() check
further down the method. This was causing FF to become
unresponsive if the event view was left open long enough
without viewing the attendee tab.

Not sure what the check for td.getWidth() is supposed to
catch/prevent...so the correct fix may be to just remove that
conditional. It looks like the insertFreeBusy method is triggered
when viewing the AttendeeTab each time anyway.
  • Loading branch information
mrubinsk committed Aug 17, 2016
1 parent e958b2b commit b9297ab
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions kronolith/js/kronolith.js
Original file line number Diff line number Diff line change
Expand Up @@ -6736,6 +6736,7 @@ KronolithCore = {
insertFreeBusy: function(attendee, start)
{
if (!$('kronolithEventDialog').visible() ||
!$('kronolithEventTabAttendees').visible() ||
!this.freeBusyRows.get(attendee)) {
return;
}
Expand Down

0 comments on commit b9297ab

Please sign in to comment.