Feature #31887

Update jQuery UI to 1.12.1

Added by Marius BALTEANU 10 months ago. Updated 19 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:UI
Target version:4.2.0
Resolution:Fixed

Description

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

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

datepicker.png (49 KB) Marius BALTEANU, 2020-05-02 21:41

overlay.png (325 KB) Marius BALTEANU, 2020-05-02 21:41

autocomplete.png (48 KB) Marius BALTEANU, 2020-05-02 21:41

0001-Update-JQuery-UI-to-1.12.1.patch.zip (281 KB) Marius BALTEANU, 2020-05-02 21:49

ScreenShot-2020-05-14-11.21.34.png (55.3 KB) Mizuki ISHIKAWA, 2020-05-14 04:35

0002-Fix-JS-languages-menu-style.patch Magnifier (1.74 KB) Marius BALTEANU, 2020-05-16 14:48


Related issues

Blocked by Redmine - Defect #31870: Remove deprecated .zIndex() method Closed
Blocks Redmine - Defect #32125: Issues autocomplete may not find issues with a subject lo... Closed

Associated revisions

Revision 19769
Added by Go MAEDA 29 days ago

Update jQuery UI to 1.12.1 (#31887).

Patch by Marius BALTEANU.

Revision 19776
Added by Go MAEDA 23 days ago

Reverts r19769 (#31887).

The change broke the Highlighted code button in the Wiki toolbar.

Revision 19783
Added by Go MAEDA 19 days ago

Update jQuery UI to 1.12.1 (#31887).

Patch by Marius BALTEANU.

History

#1 Updated by Marius BALTEANU 10 months ago

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

#2 Updated by Marius BALTEANU 10 months ago

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

#3 Updated by Marius BALTEANU 10 months ago

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

#4 Updated by Sho HASHIMOTO 10 months ago

+1!!

#5 Updated by Sho HASHIMOTO 10 months ago

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

#6 Updated by Marius BALTEANU about 1 month 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.

#7 Updated by Marius BALTEANU about 1 month ago

Some screenshots.

#8 Updated by Marius BALTEANU about 1 month ago

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

#10 Updated by Marius BALTEANU about 1 month 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.

#11 Updated by Go MAEDA about 1 month 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.

#12 Updated by Go MAEDA about 1 month ago

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

Setting the target version to 4.2.0.

#13 Updated by Go MAEDA 30 days 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.

#14 Updated by Marius BALTEANU 29 days 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.

#15 Updated by Go MAEDA 29 days ago

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

Committed the patch. Thank you.

#16 Updated by Mizuki ISHIKAWA 24 days 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.

#17 Updated by Go MAEDA 24 days ago

  • Status changed from Closed to Reopened

#18 Updated by Go MAEDA 23 days ago

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

#19 Updated by Go MAEDA 23 days 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.

#20 Updated by Marius BALTEANU 22 days ago

  • Assignee changed from Go MAEDA to Marius BALTEANU

#21 Updated by Marius BALTEANU 21 days 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.

#22 Updated by Go MAEDA 19 days 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.

Also available in: Atom PDF