From 1d6b6cf020e6fac3a80c431e45f8db0d93d4d5a9 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 01:21:57 +0800 Subject: [PATCH 01/11] fix --- models/issues/issue_project.go | 30 ------- models/project/column.go | 38 ++++---- routers/web/repo/issue_new.go | 10 +-- routers/web/repo/issue_page_meta.go | 42 ++++++++- services/projects/issue.go | 5 +- templates/repo/issue/new_form.tmpl | 2 +- .../repo/issue/sidebar/project_list.tmpl | 2 +- .../repo/issue/view_content/sidebar.tmpl | 2 +- .../features/repo-issue-sidebar-combolist.ts | 89 +++++++++++++------ web_src/js/features/repo-issue-sidebar.ts | 25 +++--- web_src/js/features/repo-issue.ts | 4 +- 11 files changed, 146 insertions(+), 103 deletions(-) diff --git a/models/issues/issue_project.go b/models/issues/issue_project.go index 01852447834c7..3bb09363019f9 100644 --- a/models/issues/issue_project.go +++ b/models/issues/issue_project.go @@ -64,36 +64,6 @@ func LoadProjectIssueColumnMap(ctx context.Context, projectID, defaultColumnID i return result, nil } -// LoadIssuesFromColumn load issues assigned to this column -func LoadIssuesFromColumn(ctx context.Context, b *project_model.Column, opts *IssuesOptions) (IssueList, error) { - issueList, err := Issues(ctx, opts.Copy(func(o *IssuesOptions) { - o.ProjectColumnID = b.ID - o.ProjectID = b.ProjectID - o.SortType = "project-column-sorting" - })) - if err != nil { - return nil, err - } - - if b.Default { - issues, err := Issues(ctx, opts.Copy(func(o *IssuesOptions) { - o.ProjectColumnID = db.NoConditionID - o.ProjectID = b.ProjectID - o.SortType = "project-column-sorting" - })) - if err != nil { - return nil, err - } - issueList = append(issueList, issues...) - } - - if err := issueList.LoadComments(ctx); err != nil { - return nil, err - } - - return issueList, nil -} - // IssueAssignOrRemoveProject changes the project associated with an issue // If newProjectID is 0, the issue is removed from the project func IssueAssignOrRemoveProject(ctx context.Context, issue *Issue, doer *user_model.User, newProjectID, newColumnID int64) error { diff --git a/models/project/column.go b/models/project/column.go index 79f6dfe911448..659f1868a81bc 100644 --- a/models/project/column.go +++ b/models/project/column.go @@ -257,9 +257,11 @@ func (p *Project) GetColumns(ctx context.Context) (ColumnList, error) { return columns, nil } -// getDefaultColumn return default column and ensure only one exists -func (p *Project) getDefaultColumn(ctx context.Context) (*Column, error) { +// GetDefaultColumnWithFallback return default column if one exists, otherwise return the first column by sorting and set it as default column +func (p *Project) GetDefaultColumnWithFallback(ctx context.Context) (*Column, error) { var column Column + + // try to find a column "default=true" has, err := db.GetEngine(ctx). Where("project_id=? AND `default` = ?", p.ID, true). Desc("id").Get(&column) @@ -270,14 +272,29 @@ func (p *Project) getDefaultColumn(ctx context.Context) (*Column, error) { if has { return &column, nil } + + // try to first the first column by sorting + has, err = db.GetEngine(ctx).Where("project_id=?", p.ID).OrderBy("sorting, id").Get(&column) + if err != nil { + return nil, err + } + if has { + column.Default = true + if _, err := db.GetEngine(ctx).ID(column.ID).Cols("`default`").Update(&column); err != nil { + return nil, err + } + return &column, nil + } + return nil, ErrProjectColumnNotExist{ColumnID: 0} } // MustDefaultColumn returns the default column for a project. // If one exists, it is returned // If none exists, the first column will be elevated to the default column of this project +// If there is no column, it creates a default column and returns it func (p *Project) MustDefaultColumn(ctx context.Context) (*Column, error) { - c, err := p.getDefaultColumn(ctx) + c, err := p.GetDefaultColumnWithFallback(ctx) if err != nil && !IsErrProjectColumnNotExist(err) { return nil, err } @@ -285,21 +302,8 @@ func (p *Project) MustDefaultColumn(ctx context.Context) (*Column, error) { return c, nil } - var column Column - has, err := db.GetEngine(ctx).Where("project_id=?", p.ID).OrderBy("sorting, id").Get(&column) - if err != nil { - return nil, err - } - if has { - column.Default = true - if _, err := db.GetEngine(ctx).ID(column.ID).Cols("`default`").Update(&column); err != nil { - return nil, err - } - return &column, nil - } - // create a default column if none is found - column = Column{ + column := Column{ ProjectID: p.ID, Default: true, Title: "Uncategorized", diff --git a/routers/web/repo/issue_new.go b/routers/web/repo/issue_new.go index 1393f62fa531a..98fb842ddf7ff 100644 --- a/routers/web/repo/issue_new.go +++ b/routers/web/repo/issue_new.go @@ -121,11 +121,9 @@ func NewIssue(ctx *context.Context) { } pageMetaData.MilestonesData.SelectedMilestoneID = ctx.FormInt64("milestone") - pageMetaData.ProjectsData.SelectedProjectID = ctx.FormInt64("project") - if pageMetaData.ProjectsData.SelectedProjectID > 0 { - if len(ctx.Req.URL.Query().Get("project")) > 0 { - ctx.Data["redirect_after_creation"] = "project" - } + pageMetaData.ProjectsData.SelectedProjectIDs, _ = base.StringsToInt64s(strings.Split(ctx.FormString("project"), ",")) + if len(pageMetaData.ProjectsData.SelectedProjectIDs) == 1 { + ctx.Data["redirect_after_creation"] = "project" } tags, err := repo_model.GetTagNamesByRepoID(ctx, ctx.Repo.Repository.ID) @@ -273,7 +271,7 @@ func ValidateRepoMetasForNewIssue(ctx *context.Context, form forms.CreateIssueFo ctx.NotFound(nil) return ret } - pageMetaData.ProjectsData.SelectedProjectID = form.ProjectID + pageMetaData.ProjectsData.SelectedProjectIDs = util.Iif(form.ProjectID > 0, []int64{form.ProjectID}, nil) // prepare assignees candidateAssignees := toSet(pageMetaData.AssigneesData.CandidateAssignees, func(user *user_model.User) int64 { return user.ID }) diff --git a/routers/web/repo/issue_page_meta.go b/routers/web/repo/issue_page_meta.go index 719b485bc5055..639333ab42599 100644 --- a/routers/web/repo/issue_page_meta.go +++ b/routers/web/repo/issue_page_meta.go @@ -34,9 +34,14 @@ type issueSidebarAssigneesData struct { } type issueSidebarProjectsData struct { - SelectedProjectID int64 - OpenProjects []*project_model.Project - ClosedProjects []*project_model.Project + SelectedProjectIDs []int64 // TODO: support multiple projects in the future + + // the "selected" fields are only valid when len(SelectedProjectIDs)==1 + SelectedProjectColumns []*project_model.Column + SelectedProjectColumn *project_model.Column + + OpenProjects []*project_model.Project + ClosedProjects []*project_model.Project } type IssuePageMetaData struct { @@ -92,6 +97,11 @@ func retrieveRepoIssueMetaData(ctx *context.Context, repo *repo_model.Repository return data } + data.retrieveProjectData(ctx) + if ctx.Written() { + return data + } + // TODO: the issue/pull permissions are quite complex and unclear // A reader could create an issue/PR with setting some meta (eg: assignees from issue template, reviewers, target branch) // A reader(creator) could update some meta (eg: target branch), but can't change assignees anymore. @@ -158,9 +168,33 @@ func (d *IssuePageMetaData) retrieveAssigneesData(ctx *context.Context) { ctx.Data["Assignees"] = d.AssigneesData.CandidateAssignees } +func (d *IssuePageMetaData) retrieveProjectData(ctx *context.Context) { + if d.Issue == nil || d.Issue.Project == nil { + return + } + d.ProjectsData.SelectedProjectIDs = []int64{d.Issue.Project.ID} + columns, err := d.Issue.Project.GetColumns(ctx) + if err != nil { + ctx.ServerError("GetProjectColumns", err) + return + } + d.ProjectsData.SelectedProjectColumns = columns + columnID, err := d.Issue.ProjectColumnID(ctx) + if err != nil { + ctx.ServerError("ProjectColumnID", err) + return + } + for _, col := range columns { + if col.ID == columnID { + d.ProjectsData.SelectedProjectColumn = col + break + } + } +} + func (d *IssuePageMetaData) retrieveProjectsDataForIssueWriter(ctx *context.Context) { if d.Issue != nil && d.Issue.Project != nil { - d.ProjectsData.SelectedProjectID = d.Issue.Project.ID + d.ProjectsData.SelectedProjectIDs = []int64{d.Issue.Project.ID} } d.ProjectsData.OpenProjects, d.ProjectsData.ClosedProjects = retrieveProjectsInternal(ctx, ctx.Repo.Repository) } diff --git a/services/projects/issue.go b/services/projects/issue.go index 590fe960d5329..f25323281a67a 100644 --- a/services/projects/issue.go +++ b/services/projects/issue.go @@ -12,6 +12,7 @@ import ( project_model "code.gitea.io/gitea/models/project" user_model "code.gitea.io/gitea/models/user" "code.gitea.io/gitea/modules/optional" + "code.gitea.io/gitea/modules/util" ) // MoveIssuesOnProjectColumn moves or keeps issues in a column and sorts them inside that column @@ -100,9 +101,9 @@ func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, return nil, err } - defaultColumn, err := project.MustDefaultColumn(ctx) + defaultColumn, err := project.GetDefaultColumnWithFallback(ctx) if err != nil { - return nil, err + return nil, util.Iif(errors.Is(err, util.ErrNotExist), nil, err) } issueColumnMap, err := issues_model.LoadProjectIssueColumnMap(ctx, project.ID, defaultColumn.ID) diff --git a/templates/repo/issue/new_form.tmpl b/templates/repo/issue/new_form.tmpl index 5475224750e5d..7d669fe4a71cf 100644 --- a/templates/repo/issue/new_form.tmpl +++ b/templates/repo/issue/new_form.tmpl @@ -46,7 +46,7 @@ -
+
{{template "repo/issue/branch_selector_field" $}}{{/* TODO: RemoveIssueRef: template "repo/issue/branch_selector_field" $*/}} {{if .PageIsComparePull}} diff --git a/templates/repo/issue/sidebar/project_list.tmpl b/templates/repo/issue/sidebar/project_list.tmpl index be8097fbbcabb..ca3b6bdd7ab34 100644 --- a/templates/repo/issue/sidebar/project_list.tmpl +++ b/templates/repo/issue/sidebar/project_list.tmpl @@ -5,7 +5,7 @@
- +
{{ctx.Locale.Tr "repo.issues.new.projects"}} {{if $pageMeta.CanModifyIssueOrPull}}{{svg "octicon-gear"}}{{end}} diff --git a/templates/repo/issue/view_content/sidebar.tmpl b/templates/repo/issue/view_content/sidebar.tmpl index 987a882be7be5..4dff50849524a 100644 --- a/templates/repo/issue/view_content/sidebar.tmpl +++ b/templates/repo/issue/view_content/sidebar.tmpl @@ -1,4 +1,4 @@ -
+
{{template "repo/issue/branch_selector_field" $}}{{/* TODO: RemoveIssueRef: template "repo/issue/branch_selector_field" $*/}} {{if .Issue.IsPull}} diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index aaff3804c71d2..dcd99a21a9bba 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -1,37 +1,22 @@ import {fomanticQuery} from '../modules/fomantic/base.ts'; -import {POST} from '../modules/fetch.ts'; +import {GET, POST} from '../modules/fetch.ts'; +import {showErrorToast} from '../modules/toast.ts'; import {addDelegatedEventListener, queryElemChildren, queryElems, toggleElem} from '../utils/dom.ts'; - -// if there are draft comments, confirm before reloading, to avoid losing comments -function issueSidebarReloadConfirmDraftComment() { - const commentTextareas = [ - document.querySelector('.edit-content-zone:not(.tw-hidden) textarea'), - document.querySelector('#comment-form textarea'), - ]; - for (const textarea of commentTextareas) { - // Most users won't feel too sad if they lose a comment with 10 chars, they can re-type these in seconds. - // But if they have typed more (like 50) chars and the comment is lost, they will be very unhappy. - if (textarea && textarea.value.trim().length > 10) { - textarea.parentElement!.scrollIntoView(); - if (!window.confirm('Page will be reloaded, but there are draft comments. Continuing to reload will discard the comments. Continue?')) { - return; - } - break; - } - } - window.location.reload(); -} +import {parseDom} from '../utils.ts'; export class IssueSidebarComboList { updateUrl: string; updateAlgo: string; selectionMode: string; elDropdown: HTMLElement; - elList: HTMLElement; + elList: HTMLElement | null; elComboValue: HTMLInputElement; initialValues: string[]; container: HTMLElement; + elIssueMainContent: HTMLElement; + elIssueSidebar: HTMLElement; + constructor(container: HTMLElement) { this.container = container; this.updateUrl = container.getAttribute('data-update-url')!; @@ -42,6 +27,9 @@ export class IssueSidebarComboList { this.elDropdown = container.querySelector(':scope > .ui.dropdown')!; this.elList = container.querySelector(':scope > .ui.list')!; this.elComboValue = container.querySelector(':scope > .combo-value')!; + + this.elIssueMainContent = document.querySelector('.issue-content-left')!; + this.elIssueSidebar = document.querySelector('.issue-content-right')!; } collectCheckedValues() { @@ -49,6 +37,7 @@ export class IssueSidebarComboList { } updateUiList(changedValues: Array) { + if (!this.elList) return; const elEmptyTip = this.elList.querySelector('.item.empty-list')!; queryElemChildren(this.elList, '.item:not(.empty-list)', (el) => el.remove()); for (const value of changedValues) { @@ -62,22 +51,68 @@ export class IssueSidebarComboList { toggleElem(elEmptyTip, !hasItems); } - async updateToBackend(changedValues: Array) { + async reloadPagePartially() { + const resp = await GET(window.location.href); + if (!resp.ok) throw new Error(`Failed to reload page: ${resp.statusText}`); + const doc = parseDom(await resp.text(), 'text/html'); + + // we can safely replace the whole right part (sidebar) because there are only some dropdowns and lists + const newSidebar = doc.querySelector('.issue-content-right')!; + this.elIssueSidebar.replaceWith(newSidebar); + + // for the main content (left side), at the moment we only support adding new timeline items + const newMainContent = doc.querySelector('.issue-content-left')!; + + // find the last ".timeline-item[id]" in current main content, and insert new items after it, some cases: + // * a new empty issue: "timeline-item comment first", "timeline-item comment form (optional)" + // * issue with timeline events: "timeline-item comment first", "timeline-item event"+id, "timeline-item comment form (optional)" + const timelineItemsWithId = this.elIssueMainContent.querySelectorAll('.timeline-item[id]'); + let lastTimelineItemForInsertion = timelineItemsWithId[timelineItemsWithId.length - 1] ?? this.elIssueMainContent.querySelector('.timeline-item'); + if (!lastTimelineItemForInsertion) return; + + const newTimelineItems = newMainContent.querySelectorAll(`.timeline-item[id]`); + for (const newItem of newTimelineItems) { + const newItemId = newItem.getAttribute('id')!; + if (this.elIssueMainContent.querySelector(`.timeline-item[id="${CSS.escape(newItemId)}"]`)) continue; + lastTimelineItemForInsertion.insertAdjacentElement('afterend', newItem); + lastTimelineItemForInsertion = newItem; + } + } + + async sendRequestToBackend(changedValues: Array): Promise { + let lastResp: Response | null = null; if (this.updateAlgo === 'diff') { for (const value of this.initialValues) { if (!changedValues.includes(value)) { - await POST(this.updateUrl, {data: new URLSearchParams({action: 'detach', id: value})}); + lastResp = await POST(this.updateUrl, {data: new URLSearchParams({action: 'detach', id: value})}); } } for (const value of changedValues) { if (!this.initialValues.includes(value)) { - await POST(this.updateUrl, {data: new URLSearchParams({action: 'attach', id: value})}); + lastResp = await POST(this.updateUrl, {data: new URLSearchParams({action: 'attach', id: value})}); } } } else { - await POST(this.updateUrl, {data: new URLSearchParams({id: changedValues.join(',')})}); + lastResp = await POST(this.updateUrl, {data: new URLSearchParams({id: changedValues.join(',')})}); + } + return lastResp; + } + + async updateToBackend(changedValues: Array) { + this.elIssueSidebar.classList.add('is-loading'); + try { + const resp = await this.sendRequestToBackend(changedValues); + if (!resp) return; // no request sent, no need to reload + if (!resp.ok) { + showErrorToast(`Failed to update to backend: ${resp.statusText}`); + return; + } + await this.reloadPagePartially(); + } catch (e) { + showErrorToast(`Failed to update to backend: ${e}`); + } finally { + this.elIssueSidebar.classList.remove('is-loading'); } - issueSidebarReloadConfirmDraftComment(); } async doUpdate() { diff --git a/web_src/js/features/repo-issue-sidebar.ts b/web_src/js/features/repo-issue-sidebar.ts index 2ff077e6db204..16873af76a6fe 100644 --- a/web_src/js/features/repo-issue-sidebar.ts +++ b/web_src/js/features/repo-issue-sidebar.ts @@ -1,12 +1,12 @@ import {POST} from '../modules/fetch.ts'; import {queryElems, toggleElem} from '../utils/dom.ts'; import {IssueSidebarComboList} from './repo-issue-sidebar-combolist.ts'; +import {registerGlobalInitFunc} from '../modules/observer.ts'; -function initBranchSelector() { +function initRepoIssueBranchSelector(elSidebar: HTMLElement) { // TODO: RemoveIssueRef: see "repo/issue/branch_selector_field.tmpl" - const elSelectBranch = document.querySelector('.ui.dropdown.select-branch.branch-selector-dropdown'); + const elSelectBranch = elSidebar.querySelector('.ui.dropdown.select-branch.branch-selector-dropdown'); if (!elSelectBranch) return; - const urlUpdateIssueRef = elSelectBranch.getAttribute('data-url-update-issueref'); const elBranchMenu = elSelectBranch.querySelector('.reference-list-menu')!; queryElems(elBranchMenu, '.item:not(.no-select)', (el) => el.addEventListener('click', async function (e) { @@ -30,23 +30,24 @@ function initBranchSelector() { })); } -function initRepoIssueDue() { - const form = document.querySelector('.issue-due-form'); +function initRepoIssueDue(elSidebar: HTMLElement) { + const form = elSidebar.querySelector('.issue-due-form'); if (!form) return; const deadline = form.querySelector('input[name=deadline]')!; - document.querySelector('.issue-due-edit')?.addEventListener('click', () => { + elSidebar.querySelector('.issue-due-edit')?.addEventListener('click', () => { toggleElem(form); }); - document.querySelector('.issue-due-remove')?.addEventListener('click', () => { + elSidebar.querySelector('.issue-due-remove')?.addEventListener('click', () => { deadline.value = ''; form.dispatchEvent(new Event('submit', {cancelable: true, bubbles: true})); }); } export function initRepoIssueSidebar() { - initBranchSelector(); - initRepoIssueDue(); - - // init the combo list: a dropdown for selecting items, and a list for showing selected items and related actions - queryElems(document, '.issue-sidebar-combo', (el) => new IssueSidebarComboList(el).init()); + registerGlobalInitFunc('initRepoIssueSidebar', (elSidebar) => { + initRepoIssueBranchSelector(elSidebar); + initRepoIssueDue(elSidebar); + // init the combo list: a dropdown for selecting items, and a list for showing selected items and related actions + queryElems(elSidebar, '.issue-sidebar-combo', (el) => new IssueSidebarComboList(el).init()); + }); } diff --git a/web_src/js/features/repo-issue.ts b/web_src/js/features/repo-issue.ts index 8da28f7f3a299..5cbfac5692990 100644 --- a/web_src/js/features/repo-issue.ts +++ b/web_src/js/features/repo-issue.ts @@ -566,6 +566,8 @@ function initIssueTemplateCommentEditors(commentForm: HTMLFormElement) { } export function initRepoCommentFormAndSidebar() { + initRepoIssueSidebar(); + const commentForm = document.querySelector('.comment.form'); if (!commentForm) return; @@ -576,6 +578,4 @@ export function initRepoCommentFormAndSidebar() { // it's quite unclear about the "comment form" elements, sometimes it's for issue comment, sometimes it's for file editor/uploader message initSingleCommentEditor(commentForm); } - - initRepoIssueSidebar(); } From 3daa9525826ad55f9f7e4b4fd015ea086747a831 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 02:04:55 +0800 Subject: [PATCH 02/11] fix --- models/project/column.go | 2 +- services/projects/issue.go | 7 ++++--- web_src/js/features/repo-issue-sidebar-combolist.ts | 4 +++- 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/models/project/column.go b/models/project/column.go index 659f1868a81bc..977a5b8dfd71b 100644 --- a/models/project/column.go +++ b/models/project/column.go @@ -273,7 +273,7 @@ func (p *Project) GetDefaultColumnWithFallback(ctx context.Context) (*Column, er return &column, nil } - // try to first the first column by sorting + // try to find the first column by sorting has, err = db.GetEngine(ctx).Where("project_id=?", p.ID).OrderBy("sorting, id").Get(&column) if err != nil { return nil, err diff --git a/services/projects/issue.go b/services/projects/issue.go index f25323281a67a..d6b7aa61b3869 100644 --- a/services/projects/issue.go +++ b/services/projects/issue.go @@ -88,7 +88,7 @@ func MoveIssuesOnProjectColumn(ctx context.Context, doer *user_model.User, colum } // LoadIssuesFromProject load issues assigned to each project column inside the given project -func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, opts *issues_model.IssuesOptions) (map[int64]issues_model.IssueList, error) { +func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, opts *issues_model.IssuesOptions) (results map[int64]issues_model.IssueList, _ error) { issueList, err := issues_model.Issues(ctx, opts.Copy(func(o *issues_model.IssuesOptions) { o.ProjectID = project.ID o.SortType = "project-column-sorting" @@ -103,7 +103,8 @@ func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, defaultColumn, err := project.GetDefaultColumnWithFallback(ctx) if err != nil { - return nil, util.Iif(errors.Is(err, util.ErrNotExist), nil, err) + // treat not exist error as empty result, because it means there is no column in the project, so no issue can be assigned to any column + return results, util.Iif(errors.Is(err, util.ErrNotExist), nil, err) } issueColumnMap, err := issues_model.LoadProjectIssueColumnMap(ctx, project.ID, defaultColumn.ID) @@ -111,7 +112,7 @@ func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, return nil, err } - results := make(map[int64]issues_model.IssueList) + results = make(map[int64]issues_model.IssueList) for _, issue := range issueList { projectColumnID, ok := issueColumnMap[issue.ID] if !ok { diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index dcd99a21a9bba..2d2fd39ac932a 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -25,7 +25,7 @@ export class IssueSidebarComboList { if (!['single', 'multiple'].includes(this.selectionMode)) throw new Error(`Invalid data-update-on: ${this.selectionMode}`); if (!['diff', 'all'].includes(this.updateAlgo)) throw new Error(`Invalid data-update-algo: ${this.updateAlgo}`); this.elDropdown = container.querySelector(':scope > .ui.dropdown')!; - this.elList = container.querySelector(':scope > .ui.list')!; + this.elList = container.querySelector(':scope > .ui.list'); this.elComboValue = container.querySelector(':scope > .combo-value')!; this.elIssueMainContent = document.querySelector('.issue-content-left')!; @@ -85,11 +85,13 @@ export class IssueSidebarComboList { for (const value of this.initialValues) { if (!changedValues.includes(value)) { lastResp = await POST(this.updateUrl, {data: new URLSearchParams({action: 'detach', id: value})}); + if (!lastResp.ok) return lastResp; } } for (const value of changedValues) { if (!this.initialValues.includes(value)) { lastResp = await POST(this.updateUrl, {data: new URLSearchParams({action: 'attach', id: value})}); + if (!lastResp.ok) return lastResp; } } } else { From e8e9f6c3a3221e6f51ebd7dbb2341208caf58e9b Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 03:51:18 +0800 Subject: [PATCH 03/11] Update web_src/js/features/repo-issue-sidebar-combolist.ts Signed-off-by: wxiaoguang --- web_src/js/features/repo-issue-sidebar-combolist.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index 2d2fd39ac932a..638b7bc520d3d 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -111,6 +111,7 @@ export class IssueSidebarComboList { } await this.reloadPagePartially(); } catch (e) { + console.error('Failed to update to backend', e); showErrorToast(`Failed to update to backend: ${e}`); } finally { this.elIssueSidebar.classList.remove('is-loading'); From e28533ced896cd3a701189bd119481ecdb72d71f Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 04:03:39 +0800 Subject: [PATCH 04/11] fix timeline item replace --- web_src/js/features/repo-issue-sidebar-combolist.ts | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index 638b7bc520d3d..d5de422354409 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -73,6 +73,14 @@ export class IssueSidebarComboList { const newTimelineItems = newMainContent.querySelectorAll(`.timeline-item[id]`); for (const newItem of newTimelineItems) { const newItemId = newItem.getAttribute('id')!; + if (newItem.classList.contains('event')) { + // for event item (e.g.: "add & remove labels"), we want to replace the existing one if exists + const oldItem = this.elIssueMainContent.querySelector(`.timeline-item.event[id="${CSS.escape(newItemId)}"]`); + if (oldItem) { + oldItem.replaceWith(newItem); + continue; + } + } if (this.elIssueMainContent.querySelector(`.timeline-item[id="${CSS.escape(newItemId)}"]`)) continue; lastTimelineItemForInsertion.insertAdjacentElement('afterend', newItem); lastTimelineItemForInsertion = newItem; From 998ead3277ede40bd4a4e6f1fd8be3689faf3d3b Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 04:30:44 +0800 Subject: [PATCH 05/11] syncIssueMainContentTimelineItems and tests --- templates/repo/issue/view_content.tmpl | 1 + .../repo-issue-sidebar-combolist.test.ts | 59 +++++++++++++++++++ .../features/repo-issue-sidebar-combolist.ts | 47 +++++++-------- 3 files changed, 84 insertions(+), 23 deletions(-) create mode 100644 web_src/js/features/repo-issue-sidebar-combolist.test.ts diff --git a/templates/repo/issue/view_content.tmpl b/templates/repo/issue/view_content.tmpl index 2bd75cdb9593a..275dd47a76b25 100644 --- a/templates/repo/issue/view_content.tmpl +++ b/templates/repo/issue/view_content.tmpl @@ -66,6 +66,7 @@
{{template "repo/issue/view_content/comments" .}} +
{{if and .Issue.IsPull (not $.Repository.IsArchived)}} {{template "repo/issue/view_content/pull_merge_box".}} diff --git a/web_src/js/features/repo-issue-sidebar-combolist.test.ts b/web_src/js/features/repo-issue-sidebar-combolist.test.ts new file mode 100644 index 0000000000000..1392bc18f4a78 --- /dev/null +++ b/web_src/js/features/repo-issue-sidebar-combolist.test.ts @@ -0,0 +1,59 @@ +import {syncIssueMainContentTimelineItems} from './repo-issue-sidebar-combolist.ts'; +import {createElementFromHTML} from '../utils/dom.ts'; + +describe('syncIssueMainContentTimelineItems', () => { + test('InsertNew', () => { + const oldContent = createElementFromHTML(` +
+
First
+
+
+ `); + const newContent = createElementFromHTML(` +
+
New
+
+ `); + syncIssueMainContentTimelineItems(oldContent, newContent); + expect(oldContent.innerHTML.replace(/>\s+<').trim()).toBe( + `
First
` + + `
New
` + + `
`, + ); + }); + + test('Sync', () => { + const oldContent = createElementFromHTML(` +
+
First
+
Item 1
+
Item 2
+
Item 3
+
+
Other
+
+ `); + const newContent = createElementFromHTML(` +
+
New 1
+
New 2
+
New 4
+
+ `); + syncIssueMainContentTimelineItems(oldContent, newContent); + + // Item 1 won't be replaced because it's not an event + // Item 2 will be replaced with New 2 + // Item 3 will be kept because it's not in new content + // New 4 will be inserted after Item 3 + expect(oldContent.innerHTML.replace(/>\s+<').trim()).toBe( + `
First
` + + `
Item 1
` + + `
New 2
` + + `
Item 3
` + + `
New 4
` + + `
` + + `
Other
`, + ); + }); +}); diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index d5de422354409..d4e40028c4c18 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -4,6 +4,29 @@ import {showErrorToast} from '../modules/toast.ts'; import {addDelegatedEventListener, queryElemChildren, queryElems, toggleElem} from '../utils/dom.ts'; import {parseDom} from '../utils.ts'; +export function syncIssueMainContentTimelineItems(oldMainContent: Element, newMainContent: Element) { + // find the last ".timeline-item[id]" in current main content, and insert new items after it, some cases: + // * a new empty issue: "timeline-item comment first", "timeline-item comment form (optional)" + // * issue with timeline events: "timeline-item comment first", "timeline-item event"+id, "timeline-item comment form (optional)" + let lastTimelineItemForInsertion = oldMainContent.querySelector('.timeline-item[id="timeline-comments-end"]'); + if (!lastTimelineItemForInsertion) return; + + const newTimelineItems = newMainContent.querySelectorAll(`.timeline-item[id]`); + for (const newItem of newTimelineItems) { + const newItemId = newItem.getAttribute('id')!; + const oldItem = oldMainContent.querySelector(`.timeline-item[id="${CSS.escape(newItemId)}"]`); + if (oldItem) { + if (oldItem.classList.contains('event')) { + // for event item (e.g.: "add & remove labels"), we want to replace the existing one if exists + oldItem.replaceWith(newItem); + } + continue; + } + lastTimelineItemForInsertion.insertAdjacentElement('beforebegin', newItem); + lastTimelineItemForInsertion = newItem; + } +} + export class IssueSidebarComboList { updateUrl: string; updateAlgo: string; @@ -62,29 +85,7 @@ export class IssueSidebarComboList { // for the main content (left side), at the moment we only support adding new timeline items const newMainContent = doc.querySelector('.issue-content-left')!; - - // find the last ".timeline-item[id]" in current main content, and insert new items after it, some cases: - // * a new empty issue: "timeline-item comment first", "timeline-item comment form (optional)" - // * issue with timeline events: "timeline-item comment first", "timeline-item event"+id, "timeline-item comment form (optional)" - const timelineItemsWithId = this.elIssueMainContent.querySelectorAll('.timeline-item[id]'); - let lastTimelineItemForInsertion = timelineItemsWithId[timelineItemsWithId.length - 1] ?? this.elIssueMainContent.querySelector('.timeline-item'); - if (!lastTimelineItemForInsertion) return; - - const newTimelineItems = newMainContent.querySelectorAll(`.timeline-item[id]`); - for (const newItem of newTimelineItems) { - const newItemId = newItem.getAttribute('id')!; - if (newItem.classList.contains('event')) { - // for event item (e.g.: "add & remove labels"), we want to replace the existing one if exists - const oldItem = this.elIssueMainContent.querySelector(`.timeline-item.event[id="${CSS.escape(newItemId)}"]`); - if (oldItem) { - oldItem.replaceWith(newItem); - continue; - } - } - if (this.elIssueMainContent.querySelector(`.timeline-item[id="${CSS.escape(newItemId)}"]`)) continue; - lastTimelineItemForInsertion.insertAdjacentElement('afterend', newItem); - lastTimelineItemForInsertion = newItem; - } + syncIssueMainContentTimelineItems(this.elIssueMainContent, newMainContent); } async sendRequestToBackend(changedValues: Array): Promise { From 857a02c70447fce18b4892a16be9b9db5d12746b Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 04:37:47 +0800 Subject: [PATCH 06/11] fine tune --- models/project/column.go | 7 +++--- services/projects/issue.go | 11 +++++---- .../repo-issue-sidebar-combolist.test.ts | 8 ++++--- .../features/repo-issue-sidebar-combolist.ts | 24 ++++++++++++------- web_src/js/features/repo-issue-sidebar.md | 3 +++ 5 files changed, 34 insertions(+), 19 deletions(-) diff --git a/models/project/column.go b/models/project/column.go index 977a5b8dfd71b..7365204f18ebf 100644 --- a/models/project/column.go +++ b/models/project/column.go @@ -257,8 +257,9 @@ func (p *Project) GetColumns(ctx context.Context) (ColumnList, error) { return columns, nil } -// GetDefaultColumnWithFallback return default column if one exists, otherwise return the first column by sorting and set it as default column -func (p *Project) GetDefaultColumnWithFallback(ctx context.Context) (*Column, error) { +// getDefaultColumnWithFallback return default column if one exists +// otherwise return the first column by sorting and set it as default column +func (p *Project) getDefaultColumnWithFallback(ctx context.Context) (*Column, error) { var column Column // try to find a column "default=true" @@ -294,7 +295,7 @@ func (p *Project) GetDefaultColumnWithFallback(ctx context.Context) (*Column, er // If none exists, the first column will be elevated to the default column of this project // If there is no column, it creates a default column and returns it func (p *Project) MustDefaultColumn(ctx context.Context) (*Column, error) { - c, err := p.GetDefaultColumnWithFallback(ctx) + c, err := p.getDefaultColumnWithFallback(ctx) if err != nil && !IsErrProjectColumnNotExist(err) { return nil, err } diff --git a/services/projects/issue.go b/services/projects/issue.go index d6b7aa61b3869..377c4b9d583f9 100644 --- a/services/projects/issue.go +++ b/services/projects/issue.go @@ -12,7 +12,6 @@ import ( project_model "code.gitea.io/gitea/models/project" user_model "code.gitea.io/gitea/models/user" "code.gitea.io/gitea/modules/optional" - "code.gitea.io/gitea/modules/util" ) // MoveIssuesOnProjectColumn moves or keeps issues in a column and sorts them inside that column @@ -96,15 +95,17 @@ func LoadIssuesFromProject(ctx context.Context, project *project_model.Project, if err != nil { return nil, err } - + if len(issueList) == 0 { + // if no issue, return directly, then no need to create a default column for an empty project + return results, nil + } if err := issueList.LoadComments(ctx); err != nil { return nil, err } - defaultColumn, err := project.GetDefaultColumnWithFallback(ctx) + defaultColumn, err := project.MustDefaultColumn(ctx) if err != nil { - // treat not exist error as empty result, because it means there is no column in the project, so no issue can be assigned to any column - return results, util.Iif(errors.Is(err, util.ErrNotExist), nil, err) + return nil, err } issueColumnMap, err := issues_model.LoadProjectIssueColumnMap(ctx, project.ID, defaultColumn.ID) diff --git a/web_src/js/features/repo-issue-sidebar-combolist.test.ts b/web_src/js/features/repo-issue-sidebar-combolist.test.ts index 1392bc18f4a78..ee2ea57a62d34 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.test.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.test.ts @@ -29,6 +29,7 @@ describe('syncIssueMainContentTimelineItems', () => {
Item 1
Item 2
Item 3
+
Item 4
Other
@@ -37,7 +38,7 @@ describe('syncIssueMainContentTimelineItems', () => {
New 1
New 2
-
New 4
+
New X
`); syncIssueMainContentTimelineItems(oldContent, newContent); @@ -45,13 +46,14 @@ describe('syncIssueMainContentTimelineItems', () => { // Item 1 won't be replaced because it's not an event // Item 2 will be replaced with New 2 // Item 3 will be kept because it's not in new content - // New 4 will be inserted after Item 3 + // Item 4 will be removed because it's not in new content, and it's an event + // New X will be inserted at the end of timeline items (before timeline-comments-end) expect(oldContent.innerHTML.replace(/>\s+<').trim()).toBe( `
First
` + `
Item 1
` + `
New 2
` + `
Item 3
` + - `
New 4
` + + `
New X
` + `
` + `
Other
`, ); diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index d4e40028c4c18..ab29a18f923ed 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -5,11 +5,19 @@ import {addDelegatedEventListener, queryElemChildren, queryElems, toggleElem} fr import {parseDom} from '../utils.ts'; export function syncIssueMainContentTimelineItems(oldMainContent: Element, newMainContent: Element) { - // find the last ".timeline-item[id]" in current main content, and insert new items after it, some cases: - // * a new empty issue: "timeline-item comment first", "timeline-item comment form (optional)" - // * issue with timeline events: "timeline-item comment first", "timeline-item event"+id, "timeline-item comment form (optional)" - let lastTimelineItemForInsertion = oldMainContent.querySelector('.timeline-item[id="timeline-comments-end"]'); - if (!lastTimelineItemForInsertion) return; + // find the end of comments timeline by "id=timeline-comments-end" in current main content, and insert new items before it + const timelineEnd = oldMainContent.querySelector('.timeline-item[id="timeline-comments-end"]'); + if (!timelineEnd) return; + + const oldTimelineItems = oldMainContent.querySelectorAll(`.timeline-item[id]`); + for (const oldItem of oldTimelineItems) { + const oldItemId = oldItem.getAttribute('id')!; + const newItem = newMainContent.querySelector(`.timeline-item[id="${CSS.escape(oldItemId)}"]`); + if (oldItem.classList.contains('event') && !newItem) { + // if the item is not in new content, we want to remove it from old content only if it's an event item, otherwise we keep it + oldItem.remove(); + } + } const newTimelineItems = newMainContent.querySelectorAll(`.timeline-item[id]`); for (const newItem of newTimelineItems) { @@ -18,12 +26,12 @@ export function syncIssueMainContentTimelineItems(oldMainContent: Element, newMa if (oldItem) { if (oldItem.classList.contains('event')) { // for event item (e.g.: "add & remove labels"), we want to replace the existing one if exists + // because the label operations can be merged into one event item, so the new item might be different from the old one oldItem.replaceWith(newItem); } continue; } - lastTimelineItemForInsertion.insertAdjacentElement('beforebegin', newItem); - lastTimelineItemForInsertion = newItem; + timelineEnd.insertAdjacentElement('beforebegin', newItem); } } @@ -83,7 +91,7 @@ export class IssueSidebarComboList { const newSidebar = doc.querySelector('.issue-content-right')!; this.elIssueSidebar.replaceWith(newSidebar); - // for the main content (left side), at the moment we only support adding new timeline items + // for the main content (left side), at the moment we only support handling known timeline items const newMainContent = doc.querySelector('.issue-content-left')!; syncIssueMainContentTimelineItems(this.elIssueMainContent, newMainContent); } diff --git a/web_src/js/features/repo-issue-sidebar.md b/web_src/js/features/repo-issue-sidebar.md index d732bc6f54133..1378d8775ffab 100644 --- a/web_src/js/features/repo-issue-sidebar.md +++ b/web_src/js/features/repo-issue-sidebar.md @@ -25,12 +25,15 @@ If there is `data-update-url`, it also calls backend to attach/detach the change Also, the changed items will be synchronized to the `ui list` items. The menu items must have correct `href`, otherwise the links of synchronized (cloned) items would be wrong. +The `ui list` is optional, so a single dropdown can also work, to select items and update them to backend. + Synchronization logic: * On page load: * If the dropdown menu contains checked items, there will be no synchronization. In this case, it's assumed that the dropdown menu is already in sync with the list. * If the dropdown menu doesn't contain checked items, it will use dropdown's value to mark the selected items as checked. And the selected (checked) items will be synchronized to the list. + Dropdown's value should be empty if the there is no dropdown item but a pre-defined list item need to be displayed. * On dropdown selection change: * The selected items will be synchronized to the list after the dropdown is hidden From bbfeee24e967e5f955cf1a193aa16b9d7492b846 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 07:28:23 +0800 Subject: [PATCH 07/11] add htmx.process --- web_src/js/features/repo-issue-sidebar-combolist.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/web_src/js/features/repo-issue-sidebar-combolist.ts b/web_src/js/features/repo-issue-sidebar-combolist.ts index ab29a18f923ed..b88bd7ead1b5d 100644 --- a/web_src/js/features/repo-issue-sidebar-combolist.ts +++ b/web_src/js/features/repo-issue-sidebar-combolist.ts @@ -28,10 +28,12 @@ export function syncIssueMainContentTimelineItems(oldMainContent: Element, newMa // for event item (e.g.: "add & remove labels"), we want to replace the existing one if exists // because the label operations can be merged into one event item, so the new item might be different from the old one oldItem.replaceWith(newItem); + window.htmx.process(newItem); } continue; } timelineEnd.insertAdjacentElement('beforebegin', newItem); + window.htmx.process(newItem); } } @@ -90,6 +92,7 @@ export class IssueSidebarComboList { // we can safely replace the whole right part (sidebar) because there are only some dropdowns and lists const newSidebar = doc.querySelector('.issue-content-right')!; this.elIssueSidebar.replaceWith(newSidebar); + window.htmx.process(newSidebar); // for the main content (left side), at the moment we only support handling known timeline items const newMainContent = doc.querySelector('.issue-content-left')!; From 12933a8a0352c9f119e0f25cd740cfe54c1a4619 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 08:51:57 +0800 Subject: [PATCH 08/11] move more legacy sidebar code --- web_src/js/features/repo-issue-sidebar.ts | 66 +++++++++++++++++++++++ web_src/js/features/repo-issue.ts | 66 ++--------------------- web_src/js/index.ts | 4 +- 3 files changed, 70 insertions(+), 66 deletions(-) diff --git a/web_src/js/features/repo-issue-sidebar.ts b/web_src/js/features/repo-issue-sidebar.ts index 16873af76a6fe..2983b6e1af926 100644 --- a/web_src/js/features/repo-issue-sidebar.ts +++ b/web_src/js/features/repo-issue-sidebar.ts @@ -2,6 +2,12 @@ import {POST} from '../modules/fetch.ts'; import {queryElems, toggleElem} from '../utils/dom.ts'; import {IssueSidebarComboList} from './repo-issue-sidebar-combolist.ts'; import {registerGlobalInitFunc} from '../modules/observer.ts'; +import {parseIssuePageInfo} from '../utils.ts'; +import {html} from '../utils/html.ts'; +import {fomanticQuery} from '../modules/fomantic/base.ts'; +import {showTemporaryTooltip} from '../modules/tippy.ts'; + +const {appSubUrl} = window.config; function initRepoIssueBranchSelector(elSidebar: HTMLElement) { // TODO: RemoveIssueRef: see "repo/issue/branch_selector_field.tmpl" @@ -43,10 +49,70 @@ function initRepoIssueDue(elSidebar: HTMLElement) { }); } +export function initRepoIssueSidebarDependency(elSidebar: HTMLElement) { + const elDropdown = elSidebar.querySelector('#new-dependency-drop-list'); + if (!elDropdown) return; + + const issuePageInfo = parseIssuePageInfo(); + const crossRepoSearch = elDropdown.getAttribute('data-issue-cross-repo-search'); + let issueSearchUrl = `${issuePageInfo.repoLink}/issues/search?q={query}&type=${issuePageInfo.issueDependencySearchType}`; + if (crossRepoSearch === 'true') { + issueSearchUrl = `${appSubUrl}/issues/search?q={query}&priority_repo_id=${issuePageInfo.repoId}&type=${issuePageInfo.issueDependencySearchType}`; + } + fomanticQuery(elDropdown).dropdown({ + fullTextSearch: true, + apiSettings: { + cache: false, + rawResponse: true, + url: issueSearchUrl, + onResponse(response: any) { + const filteredResponse = {success: true, results: [] as Array>}; + const currIssueId = elDropdown.getAttribute('data-issue-id'); + // Parse the response from the api to work with our dropdown + for (const issue of response) { + // Don't list current issue in the dependency list. + if (String(issue.id) === currIssueId) continue; + filteredResponse.results.push({ + value: issue.id, + name: html`
#${issue.number} ${issue.title}
${issue.repository.full_name}
`, + }); + } + return filteredResponse; + }, + }, + }); +} + +export function initRepoPullRequestAllowMaintainerEdit(elSidebar: HTMLElement) { + const wrapper = elSidebar.querySelector('#allow-edits-from-maintainers')!; + if (!wrapper) return; + const checkbox = wrapper.querySelector('input[type="checkbox"]')!; + checkbox.addEventListener('input', async () => { + const url = `${wrapper.getAttribute('data-url')}/set_allow_maintainer_edit`; + wrapper.classList.add('is-loading'); + try { + const resp = await POST(url, {data: new URLSearchParams({allow_maintainer_edit: String(checkbox.checked)})}); + if (!resp.ok) { + throw new Error('Failed to update maintainer edit permission'); + } + const data = await resp.json(); + checkbox.checked = data.allow_maintainer_edit; + } catch (error) { + checkbox.checked = !checkbox.checked; + console.error(error); + showTemporaryTooltip(wrapper, wrapper.getAttribute('data-prompt-error')!); + } finally { + wrapper.classList.remove('is-loading'); + } + }); +} + export function initRepoIssueSidebar() { registerGlobalInitFunc('initRepoIssueSidebar', (elSidebar) => { initRepoIssueBranchSelector(elSidebar); initRepoIssueDue(elSidebar); + initRepoIssueSidebarDependency(elSidebar); + initRepoPullRequestAllowMaintainerEdit(elSidebar); // init the combo list: a dropdown for selecting items, and a list for showing selected items and related actions queryElems(elSidebar, '.issue-sidebar-combo', (el) => new IssueSidebarComboList(el).init()); }); diff --git a/web_src/js/features/repo-issue.ts b/web_src/js/features/repo-issue.ts index 5cbfac5692990..03b0c93d85fcd 100644 --- a/web_src/js/features/repo-issue.ts +++ b/web_src/js/features/repo-issue.ts @@ -1,5 +1,5 @@ -import {html, htmlEscape} from '../utils/html.ts'; -import {createTippy, showTemporaryTooltip} from '../modules/tippy.ts'; +import {htmlEscape} from '../utils/html.ts'; +import {createTippy} from '../modules/tippy.ts'; import { addDelegatedEventListener, createElementFromHTML, @@ -10,7 +10,7 @@ import { } from '../utils/dom.ts'; import {setFileFolding} from './file-fold.ts'; import {ComboMarkdownEditor, getComboMarkdownEditor, initComboMarkdownEditor} from './comp/ComboMarkdownEditor.ts'; -import {parseIssuePageInfo, toAbsoluteUrl} from '../utils.ts'; +import {toAbsoluteUrl} from '../utils.ts'; import {GET, POST} from '../modules/fetch.ts'; import {showErrorToast} from '../modules/toast.ts'; import {initRepoIssueSidebar} from './repo-issue-sidebar.ts'; @@ -20,40 +20,6 @@ import {registerGlobalInitFunc} from '../modules/observer.ts'; const {appSubUrl} = window.config; -export function initRepoIssueSidebarDependency() { - const elDropdown = document.querySelector('#new-dependency-drop-list'); - if (!elDropdown) return; - - const issuePageInfo = parseIssuePageInfo(); - const crossRepoSearch = elDropdown.getAttribute('data-issue-cross-repo-search'); - let issueSearchUrl = `${issuePageInfo.repoLink}/issues/search?q={query}&type=${issuePageInfo.issueDependencySearchType}`; - if (crossRepoSearch === 'true') { - issueSearchUrl = `${appSubUrl}/issues/search?q={query}&priority_repo_id=${issuePageInfo.repoId}&type=${issuePageInfo.issueDependencySearchType}`; - } - fomanticQuery(elDropdown).dropdown({ - fullTextSearch: true, - apiSettings: { - cache: false, - rawResponse: true, - url: issueSearchUrl, - onResponse(response: any) { - const filteredResponse = {success: true, results: [] as Array>}; - const currIssueId = elDropdown.getAttribute('data-issue-id'); - // Parse the response from the api to work with our dropdown - for (const issue of response) { - // Don't list current issue in the dependency list. - if (String(issue.id) === currIssueId) continue; - filteredResponse.results.push({ - value: issue.id, - name: html`
#${issue.number} ${issue.title}
${issue.repository.full_name}
`, - }); - } - return filteredResponse; - }, - }, - }); -} - function initRepoIssueLabelFilter(elDropdown: HTMLElement) { const url = new URL(window.location.href); const showArchivedLabels = url.searchParams.get('archived_labels') === 'true'; @@ -197,32 +163,6 @@ export function initRepoIssueCodeCommentCancel() { }); } -export function initRepoPullRequestAllowMaintainerEdit() { - const wrapper = document.querySelector('#allow-edits-from-maintainers')!; - if (!wrapper) return; - const checkbox = wrapper.querySelector('input[type="checkbox"]')!; - checkbox.addEventListener('input', async () => { - const url = `${wrapper.getAttribute('data-url')}/set_allow_maintainer_edit`; - wrapper.classList.add('is-loading'); - try { - const resp = await POST(url, {data: new URLSearchParams({ - allow_maintainer_edit: String(checkbox.checked), - })}); - if (!resp.ok) { - throw new Error('Failed to update maintainer edit permission'); - } - const data = await resp.json(); - checkbox.checked = data.allow_maintainer_edit; - } catch (error) { - checkbox.checked = !checkbox.checked; - console.error(error); - showTemporaryTooltip(wrapper, wrapper.getAttribute('data-prompt-error')!); - } finally { - wrapper.classList.remove('is-loading'); - } - }); -} - export function initRepoIssueComments() { if (!document.querySelector('.repository.view.issue .timeline')) return; diff --git a/web_src/js/index.ts b/web_src/js/index.ts index e0b4a3e521afc..5e2cde28eeaff 100644 --- a/web_src/js/index.ts +++ b/web_src/js/index.ts @@ -21,7 +21,7 @@ import {initRepoFileSearch} from './features/repo-findfile.ts'; import {initMarkupContent} from './markup/content.ts'; import {initRepoFileView} from './features/file-view.ts'; import {initUserAuthOauth2, initUserCheckAppUrl} from './features/user-auth.ts'; -import {initRepoPullRequestAllowMaintainerEdit, initRepoPullRequestReview, initRepoIssueSidebarDependency, initRepoIssueFilterItemLabel} from './features/repo-issue.ts'; +import {initRepoPullRequestReview, initRepoIssueFilterItemLabel} from './features/repo-issue.ts'; import {initRepoEllipsisButton, initCommitStatuses} from './features/repo-commit.ts'; import {initRepoTopicBar} from './features/repo-home.ts'; import {initAdminCommon} from './features/admin/common.ts'; @@ -130,11 +130,9 @@ const initPerformanceTracer = callInitFunctions([ initRepoIssueContentHistory, initRepoIssueList, initRepoIssueFilterItemLabel, - initRepoIssueSidebarDependency, initRepoMigration, initRepoMigrationStatusChecker, initRepoProject, - initRepoPullRequestAllowMaintainerEdit, initRepoPullRequestReview, initRepoReleaseNew, initRepoTopicBar, From c0a341cced57f9b93bff137fd51d4a7b88add4b4 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 09:07:04 +0800 Subject: [PATCH 09/11] fix layout problems for Windows --- templates/repo/issue/view_content/watching.tmpl | 4 ++-- web_src/css/modules/button.css | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/templates/repo/issue/view_content/watching.tmpl b/templates/repo/issue/view_content/watching.tmpl index 05936d090b7e2..1abc92742ad41 100644 --- a/templates/repo/issue/view_content/watching.tmpl +++ b/templates/repo/issue/view_content/watching.tmpl @@ -2,10 +2,10 @@ diff --git a/web_src/css/modules/button.css b/web_src/css/modules/button.css index 9458f11e53e18..9b509f8cd40bd 100644 --- a/web_src/css/modules/button.css +++ b/web_src/css/modules/button.css @@ -214,7 +214,6 @@ .ui.fluid.button { width: 100%; - display: block; } .ui.primary.button, From 4caebd391e1e1d8f6c712409966ef7af2e1d4fdf Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 09:17:31 +0800 Subject: [PATCH 10/11] fix more layouts for Windows --- templates/base/head_navbar.tmpl | 8 ++++---- templates/base/head_navbar_icons.tmpl | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/templates/base/head_navbar.tmpl b/templates/base/head_navbar.tmpl index 447f78565e0c3..cc7e4e6775b70 100644 --- a/templates/base/head_navbar.tmpl +++ b/templates/base/head_navbar.tmpl @@ -47,7 +47,7 @@ {{ctx.AvatarUtils.Avatar .SignedUser 24 "tw-mr-1"}} {{.SignedUser.Name}} - {{svg "octicon-triangle-down"}} + {{svg "octicon-triangle-down"}}
@@ -64,9 +64,9 @@ {{else if .IsSigned}} {{template "base/head_navbar_icons" dict "ItemExtraClass" "not-mobile" "PageGlobalData" .PageGlobalData}}
- + {{svg "octicon-plus"}} - {{svg "octicon-triangle-down"}} + {{svg "octicon-triangle-down"}} {{ctx.Locale.Tr "create_new"}}
@@ -93,7 +93,7 @@ {{if .IsAdmin}}{{svg "octicon-shield-check" 16 "navbar-admin-badge"}}{{end}} {{.SignedUser.Name}} - {{svg "octicon-triangle-down"}} + {{svg "octicon-triangle-down"}}
diff --git a/templates/base/head_navbar_icons.tmpl b/templates/base/head_navbar_icons.tmpl index 89b02389fc8c2..019e78cb60f9b 100644 --- a/templates/base/head_navbar_icons.tmpl +++ b/templates/base/head_navbar_icons.tmpl @@ -5,14 +5,14 @@ {{- $notificationUnreadCount := call $data.GetNotificationUnreadCount -}} {{if $activeStopwatch}} -
+
{{svg "octicon-stopwatch"}}
{{end}} -
+
{{svg "octicon-bell"}} {{$notificationUnreadCount}}
From e7095e0957a6b46273c2c21afff3450543cb8257 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 31 Mar 2026 09:33:12 +0800 Subject: [PATCH 11/11] fix more layouts for Windows and also checked for macOS --- templates/shared/issuelist.tmpl | 2 +- web_src/css/modules/label.css | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/templates/shared/issuelist.tmpl b/templates/shared/issuelist.tmpl index 9f1066385559f..aaf80a1ec94d4 100644 --- a/templates/shared/issuelist.tmpl +++ b/templates/shared/issuelist.tmpl @@ -5,7 +5,7 @@
{{/* using some tw helpers is the only way to align the checkbox */}} -
+
{{if $.CanWriteIssuesOrPulls}} {{end}} diff --git a/web_src/css/modules/label.css b/web_src/css/modules/label.css index cf850e4c5aac7..6db13e2954ce2 100644 --- a/web_src/css/modules/label.css +++ b/web_src/css/modules/label.css @@ -292,6 +292,7 @@ If the labels-list itself needs some layouts, use extra classes or "tw" helpers. } .labels-list a { + display: inline-flex; max-width: 100%; /* for ellipsis */ }