Skip to content

refactor: replace delete-button with link-action in user settings organization#36780

Open
santoshchandu wants to merge 4 commits intogo-gitea:mainfrom
santoshchandu:refactor/delete-button-to-link-action
Open

refactor: replace delete-button with link-action in user settings organization#36780
santoshchandu wants to merge 4 commits intogo-gitea:mainfrom
santoshchandu:refactor/delete-button-to-link-action

Conversation

@santoshchandu
Copy link

The issue number is #35015

This PR simplifies the DOM structure for the "Leave Organization" action in the user settings by replacing the form-based button with a standard link-action anchor. It also removes the redundant modal wrapper, leveraging the standard global confirm modal instead.

Changes:

Replaced the

and for leaving an organization with an tag using the .link-action class.
Updated the modal trigger from data-modal-id to data-modal-confirm.
Removed the obsolete #leave-organization modal
wrapper.
Added missing trailing newline to organization.tmpl.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 28, 2026
@github-actions github-actions bot added the modifies/templates This PR modifies the template files label Feb 28, 2026
Copy link
Contributor

@wxiaoguang wxiaoguang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you ever tested the change.

@GiteaBot GiteaBot added lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged modifies/templates This PR modifies the template files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants