Changeset:
8e4ed8a49b50
https://sourceforge.net/p/mrbs/hg-code/ci/8e4ed8a49b50eb1156ed87c566046088928174e0
Author:
Campbell Morrison <[email protected]>
Date:
Tue Oct 18 17:20:51 2016 +0100
Log message:
Fixed a performance problem when clicking on an empty cell in the week view
when using hidden days on a large table.
diffstat:
web/js/resizable.js.php | 12 ++++++++----
1 files changed, 8 insertions(+), 4 deletions(-)
diffs (30 lines):
diff -r fc1092d92649 -r 8e4ed8a49b50 web/js/resizable.js.php
--- a/web/js/resizable.js.php Tue Oct 18 13:52:43 2016 +0100
+++ b/web/js/resizable.js.php Tue Oct 18 17:20:51 2016 +0100
@@ -611,8 +611,11 @@
var downHandler = function(e) {
mouseDown = true;
turnOffPageRefresh();
- <?php // Build the map of booked cells ?>
- table.find('td').not('td.new, td.row_labels').each(function() {
+ <?php
+ // Build the map of booked cells. (We select just the visible
cells
+ // because there could be hidden days).
+ ?>
+ table.find('td:visible').not('td.new,
td.row_labels').each(function() {
bookedMap.push(getSides($(this)));
});
<?php // Apply a wrapper to turn off highlighting ?>
@@ -933,9 +936,10 @@
.appendTo($('div.resizing'));
<?php
// Build the map of booked cells, excluding this cell
(because we're
- // allowed to be in our own cell
+ // allowed to be in our own cell. (We select just the
visible cells
+ // because there could be hidden days).
?>
- table.find('td').not('td.new,
td.row_labels').not(divBooking.closest('td')).each(function() {
+ table.find('td:visible').not('td.new,
td.row_labels').not(divBooking.closest('td')).each(function() {
bookedMap.push(getSides($(this)));
});
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Mrbs-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mrbs-commits
