Unverified Commit 8cb35ba0 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #1870006 by nod_, BarisW, alwaysworking, Renrhaf, pk188,... 

Issue #1870006 by nod_, BarisW, alwaysworking, Renrhaf, pk188, andrewmacpherson, ranjith_kumar_k_u, vikashsoni, Gauravmahlawat, wolffereast, Wim Leers, jessebeach, xjm: HTML5 validation with table sticky header is misaligned over the toolbar

(cherry picked from commit 9ec511c8)
parent a4f3ff93

core/misc/tableheader.es6.js

+7 −0
Original line number Diff line number Diff line
@@ -208,6 +208,8 @@ 
       * Create the duplicate header.

       */

      createSticky() {

        // For caching purposes.

        this.$html = $('html');

        // Clone the table header so it inherits original jQuery properties.

        const $stickyHeader = this.$originalHeader.clone(true);

        // Hide the table to avoid a flash of the header clone upon page load.
@@ -245,6 +247,11 @@ 
        if (typeof offsetLeft === 'number') {

          css.left = `${this.tableOffset.left - offsetLeft}px`;

        }

        this.$html.css(

          'scroll-padding-top',

          displace.offsets.top +

            (this.stickyVisible ? this.$stickyTable.height() : 0),

        );

        return this.$stickyTable.css(css);

      },

core/misc/tableheader.js

+2 −0
Original line number Diff line number Diff line
@@ -90,6 +90,7 @@ 
    tableHeight: null,

    stickyVisible: false,

    createSticky: function createSticky() {

      this.$html = $('html');

      var $stickyHeader = this.$originalHeader.clone(true);

      this.$stickyTable = $('<table class="sticky-header"></table>').css({

        visibility: 'hidden',
@@ -110,6 +111,7 @@ 
        css.left = "".concat(this.tableOffset.left - offsetLeft, "px");

      }



      this.$html.css('scroll-padding-top', displace.offsets.top + (this.stickyVisible ? this.$stickyTable.height() : 0));

      return this.$stickyTable.css(css);

    },

    checkStickyVisible: function checkStickyVisible() {

core/modules/toolbar/js/views/ToolbarVisualView.es6.js

+3 −0
Original line number Diff line number Diff line
@@ -86,6 +86,9 @@ 
        $('body').css({

          'padding-top': this.model.get('height'),

        });

        $('html').css({

          'scroll-padding-top': this.model.get('height'),

        });



        this.triggerDisplace();

      },

core/modules/toolbar/js/views/ToolbarVisualView.js

+3 −0
Original line number Diff line number Diff line
@@ -36,6 +36,9 @@ 
      $('body').css({

        'padding-top': this.model.get('height')

      });

      $('html').css({

        'scroll-padding-top': this.model.get('height')

      });

      this.triggerDisplace();

    },

    triggerDisplace: function triggerDisplace() {