From 249e67672aea8a5a0ed73bedc655f9dcd37437ea Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Fri, 15 Nov 2024 02:48:41 +0800 Subject: [PATCH 01/16] Remove jQuery import from some files (#32512) Many files do not directly depend on jQuery now. To clarify the usage: use `fomanticQuery` to operate Fomantic components. Then developers could focus on removing the remaining jQuery usages by searching `import $` globally. 21 files now: ``` ./components/RepoBranchTagSelector.vue:3:import $ from 'jquery'; ./features/admin/common.ts:1:import $ from 'jquery'; ./features/admin/emails.ts:1:import $ from 'jquery'; ./features/common-button.ts:1:import $ from 'jquery'; ./features/comp/ComboMarkdownEditor.ts:3:import $ from 'jquery'; (I am working on it, there will be a new PR) ./features/comp/LabelEdit.ts:1:import $ from 'jquery'; ./features/notification.ts:1:import $ from 'jquery'; ./features/org-team.ts:1:import $ from 'jquery'; ./features/repo-code.ts:1:import $ from 'jquery'; ./features/repo-common.ts:1:import $ from 'jquery'; ./features/repo-diff.ts:1:import $ from 'jquery'; ./features/repo-editor.ts:1:import $ from 'jquery'; ./features/repo-issue-content.ts:1:import $ from 'jquery'; ./features/repo-issue-list.ts:1:import $ from 'jquery'; ./features/repo-issue-sidebar.ts:1:import $ from 'jquery'; ./features/repo-issue.ts:1:import $ from 'jquery'; ./features/repo-legacy.ts:1:import $ from 'jquery'; ./features/repo-new.ts:1:import $ from 'jquery'; ./features/repo-projects.ts:1:import $ from 'jquery'; ./features/repo-settings.ts:1:import $ from 'jquery'; ./features/repo-template.ts:1:import $ from 'jquery'; ``` --- web_src/js/components/DashboardRepoList.vue | 4 ++-- web_src/js/components/RepoContributors.vue | 4 ++-- web_src/js/features/citation.ts | 4 ++-- web_src/js/features/common-form.ts | 7 +++---- web_src/js/features/comp/ConfirmModal.ts | 4 ++-- web_src/js/features/comp/ReactionSelector.ts | 4 ++-- web_src/js/features/comp/SearchUserBox.ts | 4 ++-- web_src/js/features/repo-branch.ts | 4 ++-- web_src/js/features/repo-graph.ts | 6 +++--- web_src/js/features/repo-home.ts | 6 +++--- web_src/js/vendor/jquery.are-you-sure.ts | 4 ++++ 11 files changed, 27 insertions(+), 24 deletions(-) diff --git a/web_src/js/components/DashboardRepoList.vue b/web_src/js/components/DashboardRepoList.vue index 986fcc1181..a6a8ccd2d1 100644 --- a/web_src/js/components/DashboardRepoList.vue +++ b/web_src/js/components/DashboardRepoList.vue @@ -1,8 +1,8 @@