プロジェクト

全般

プロフィール

Vote #80231

完了

Remove deprecated .zIndex() method

Admin Redmine さんが約4年前に追加. 約4年前に更新.

ステータス:
Closed
優先度:
通常
担当者:
-
カテゴリ:
Code cleanup/refactoring_30
対象バージョン:
開始日:
2022/05/09
期日:
進捗率:

0%

予定工数:
category_id:
30
version_id:
153
issue_org_id:
31870
author_id:
411315
assigned_to_id:
332
comments:
10
status_id:
5
tracker_id:
1
plus1:
0
affected_version:
closed_on:
affected_version_id:
151
ステータス-->[Closed]

説明

Système : Debian 10
Redmine version : 4.0.1-2
libjs-jquery-ui version : 1.12.1+dfsg-5

.zIndex() method is deprecated from jQuery UI
https://jqueryui.com/upgrade-guide/1.12/#removed-zindex

.zIndex() is used in public/javascript/application.js
https://www.redmine.org/projects/redmine/repository/entry/branches/4.0-stable/public/javascripts/application.js
lines 429, 439

problem occurs when adding member to a project.

proposed correction :

function showModal(id, width, title) {
  var el = $('#'+id).first();
  if (el.length === 0 || el.is(':visible')) {return;}
  if (!title) title = el.find('h3.title').text();
  // moves existing modals behind the transparent background
-  $(".modal").zIndex(99);
+  $(".modal").css('zIndex',99);
  el.dialog({
    width: width,
    modal: true,
    resizable: false,
    dialogClass: 'modal',
    title: title
  }).on('dialogclose', function(){
-    $(".modal").zIndex(101);
+    $(".modal").css('zIndex',101);
  });
  el.find("input[type=text], input[type=submit]").first().focus();
}

journals

Related URLs:
* https://github.com/jquery/jquery-ui/commit/faf87d992455280182edef015c8733313744a739
* https://jqueryui.com/upgrade-guide/1.11/#deprecated-zindex
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Committed the patch. Thank you for your contribution.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Hi, already asked this in Defect #31708, there is a "close" text on the close button, any way to remove it? Appreciate any answer.
--------------------------------------------------------------------------------


related_issues

relates,Closed,31196,Updates jQuery to 2.2.4 and adds jQuery Migrate library
relates,Closed,31708,Not able to add member to a project
blocks,Closed,31887,Update jQuery UI to 1.12.1

Admin Redmine さんが約4年前に更新

  • カテゴリCode cleanup/refactoring_30 にセット
  • 対象バージョン4.0.5_153 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0