Browse Source

Fix Escalator spacer visibility calucation (#17826)

Now calls directly to correct spacer container to update visibility
instead of doing extra work with recalculating everything.

Change-Id: I8625033012773a10983cd5d4f6ef3ce1be50ee26
tags/7.6.0.alpha2
Teemu Suo-Anttila 9 years ago
parent
commit
be9f7f3128
1 changed files with 4 additions and 4 deletions
  1. 4
    4
      client/src/com/vaadin/client/widgets/Escalator.java

+ 4
- 4
client/src/com/vaadin/client/widgets/Escalator.java View File

@@ -1278,7 +1278,7 @@ public class Escalator extends Widget implements RequiresResize,
* <p>
* The implementation must call {@link #paintRemoveRow(Element, int)}
* for each row that is removed from the DOM.
*
*
* @param index
* the logical index of the first removed row
* @param numberOfRows
@@ -4994,7 +4994,8 @@ public class Escalator extends Widget implements RequiresResize,
spacerScrollerRegistration.removeHandler();
spacerScrollerRegistration = null;
}
recalculateElementSizes();

updateSpacerDecosVisibility();
}

public Map<Integer, SpacerImpl> getSpacers() {
@@ -5305,7 +5306,7 @@ public class Escalator extends Widget implements RequiresResize,
initSpacerContent(spacer);

body.sortDomElements();
recalculateElementSizes();
updateSpacerDecosVisibility();
}

private void updateExistingSpacer(int rowIndex, double newHeight) {
@@ -6286,7 +6287,6 @@ public class Escalator extends Widget implements RequiresResize,
body.recalculateSectionHeight();
footer.recalculateSectionHeight();

body.spacerContainer.updateSpacerDecosVisibility();
scroller.recalculateScrollbarsForVirtualViewport();
body.verifyEscalatorCount();
body.reapplySpacerWidths();

Loading…
Cancel
Save