Contrib.social

Remove JavaScript from Views configuration form now that it is only CSS

Open on Drupal.org →
Created on 1 June 2024, 6 months ago
Updated 15 June 2024, 6 months ago

Problem/Motivation

📌 Make drupal.tableheader only use CSS for sticky table headers Needs review allowed the sticky header in table feature to be done with CSS only instead of JavaScript.

In https://git.drupalcode.org/project/drupal/-/blob/10.3.x/core/modules/vie...

    $form['sticky'] = [
      '#type' => 'checkbox',
      '#title' => $this->t('Enable Drupal style "sticky" table headers (JavaScript)'),
      '#default_value' => !empty($this->options['sticky']),
      '#description' => $this->t('(Sticky header effects will not be active for preview below, only on live output.)'),
    ];

The checkbox title still mentions JavaScript.

Proposed resolution

The (JavaScript) in the checkbox title should be removed.

Remaining tasks

- do the change in MR
- review
- approval

User interface changes

A checkbox title will be updated.

Before: Enable Drupal style "sticky" table headers (JavaScript)

After: Enable Drupal style "sticky" table headers

API changes

None

Data model changes

None

Release notes snippet

Needed?

📌 Task
Status

Fixed

Version

10.3 ✨

Component
UI text  →

Last updated about 1 month ago

No maintainer
Created by

🇫🇷France Grimreaper France 🇫🇷

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

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024