Project

General

Profile

Actions

Patch #43259

open

Convert jstoolbar to ES modules and class syntax.

Added by Takashi Kato 3 days ago. Updated 2 days ago.

Status:
New
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

This patch is based on a branch from #43165, with minor modifications.

  • Converting the entire library to ES modules eliminates the risk of variable name collisions.
  • Refactored jstoolbar from the old prototype-based syntax to a more readable class syntax.
  • Switched DOM element creation from JavaScript to ERB, which makes preparing and cloning DOM elements easier by using the <template> element.
  • Moved all jstoolbar locales to config/locales to simplify i18n. * Added the missing tasklist locale for both ja and en.
  • Integrated the list-autofill controller into jstoolbar as both controllers are used in the same places.

Applying this patch along with #42521 enables the implementation of #40310.

This patch can be applied to r24014


Files

Actions #1

Updated by Marius BĂLTEANU 2 days ago

  • Assignee set to Marius BĂLTEANU
  • Target version set to 7.0.0

I'm assigning this 7.0.0 in order to review the changes.

Actions

Also available in: Atom PDF