From c0085921c342bab5444c23f90316f0b45b18d7c7 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Mon, 8 May 2023 01:49:22 +0800 Subject: [PATCH 1/6] fix --- templates/repo/issue/search.tmpl | 8 +-- templates/user/dashboard/issues.tmpl | 4 +- web_src/js/features/common-issue-list.js | 70 +++++++++++++++++++ web_src/js/features/common-issue-list.test.js | 17 +++++ web_src/js/features/repo-issue.js | 29 -------- web_src/js/index.js | 5 +- web_src/js/utils/dom.js | 7 ++ 7 files changed, 103 insertions(+), 37 deletions(-) create mode 100644 web_src/js/features/common-issue-list.js create mode 100644 web_src/js/features/common-issue-list.test.js diff --git a/templates/repo/issue/search.tmpl b/templates/repo/issue/search.tmpl index 3e7bf1d9ca1c..f24c2dbb7e0a 100644 --- a/templates/repo/issue/search.tmpl +++ b/templates/repo/issue/search.tmpl @@ -8,9 +8,9 @@ - - + {{if .PageIsIssueList}} + + {{end}} + diff --git a/templates/user/dashboard/issues.tmpl b/templates/user/dashboard/issues.tmpl index bf13cf993b6a..a0b02887b018 100644 --- a/templates/user/dashboard/issues.tmpl +++ b/templates/user/dashboard/issues.tmpl @@ -78,8 +78,8 @@ - - + + diff --git a/web_src/js/features/common-issue-list.js b/web_src/js/features/common-issue-list.js new file mode 100644 index 000000000000..2738993e7ab6 --- /dev/null +++ b/web_src/js/features/common-issue-list.js @@ -0,0 +1,70 @@ +import $ from 'jquery'; +import {isElemHidden, toggleElem} from '../utils/dom.js'; +const {appSubUrl} = window.config; + +const reIssueIndex = /^(\d+)$/; // eg: "123" +const reIssueSharpIndex = /^#(\d+)$/; // eg: "#123" +const reIssueOwnerRepoIndex = /^([-.\w]+)\/([-.\w]+)#(\d+)$/; // eg: "{owner}/{repo}#{index}" + +// if the searchText can be parsed to an "issue goto link", return the link, otherwise return empty string +export function parseIssueListQuickGotoLink (repoLink, searchText) { + searchText = searchText.trim(); + let targetUrl = ''; + if (repoLink) { + // try to parse it in current repo + if (reIssueIndex.test(searchText)) { + targetUrl = `${repoLink}/issues/${searchText}`; + } else if (reIssueSharpIndex.test(searchText)) { + targetUrl = `${repoLink}/issues/${searchText.substr(1)}`; + } + } else { + // try to parse it for a global search (eg: "owner/repo#123") + const matchIssueOwnerRepoIndex = searchText.match(reIssueOwnerRepoIndex); + if (matchIssueOwnerRepoIndex) { + const [_, owner, repo, index] = matchIssueOwnerRepoIndex; + targetUrl = `${appSubUrl}/${owner}/${repo}/issues/${index}`; + } + } + return targetUrl; +} + +export function initCommonIssueListQuickGoto() { + const $goto = $('#issue-list-quick-goto'); + if (!$goto.length) return; + + const $form = $goto.closest('form'); + const $input = $form.find('input[name=q]'); + const repoLink = $goto.attr('data-repo-link'); + + $form.on('submit', (e) => { + // if there is no goto button, or the form is submitted by non-quick-goto elements, submit the form directly + let doQuickGoto = !isElemHidden($goto); + const submitter = e.originalEvent.submitter; + if (submitter !== $form[0] && submitter !== $input[0] && submitter !== $goto[0]) doQuickGoto = false; + if (!doQuickGoto) return; + + // if there is a goto button, use its link + e.preventDefault(); + window.location.href = $goto.attr('data-issue-goto-link'); + }); + + const onInput = async () => { + const searchText = $input.val(); + + // try to check whether the parsed goto link is valid + let targetUrl = parseIssueListQuickGotoLink(repoLink, searchText); + if (targetUrl) { + const res = await fetch(`${targetUrl}/info`); + if (res.status !== 200) targetUrl = ''; + } + + // if the input value has changed, then ignore the result + if ($input.val() !== searchText) return; + + toggleElem($goto, Boolean(targetUrl)); + $goto.attr('data-issue-goto-link', targetUrl); + }; + + $input.on('input', onInput); + const _ = onInput(); +} diff --git a/web_src/js/features/common-issue-list.test.js b/web_src/js/features/common-issue-list.test.js new file mode 100644 index 000000000000..271acc718aab --- /dev/null +++ b/web_src/js/features/common-issue-list.test.js @@ -0,0 +1,17 @@ +import {test, expect} from 'vitest'; +import {parseIssueListQuickGotoLink} from './common-issue-list.js'; + +test('parseIssueListQuickGotoLink', () => { + expect(parseIssueListQuickGotoLink('/link', '')).toEqual(''); + expect(parseIssueListQuickGotoLink('/link', 'abc')).toEqual(''); + expect(parseIssueListQuickGotoLink('/link', '123')).toEqual('/link/issues/123'); + expect(parseIssueListQuickGotoLink('/link', '#123')).toEqual('/link/issues/123'); + expect(parseIssueListQuickGotoLink('/link', 'owner/repo#123')).toEqual(''); + + expect(parseIssueListQuickGotoLink('', '')).toEqual(''); + expect(parseIssueListQuickGotoLink('', 'abc')).toEqual(''); + expect(parseIssueListQuickGotoLink('', '123')).toEqual(''); + expect(parseIssueListQuickGotoLink('', '#123')).toEqual(''); + expect(parseIssueListQuickGotoLink('', 'owner/repo#')).toEqual(''); + expect(parseIssueListQuickGotoLink('', 'owner/repo#123')).toEqual('/owner/repo/issues/123'); +}); diff --git a/web_src/js/features/repo-issue.js b/web_src/js/features/repo-issue.js index 7e1249ed2f68..265eceab59db 100644 --- a/web_src/js/features/repo-issue.js +++ b/web_src/js/features/repo-issue.js @@ -4,7 +4,6 @@ import {showTemporaryTooltip, createTippy} from '../modules/tippy.js'; import {hideElem, showElem, toggleElem} from '../utils/dom.js'; import {setFileFolding} from './file-fold.js'; import {getComboMarkdownEditor, initComboMarkdownEditor} from './comp/ComboMarkdownEditor.js'; -import {parseIssueHref} from '../utils.js'; const {appSubUrl, csrfToken} = window.config; @@ -637,31 +636,3 @@ export function initRepoIssueBranchSelect() { }; $('#branch-select > .item').on('click', changeBranchSelect); } - -export function initRepoIssueGotoID() { - const issueidre = /^(?:\w+\/\w+#\d+|#\d+|\d+)$/; - const isGlobalIssuesArea = $('.repo.name.item').length > 0; // for global issues area or repository issues area - $('form.list-header-search').on('submit', (e) => { - const qval = e.target.q.value; - const aElm = document.activeElement; - if (!$('#hashtag-button').length || aElm.id === 'search-button' || (aElm.name === 'q' && !qval.includes('#')) || (isGlobalIssuesArea && !qval.includes('/')) || !issueidre.test(qval)) return; - const pathname = window.location.pathname; - let gotoUrl = qval.includes('/') ? `${qval.replace('#', '/issues/')}` : `${pathname}/${qval.replace('#', '')}`; - if (appSubUrl.length) { - gotoUrl = qval.includes('/') ? `/${appSubUrl}/${qval.replace('#', '/issues/')}` : `/${appSubUrl}/${pathname}/${qval.replace('#', '')}`; - } - const {owner, repo, type, index} = parseIssueHref(gotoUrl); - if (owner && repo && type && index) { - e.preventDefault(); - window.location.href = gotoUrl; - } - }); - $('form.list-header-search input[name=q]').on('input', (e) => { - const qval = e.target.value; - if (isGlobalIssuesArea && qval.includes('/') && issueidre.test(qval) || !isGlobalIssuesArea && issueidre.test(qval)) { - showElem($('#hashtag-button')); - } else { - hideElem($('#hashtag-button')); - } - }); -} diff --git a/web_src/js/index.js b/web_src/js/index.js index 10e2ecd28919..a837375b4ec5 100644 --- a/web_src/js/index.js +++ b/web_src/js/index.js @@ -30,7 +30,7 @@ import { initRepoIssueWipTitle, initRepoPullRequestMergeInstruction, initRepoPullRequestAllowMaintainerEdit, - initRepoPullRequestReview, initRepoIssueSidebarList, initRepoIssueGotoID + initRepoPullRequestReview, initRepoIssueSidebarList } from './features/repo-issue.js'; import { initRepoEllipsisButton, @@ -81,6 +81,7 @@ import {initGlobalTooltips} from './modules/tippy.js'; import {initGiteaFomantic} from './modules/fomantic.js'; import {onDomReady} from './utils/dom.js'; import {initRepoIssueList} from './features/repo-issue-list.js'; +import {initCommonIssueListQuickGoto} from './features/common-issue-list.js'; // Init Gitea's Fomantic settings initGiteaFomantic(); @@ -98,6 +99,7 @@ onDomReady(() => { initGlobalLinkActions(); initCommonOrganization(); + initCommonIssueListQuickGoto(); initCompSearchUserBox(); initCompWebHookEditor(); @@ -175,5 +177,4 @@ onDomReady(() => { initUserAuthWebAuthnRegister(); initUserSettings(); initRepoDiffView(); - initRepoIssueGotoID(); }); diff --git a/web_src/js/utils/dom.js b/web_src/js/utils/dom.js index 6a9ee56eebec..fdab1bf637f7 100644 --- a/web_src/js/utils/dom.js +++ b/web_src/js/utils/dom.js @@ -42,6 +42,13 @@ export function toggleElem(el, force) { elementsCall(el, toggleShown, force); } +export function isElemHidden(el) { + const res = []; + elementsCall(el, (e) => res.push(e.classList.contains('gt-hidden'))); + if (res.length > 1) throw new Error(`the hidden check doesn't work for multiple elements`); + return res[0]; +} + export function onDomReady(cb) { if (document.readyState === 'loading') { document.addEventListener('DOMContentLoaded', cb); From c5c3386f87dbefa4f5cd26ff9136e4a4868ce62e Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 9 May 2023 09:41:42 +0800 Subject: [PATCH 2/6] Update web_src/js/utils/dom.js Co-authored-by: silverwind --- web_src/js/utils/dom.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web_src/js/utils/dom.js b/web_src/js/utils/dom.js index fdab1bf637f7..e8508f94aa78 100644 --- a/web_src/js/utils/dom.js +++ b/web_src/js/utils/dom.js @@ -45,7 +45,7 @@ export function toggleElem(el, force) { export function isElemHidden(el) { const res = []; elementsCall(el, (e) => res.push(e.classList.contains('gt-hidden'))); - if (res.length > 1) throw new Error(`the hidden check doesn't work for multiple elements`); + if (res.length > 1) throw new Error(`isElemHidden doesn't work for multiple elements`); return res[0]; } From 16777fd0b315dedd87a11d6c43fece48fc473f59 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 9 May 2023 09:41:50 +0800 Subject: [PATCH 3/6] Update web_src/js/features/common-issue-list.js Co-authored-by: silverwind --- web_src/js/features/common-issue-list.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web_src/js/features/common-issue-list.js b/web_src/js/features/common-issue-list.js index 2738993e7ab6..172c8d1e3e23 100644 --- a/web_src/js/features/common-issue-list.js +++ b/web_src/js/features/common-issue-list.js @@ -66,5 +66,5 @@ export function initCommonIssueListQuickGoto() { }; $input.on('input', onInput); - const _ = onInput(); + onInput(); } From b5371f8151a6836dcadb817b8b729b2e14ec842b Mon Sep 17 00:00:00 2001 From: silverwind Date: Tue, 9 May 2023 21:30:56 +0200 Subject: [PATCH 4/6] Update web_src/js/features/common-issue-list.js --- web_src/js/features/common-issue-list.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web_src/js/features/common-issue-list.js b/web_src/js/features/common-issue-list.js index 172c8d1e3e23..3988dbd300b2 100644 --- a/web_src/js/features/common-issue-list.js +++ b/web_src/js/features/common-issue-list.js @@ -7,7 +7,7 @@ const reIssueSharpIndex = /^#(\d+)$/; // eg: "#123" const reIssueOwnerRepoIndex = /^([-.\w]+)\/([-.\w]+)#(\d+)$/; // eg: "{owner}/{repo}#{index}" // if the searchText can be parsed to an "issue goto link", return the link, otherwise return empty string -export function parseIssueListQuickGotoLink (repoLink, searchText) { +export function parseIssueListQuickGotoLink(repoLink, searchText) { searchText = searchText.trim(); let targetUrl = ''; if (repoLink) { From 30f00ffa8a5610064775e216b34cea2a7b19b00b Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Wed, 10 May 2023 10:34:39 +0800 Subject: [PATCH 5/6] debounce --- web_src/js/features/codeeditor.js | 4 ++-- web_src/js/features/common-issue-list.js | 6 +++--- web_src/js/utils/dom.js | 6 ++++++ 3 files changed, 11 insertions(+), 5 deletions(-) diff --git a/web_src/js/features/codeeditor.js b/web_src/js/features/codeeditor.js index 3a925e08c1e0..76eeaa9898d4 100644 --- a/web_src/js/features/codeeditor.js +++ b/web_src/js/features/codeeditor.js @@ -1,5 +1,5 @@ import {basename, extname, isObject, isDarkTheme} from '../utils.js'; -import {debounce} from 'throttle-debounce'; +import {onInputDebounce} from '../utils/dom.js'; const languagesByFilename = {}; const languagesByExt = {}; @@ -164,7 +164,7 @@ export async function createCodeEditor(textarea, filenameInput) { ...getEditorConfigOptions(editorConfig), }); - filenameInput.addEventListener('input', debounce(500, () => { + filenameInput.addEventListener('input', onInputDebounce(() => { const filename = filenameInput.value; const previewable = previewableExts.has(extname(filename)); togglePreviewDisplay(previewable); diff --git a/web_src/js/features/common-issue-list.js b/web_src/js/features/common-issue-list.js index 3988dbd300b2..66069013e4a9 100644 --- a/web_src/js/features/common-issue-list.js +++ b/web_src/js/features/common-issue-list.js @@ -1,5 +1,5 @@ import $ from 'jquery'; -import {isElemHidden, toggleElem} from '../utils/dom.js'; +import {isElemHidden, onInputDebounce, toggleElem} from '../utils/dom.js'; const {appSubUrl} = window.config; const reIssueIndex = /^(\d+)$/; // eg: "123" @@ -65,6 +65,6 @@ export function initCommonIssueListQuickGoto() { $goto.attr('data-issue-goto-link', targetUrl); }; - $input.on('input', onInput); - onInput(); + $input.on('input', onInputDebounce(onInput)); + const _ = onInput(); } diff --git a/web_src/js/utils/dom.js b/web_src/js/utils/dom.js index e8508f94aa78..cdc52b1a7425 100644 --- a/web_src/js/utils/dom.js +++ b/web_src/js/utils/dom.js @@ -1,3 +1,5 @@ +import {debounce} from 'throttle-debounce'; + function elementsCall(el, func, ...args) { if (typeof el === 'string' || el instanceof String) { el = document.querySelectorAll(el); @@ -177,3 +179,7 @@ export function autosize(textarea, {viewportMarginBottom = 0} = {}) { } }; } + +export function onInputDebounce(fn) { + return debounce(300, fn); +} From 6b69dc50a534c896f4d5d28c7f8fb30da58fffa0 Mon Sep 17 00:00:00 2001 From: silverwind Date: Wed, 10 May 2023 13:52:47 +0200 Subject: [PATCH 6/6] Update web_src/js/features/common-issue-list.js --- web_src/js/features/common-issue-list.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web_src/js/features/common-issue-list.js b/web_src/js/features/common-issue-list.js index 66069013e4a9..ecada11988cf 100644 --- a/web_src/js/features/common-issue-list.js +++ b/web_src/js/features/common-issue-list.js @@ -66,5 +66,5 @@ export function initCommonIssueListQuickGoto() { }; $input.on('input', onInputDebounce(onInput)); - const _ = onInput(); + onInput(); }