Problem/Motivation

Drupal core have been replacing jquery usage with vanilla javascript wherever possible. I think could be a good idea for the chart module also to take the same approach and remove the dependency to core/jquery if possible.

Steps to reproduce

Proposed resolution

Replace usage of jquery wherever possible

Remaining tasks

Submit a MR

User interface changes

N/A

API changes

Modules depending on libraries provided by this module will need to add core/jquery as a dependency.

Data model changes

N/A

Issue fork charts-3267238

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

nikathone created an issue. See original summary.

nikathone’s picture

nikathone
Primary language Swahili
Location Ontario
commented
Title: Remove dependency to jquery wherever possible » Replace JQuery with vanilla Javascript in charts
Parent issue: » #3052002: [meta] Replace JQuery with vanilla Javascript in core

nikathone opened merge request !19

nikathone’s picture

nikathone
Primary language Swahili
Location Ontario
commented
Assigned: nikathone » Unassigned
Status: Active » Needs review

I think I covered all Jquery occurrences.

  • andileco committed 2d98c66 on 5.0.x authored by nikathone 
    Issue #3267238 by nikathone: Replace JQuery with vanilla Javascript in...
andileco’s picture

andileco commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.