Drupal 10.3 changes the sticky implementation, bootstrap sticky functionality.

Created on 5 July 2024, 3 months ago
Updated 11 August 2024, about 2 months ago

Problem/Motivation

See related issue for the issue history. See the change notice: https://www.drupal.org/node/3440477 β†’

Steps to reproduce

1. Update to core 10.3
2. Use sticky for any table
3. Observe no sticky behavior.
4. Observe javascript console displays error "Uncaught TypeError: Drupal.TableHeader is undefined"

Proposed resolution

Unsure. Also unsure how to support D10.2 and below and also D10.3 and above.

See related issue for the issue history. See the change notice: https://www.drupal.org/node/3440477 β†’

πŸ› Bug report
Status

Active

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States DanChadwick

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @DanChadwick
  • Pipeline finished with Success
    about 2 months ago
    Total: 157s
    #250815
  • πŸ‡ΊπŸ‡ΈUnited States DanChadwick

    What a mess. This MR and patch update bootstrap to
    a) set the updated class for stick tables (sticky-header),
    b) remove the bootstrap custom javascript related to TableHeader, now removed in D10.3, and because of that,
    c) require D10.3, dropping D9 support.

    In order to make sticky headers work properly, I also had to
    a) disable the responsive table option in the theme settings (incompatible with sticky headers) and
    b) add custom CSS to set the sticky table's thead to an appropriate background color and change the z-index to 90 (500 is incompatible with other bootstrap z-indices).

  • πŸ‡ΊπŸ‡ΈUnited States jwag956 Monterey, ca

    This patch worked for me .. Thanks!

    It should be noted that since this causes an exception it stops further JS parsing - which when using aggregated JS means that many other JS libraries may not be loaded - so the effect may be far removed from tables.

Production build 0.71.5 2024