Project

General

Profile

Actions

Feature #31887

closed

Update jQuery UI to 1.12.1

Added by Marius BĂLTEANU over 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

We should upgrade jQuery UI to latest version (currently 1.12.1).

Upgrade guide: https://jqueryui.com/upgrade-guide/1.12/


Files

datepicker.png (49 KB) datepicker.png Marius BĂLTEANU, 2020-05-02 21:41
overlay.png (325 KB) overlay.png Marius BĂLTEANU, 2020-05-02 21:41
autocomplete.png (48 KB) autocomplete.png Marius BĂLTEANU, 2020-05-02 21:41
0001-Update-JQuery-UI-to-1.12.1.patch.zip (281 KB) 0001-Update-JQuery-UI-to-1.12.1.patch.zip Marius BĂLTEANU, 2020-05-02 21:49
ScreenShot-2020-05-14-11.21.34.png (55.3 KB) ScreenShot-2020-05-14-11.21.34.png Mizuki ISHIKAWA, 2020-05-14 04:35
0002-Fix-JS-languages-menu-style.patch (1.74 KB) 0002-Fix-JS-languages-menu-style.patch Marius BĂLTEANU, 2020-05-16 14:48

Related issues

Blocked by Redmine - Defect #31870: Remove deprecated .zIndex() methodClosedGo MAEDA

Actions
Blocks Redmine - Defect #32125: Issues autocomplete may not find issues with a subject longer than 60 charactersClosedGo MAEDA

Actions
Actions #1

Updated by Marius BĂLTEANU over 4 years ago

  • Related to Defect #31870: Remove deprecated .zIndex() method added
Actions #2

Updated by Marius BĂLTEANU over 4 years ago

  • Related to deleted (Defect #31870: Remove deprecated .zIndex() method)
Actions #3

Updated by Marius BĂLTEANU over 4 years ago

  • Blocked by Defect #31870: Remove deprecated .zIndex() method added
Actions #4

Updated by Sho HASHIMOTO over 4 years ago

+1!!

Actions #5

Updated by Sho HASHIMOTO over 4 years ago

note: jQuery 1.x and 2.x was already EOL. See below.

Actions #6

Updated by Marius BĂLTEANU almost 4 years ago

  • File 0001-Update-JQuery-UI-to-1.12.1.patch.zip added
  • Target version set to Candidate for next major release

Here is a patch that updates jQuery UI to latest version (1.12.1). The patch is zip archive because of the size.

Among all the changes, the new version of jQuery UI brings a new default theme (Smoothness) and Font size changes which do not require anymore some overrides made in Redmine theme.

Beside the update, the patch removes those overrides, adds a new overlay background and makes the autocomplete to have the same look and feel from inline auto complete, context menu or quick project.

Also, starting with this version, the library is divided in multiple components and we can decrease the size of it if we drop those components not used by Redmine, but the change can affect plugin developers.

Actions #7

Updated by Marius BĂLTEANU almost 4 years ago

Some screenshots.

Actions #8

Updated by Marius BĂLTEANU almost 4 years ago

  • File deleted (0001-Update-JQuery-UI-to-1.12.1.patch.zip)
Actions #10

Updated by Marius BĂLTEANU almost 4 years ago

Sho HASHIMOTO wrote:

note: jQuery 1.x and 2.x was already EOL. See below.

Here we track jQuery UI update and in #33383 we track jQuery.

Actions #11

Updated by Go MAEDA almost 4 years ago

Marius BALTEANU wrote:

Here is a patch that updates jQuery UI to latest version (1.12.1). The patch is zip archive because of the size.

Thank you for working on this. Features that depend on JQuery UI such as dialogs and auto-completes seem to be working fine.

Actions #12

Updated by Go MAEDA almost 4 years ago

  • Target version changed from Candidate for next major release to 4.2.0

Setting the target version to 4.2.0.

Actions #13

Updated by Go MAEDA almost 4 years ago

Marius BALTEANU wrote:

Also, starting with this version, the library is divided in multiple components and we can decrease the size of it if we drop those components not used by Redmine, but the change can affect plugin developers.

Dropping some components is a big change, so I think it is better to put off the change until Redmine 5.0.

Actions #14

Updated by Marius BĂLTEANU almost 4 years ago

Go MAEDA wrote:

Marius BALTEANU wrote:

Also, starting with this version, the library is divided in multiple components and we can decrease the size of it if we drop those components not used by Redmine, but the change can affect plugin developers.

Dropping some components is a big change, so I think it is better to put off the change until Redmine 5.0.

Agree.

Actions #15

Updated by Go MAEDA almost 4 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch. Thank you.

Actions #16

Updated by Mizuki ISHIKAWA almost 4 years ago

The style of the highlighted code listings on the Wiki toolbar has changed since this change was committed.
The background color of the li element does not change when hovering.

Actions #17

Updated by Go MAEDA almost 4 years ago

  • Status changed from Closed to Reopened
Actions #18

Updated by Go MAEDA almost 4 years ago

  • Blocks Defect #32125: Issues autocomplete may not find issues with a subject longer than 60 characters added
Actions #19

Updated by Go MAEDA almost 4 years ago

Mizuki ISHIKAWA wrote:

The style of the highlighted code listings on the Wiki toolbar has changed since this change was committed.
The background color of the li element does not change when hovering.

Reverted r19769 in r19776.

Actions #20

Updated by Marius BĂLTEANU almost 4 years ago

  • Assignee changed from Go MAEDA to Marius BĂLTEANU
Actions #21

Updated by Marius BĂLTEANU almost 4 years ago

Here is the patch to fix the reported issue, thanks for pointing this out. I didn't know that the JS Toolbar depends on the menu component from jQuery UI (issue cause). We should drop this dependency in the future.

@Go Maeda, I don't think that it's necessary to revert the changes, we're on trunk and for the patches that I work on, I will provide fixes as soon as I can.

Actions #22

Updated by Go MAEDA almost 4 years ago

  • Status changed from Reopened to Closed

Committed the patch again with 0002-Fix-JS-languages-menu-style.patch. Thank you.

@Go Maeda, I don't think that it's necessary to revert the changes, we're on trunk and for the patches that I work on, I will provide fixes as soon as I can.

I thought it was a rather serious issue. And I didn't expect the issue is fixed so quickly. But now I think that reverting r19769 was too early.

Actions

Also available in: Atom PDF