forgejo/forgejo
120
5.1k
Fork
You've already forked forgejo
863

12975: Fix HTMX attributes in watch, star, and notification buttons #12976

Merged
0ko merged 5 commits from birdgoose/forgejo:12975-prevent-url-change-when-watching-or-starring into forgejo 2026年06月28日 18:00:29 +02:00
Contributor
Copy link

Description

This PR addresses the issue forgejo/forgejo#12951 by implementing the fixes suggested in this comment.

The changes are made to the following template files:

  • on the repo star/unstar button
  • on the repo watch/unwatch button
  • on the issue notification button

Tests have been added to ensure that the URL of the page doesn't change when the user clicks those buttons.

Checklist

The contributor guide contains information that will be helpful to first time contributors. All work and communication must conform to Forgejo's AI Agreement. There also are a few conditions for merging Pull Requests in Forgejo repositories. You are also welcome to join the Forgejo development chatroom.

Tests for JavaScript changes

(can be removed for Go changes)

  • I added test coverage for JavaScript changes...

Documentation

  • I created a pull request to the documentation to explain to Forgejo users how to use this change.
  • I did not document these changes and I do not expect someone else to do it.

Release notes

  • This change will be noticed by a Forgejo user or admin (feature, bug fix, performance, etc.). I suggest to include a release note for this change. (merger note: it will not because no stable versions with the regression were published)
  • This change is not visible to a Forgejo user or admin (refactor, dependency upgrade, etc.). I think there is no need to add a release note for this change.

The decision if the pull request will be shown in the release notes is up to the mergers / release team.

The content of the release-notes/<pull request number>.md file will serve as the basis for the release notes. If the file does not exist, the title of the pull request will be used instead.

## Description This PR addresses the issue forgejo/forgejo#12951 by implementing the fixes suggested in [this comment](https://codeberg.org/forgejo/forgejo/issues/12975#issuecomment-16653671). The changes are made to the following template files: - on the repo star/unstar button - on the repo watch/unwatch button - on the issue notification button Tests have been added to ensure that the URL of the page doesn't change when the user clicks those buttons. ## Checklist The [contributor guide](https://forgejo.org/docs/next/contributor/) contains information that will be helpful to first time contributors. All work and communication must conform to Forgejo's [AI Agreement](https://codeberg.org/forgejo/governance/src/branch/main/AIAgreement.md). There also are a few [conditions for merging Pull Requests in Forgejo repositories](https://codeberg.org/forgejo/governance/src/branch/main/PullRequestsAgreement.md). You are also welcome to join the [Forgejo development chatroom](https://matrix.to/#/#forgejo-development:matrix.org). ### Tests for JavaScript changes (can be removed for Go changes) - I added test coverage for JavaScript changes... - [ ] in `web_src/js/*.test.js` if it can be unit tested. - [x] in `tests/e2e/*.test.e2e.js` if it requires interactions with a live Forgejo server (see also the [developer guide for JavaScript testing](https://codeberg.org/forgejo/forgejo/src/branch/forgejo/tests/e2e/README.md#end-to-end-tests)). ### Documentation - [ ] I created a pull request [to the documentation](https://codeberg.org/forgejo/docs) to explain to Forgejo users how to use this change. - [x] I did not document these changes and I do not expect someone else to do it. ### Release notes - [x] This change will be noticed by a Forgejo user or admin (feature, bug fix, performance, etc.). I suggest to include a release note for this change. (merger note: it will not because no stable versions with the regression were published) - [ ] This change is not visible to a Forgejo user or admin (refactor, dependency upgrade, etc.). I think there is no need to add a release note for this change. *The decision if the pull request will be shown in the release notes is up to the mergers / release team.* The content of the `release-notes/<pull request number>.md` file will serve as the basis for the release notes. If the file does not exist, the title of the pull request will be used instead.
12975: Remove the hx-boosted attribute when it causes the URL to change erroneously.
Some checks failed
issue-labels / backporting (pull_request_target) Has been skipped
issue-labels / cascade (pull_request_target) Has been skipped
testing / backend-checks (pull_request) Has been cancelled
testing / frontend-checks (pull_request) Has been cancelled
testing / test-unit (pull_request) Has been cancelled
testing / test-e2e (pull_request) Has been cancelled
testing / test-remote-cacher (redis) (pull_request) Has been cancelled
testing / test-remote-cacher (valkey) (pull_request) Has been cancelled
testing / test-remote-cacher (garnet) (pull_request) Has been cancelled
testing / test-remote-cacher (redict) (pull_request) Has been cancelled
testing / test-mysql (pull_request) Has been cancelled
testing / test-pgsql (pull_request) Has been cancelled
testing / test-sqlite (pull_request) Has been cancelled
testing / security-check (pull_request) Has been cancelled
testing / semgrep/ci (pull_request) Has been cancelled
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Has been cancelled
06a960616a 
- on the repo star/unstar button
- on the repo watch/unwatch button
- on the issue notification button
12975: Add e2e tests for the repo home and issue sidebar.
Some checks failed
issue-labels / backporting (pull_request_target) Has been skipped
issue-labels / cascade (pull_request_target) Has been skipped
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Successful in 2s
testing / semgrep/ci (pull_request) Successful in 14s
testing / frontend-checks (pull_request) Failing after 42s
testing / backend-checks (pull_request) Successful in 2m36s
testing / test-sqlite (pull_request) Has been skipped
testing / test-e2e (pull_request) Has been skipped
testing / test-mysql (pull_request) Has been skipped
testing / test-pgsql (pull_request) Has been skipped
testing / test-unit (pull_request) Has been skipped
testing / test-remote-cacher (redis) (pull_request) Has been skipped
testing / test-remote-cacher (redict) (pull_request) Has been skipped
testing / test-remote-cacher (garnet) (pull_request) Has been skipped
testing / test-remote-cacher (valkey) (pull_request) Has been skipped
testing / security-check (pull_request) Has been skipped
8e6aa25db1 
These tests check that the page URL doesn't change when:
- watching/unwatching the issue
- watching/unwatching the repo
- starring/unstarring the repo
birdgoose changed title from (削除) WIP: 12975: Fix HTMX attributes in watch, star, and notification buttons (削除ここまで) to 12975: Fix HTMX attributes in watch, star, and notification buttons 2026年06月08日 21:00:24 +02:00

I'm a little unsure about this direction for a fix, as opposed to rolling back the dependency and filing an issue with htmx. It doesn't seem like this is intended behaviour from the upstream change, right?

I'm a little unsure about this direction for a fix, as opposed to rolling back the dependency and filing an issue with htmx. It doesn't seem like this is intended behaviour from the upstream change, right?
birdgoose force-pushed 12975-prevent-url-change-when-watching-or-starring from 8e6aa25db1
Some checks failed
issue-labels / backporting (pull_request_target) Has been skipped
issue-labels / cascade (pull_request_target) Has been skipped
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Successful in 2s
testing / semgrep/ci (pull_request) Successful in 14s
testing / frontend-checks (pull_request) Failing after 42s
testing / backend-checks (pull_request) Successful in 2m36s
testing / test-sqlite (pull_request) Has been skipped
testing / test-e2e (pull_request) Has been skipped
testing / test-mysql (pull_request) Has been skipped
testing / test-pgsql (pull_request) Has been skipped
testing / test-unit (pull_request) Has been skipped
testing / test-remote-cacher (redis) (pull_request) Has been skipped
testing / test-remote-cacher (redict) (pull_request) Has been skipped
testing / test-remote-cacher (garnet) (pull_request) Has been skipped
testing / test-remote-cacher (valkey) (pull_request) Has been skipped
testing / security-check (pull_request) Has been skipped
to 45a7f31a61
Some checks failed
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Has been cancelled
testing / backend-checks (pull_request) Has been cancelled
testing / frontend-checks (pull_request) Has been cancelled
testing / test-unit (pull_request) Has been cancelled
testing / test-e2e (pull_request) Has been cancelled
testing / test-remote-cacher (redis) (pull_request) Has been cancelled
testing / test-remote-cacher (valkey) (pull_request) Has been cancelled
testing / test-remote-cacher (garnet) (pull_request) Has been cancelled
testing / test-remote-cacher (redict) (pull_request) Has been cancelled
testing / test-mysql (pull_request) Has been cancelled
testing / test-pgsql (pull_request) Has been cancelled
testing / test-sqlite (pull_request) Has been cancelled
testing / security-check (pull_request) Has been cancelled
testing / semgrep/ci (pull_request) Has been cancelled
Integration tests for the release process / release-simulation (pull_request) Successful in 7m55s
2026年06月11日 06:28:35 +02:00 Compare
12975: Allow watching and starring to degrade gracefully when JS is disabled.
Some checks failed
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Successful in 3s
testing / semgrep/ci (pull_request) Successful in 26s
testing / test-unit (pull_request) Has been cancelled
testing / test-e2e (pull_request) Has been cancelled
testing / test-remote-cacher (redis) (pull_request) Has been cancelled
testing / test-remote-cacher (valkey) (pull_request) Has been cancelled
testing / test-remote-cacher (garnet) (pull_request) Has been cancelled
testing / test-remote-cacher (redict) (pull_request) Has been cancelled
testing / test-mysql (pull_request) Has been cancelled
testing / test-pgsql (pull_request) Has been cancelled
testing / test-sqlite (pull_request) Has been cancelled
testing / security-check (pull_request) Has been cancelled
testing / backend-checks (pull_request) Has been cancelled
testing / frontend-checks (pull_request) Has been cancelled
35f8a92b95
Author
Contributor
Copy link

@mfenniak wrote in #12976 (comment):

I'm a little unsure about this direction for a fix, as opposed to rolling back the dependency and filing an issue with htmx. It doesn't seem like this is intended behaviour from the upstream change, right?

I was wondering about this myself. I personally don't know enough about the internals of HTMX to be able to judge whether or not the upstream change should have caused this behaviour. If this does turn out to be an issue with HTMX then we don't lose anything (see note below) by implementing this fix since it is just another documented way of using the library.

Note: Your comment did make me think about our friends that don't have JavaScript enabled and so I've pushed a commit that would mean this behaviour degrades gracefully with JS turned off.

@mfenniak wrote in https://codeberg.org/forgejo/forgejo/pulls/12976#issuecomment-16856357: > I'm a little unsure about this direction for a fix, as opposed to rolling back the dependency and filing an issue with htmx. It doesn't seem like this is intended behaviour from the upstream change, right? I was wondering about this myself. I personally don't know enough about the internals of HTMX to be able to judge whether or not the upstream change should have caused this behaviour. If this does turn out to be an issue with HTMX then we don't lose anything (see note below) by implementing this fix since it is just another documented way of using the library. Note: Your comment did make me think about our friends that don't have JavaScript enabled and so I've pushed a commit that would mean this behaviour degrades gracefully with JS turned off.
12975: Fix linting issues in templates.
Some checks failed
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Has been cancelled
testing / semgrep/ci (pull_request) Successful in 22s
testing / frontend-checks (pull_request) Successful in 1m11s
testing / backend-checks (pull_request) Successful in 7m21s
testing / test-unit (pull_request) Successful in 4m27s
testing / test-remote-cacher (redis) (pull_request) Successful in 1m54s
testing / test-remote-cacher (valkey) (pull_request) Successful in 1m55s
testing / security-check (pull_request) Has been cancelled
testing / test-mysql (pull_request) Has been cancelled
testing / test-e2e (pull_request) Has been cancelled
testing / test-remote-cacher (garnet) (pull_request) Has been cancelled
testing / test-remote-cacher (redict) (pull_request) Has been cancelled
testing / test-pgsql (pull_request) Has been cancelled
testing / test-sqlite (pull_request) Has been cancelled
650ffb58ff

My preference would be to rollback the htmx dependency upgrade, and take the issue upstream. If htmx maintainers report that this is an intended behaviour change in a minor version -- a combination which seems very unlikely to me -- then we move this PR forward.

My preference would be to rollback the htmx dependency upgrade, and take the issue upstream. If htmx maintainers report that this is an intended behaviour change in a minor version -- a combination which seems very unlikely to me -- then we move this PR forward.
First-time contributor
Copy link

Im thinking this will conflict with my PR.
#12247

Im thinking this will conflict with my PR. https://codeberg.org/forgejo/forgejo/pulls/12247

@0ko As you've noted this bug as well and may be more versed in htmx usage than me, do you have an opinion to share on a fix direction? 🙂

@0ko As you've noted this bug as well and may be more versed in htmx usage than me, do you have an opinion to share on a fix direction? 🙂
Owner
Copy link

@birdgoose wrote in #12976 (comment):

This PR addresses the issue forgejo/forgejo#12951 by implementing the fixes suggested in this comment.

This is now outdated, but I appreciate the attention to having noJS work.

@mfenniak wrote in #12976 (comment):

As you've noted this bug as well and may be more versed in htmx usage than me

Only in a way that since I'm longer in Forgejo I had more time to poke it with a stick. But I am confused and terrified by HTMX and have no idea how it works. I only know how it breaks various parts of UI where JS only changes element attributes on page load.

@birdgoose wrote in https://codeberg.org/forgejo/forgejo/pulls/12976#issue-5579483: > This PR addresses the issue forgejo/forgejo#12951 by implementing the fixes suggested in [this comment](https://codeberg.org/forgejo/forgejo/issues/12975#issuecomment-16653671). This is now outdated, but I appreciate the attention to having noJS work. @mfenniak wrote in https://codeberg.org/forgejo/forgejo/pulls/12976#issuecomment-17353634: > As you've noted this bug as well and may be more versed in htmx usage than me Only in a way that since I'm longer in Forgejo I had more time to poke it with a stick. But I am confused and terrified by HTMX and have no idea how it works. I only know how it breaks various parts of UI where JS only changes element attributes on page load.
@ -70,2 +70,4 @@
).toBeVisible();
});
test('Star/unstar button URL retention', async ({page}) => {
Owner
Copy link

I'm unhappy that we now have separate tests that do almost the same time-costly things to get into a state and only verify one different thing. Can this test be merged with Star button focus retention and named Star and unstar?

I'm unhappy that we now have separate tests that do almost the same time-costly things to get into a state and only verify one different thing. Can this test be merged with `Star button focus retention` and named `Star and unstar`?
0ko marked this conversation as resolved
Owner
Copy link

As for the fix I don't really understand how it works and why without hx-boost page doesn't get reloaded on request (why hx-boost was there in the first place), but I'm in favor of having it merged rather than rolling back HTMX, but also probably avoiding to upgrade HTMX later unless it's actually needed to fix something.

As for the fix I don't really understand how it works and why without `hx-boost` page doesn't get reloaded on request (why `hx-boost` was there in the first place), but I'm in favor of having it merged rather than rolling back HTMX, but also probably avoiding to upgrade HTMX later unless it's actually needed to fix something.
12975: Merge star/unstar button E2E tests for clarity and performance.
Some checks failed
testing / semgrep/ci (pull_request) Successful in 25s
testing / frontend-checks (pull_request) Successful in 1m10s
testing / backend-checks (pull_request) Successful in 2m58s
testing / test-unit (pull_request) Successful in 5m51s
testing / test-remote-cacher (redis) (pull_request) Successful in 1m57s
testing / test-remote-cacher (valkey) (pull_request) Successful in 2m4s
testing / test-remote-cacher (garnet) (pull_request) Successful in 1m43s
testing / test-remote-cacher (redict) (pull_request) Failing after 7m10s
testing / test-e2e (pull_request) Successful in 25m4s
testing / test-mysql (pull_request) Successful in 27m6s
testing / test-sqlite (pull_request) Successful in 42m16s
testing / test-pgsql (pull_request) Successful in 45m23s
testing / security-check (pull_request) Has been skipped
issue-labels / cascade (pull_request_target) Has been skipped
milestone / set (pull_request_target) Successful in 10s
issue-labels / backporting (pull_request_target) Successful in 3m55s
issue-labels / release-notes (pull_request_target) Has been skipped
requirements / merge-conditions (pull_request) Successful in 3s
f05651d3d9
Author
Contributor
Copy link

@0ko thanks for your feedback, I've addressed that issue with the tests.

@0ko thanks for your feedback, I've addressed that issue with the tests.
0ko approved these changes 2026年06月28日 17:59:20 +02:00
0ko left a comment
Copy link

This looks good to go. Nice to have it test covered.

I haven't found anything in upstream issue tracker about this issue, and do no intend to ask in upstream. v2.0.9 was published like 2.5 months ago, so I would assume we were doing something wrong here or everyone who was impacted by the issue did not care as well.

We need this fixed before v16 release, so will merge.

This looks good to go. Nice to have it test covered. I haven't found anything in upstream issue tracker about this issue, and do no intend to ask in upstream. v2.0.9 was published like 2.5 months ago, so I would assume we were doing something wrong here or everyone who was impacted by the issue did not care as well. We need this fixed before v16 release, so will merge.
0ko merged commit 1ab4c8d915 into forgejo 2026年06月28日 18:00:29 +02:00
Owner
Copy link

(Thanks!)

(Thanks!)
Author
Contributor
Copy link

Glad I could help!

Glad I could help!
Sign in to join this conversation.
No reviewers
Labels
Clear labels
arch
riscv64

Archived

backport/v1.19
Scheduled for backport to Forgejo v1.19

Archived

backport/v1.20
Scheduled for backport to Forgejo v1.20

Archived

backport/v1.21/forgejo
Scheduled for backport to Forgejo v1.21

Archived

backport/v10.0/forgejo
Automated backport to v10.0

Archived

backport/v11.0/forgejo
Automated backport to v11.0
backport/v12.0/forgejo
Automated backport to v12.0

Archived

backport/v13.0/forgejo
Automated backport to v13.0

Archived

backport/v14.0/forgejo
Automated backport to v14.0

Archived

backport/v15.0/forgejo
Automated backport to v15.0
backport/v16.0/forgejo
Automated backport to v16.0
backport/v7.0/forgejo
Scheduled for backport to Forgejo v7.0

Archived

backport/v8.0/forgejo
Scheduled for backport to Forgejo v8.0

Archived

backport/v9.0/forgejo
Scheduled for backport to Forgejo v9.0

Archived

breaking
The release containing this change is not backward compatible
bug
Something is not working

Archived

bug
confirmed
 it can be reproduced
bug
duplicate
 bug has already been reported in the Forgejo tracker
bug
needs-more-info
 the information provided does not contain enough details
bug
new-report
 bug has just been reported and need triage (default label on issue creation)
bug
reported-upstream
 bug cannot be fixed within Forgejo easily, it has been reported upstream
code/actions
Forgejo Actions feature
code/api
API
code/auth
Forgejo Authentication
code/auth/faidp
Forgejo as Identity Provider (in OAuth/OIDC flow)
code/auth/farp
Forgejo as Relying Party / Client (in OAuth/OIDC flow)
code/email
Everything related to email in Forgejo
code/federation
Federation
code/git
Related to the Git backend in Forgejo
code/migrations
Migration between Git forges (i.e. for GitHub, GitLab, Gitea, Forgejo, etc.). NOT for database migrations.
code/packages
Forgejo package and container registry
code/wiki
database
MySQL
 https://www.mysql.com/
database
PostgreSQL
 https://www.postgresql.org/
database
SQLite
 https://sqlite.org/
dependency-upgrade
https://codeberg.org/forgejo/forgejo/issues/2779
dependency
Chi
 https://github.com/go-chi/chi/

Archived

dependency
Chroma
 https://github.com/alecthomas/chroma/

Archived

dependency
F3
 https://f3.forgefriends.org/
dependency
ForgeFed
 https://forgefed.org
dependency
garage
 https://git.deuxfleurs.fr/Deuxfleurs/garage
dependency
Gitea
 https://github.com/go-gitea/gitea

Archived

dependency
Golang
 https://go.dev/
Discussion
duplicate
This issue or pull request already exists
enhancement/feature
New feature
forgejo/accessibility
Accessibility (a11y)
forgejo/branding
Branding (logo, name, tagline etc.)
forgejo/ci
Forgejo Actions CI configuration
forgejo/commit-graph
The commit graph feature and page.
forgejo/documentation
forgejo/furnace cleanup
Keeping Forgejo in sync with its dependencies and contributing back to them

Archived

forgejo/i18n
t9n/translation, l10n/localization, and i18n/internationalization of Forgejo
forgejo/interop
Interoperability with other services: Webhooks, bridges, integrations
forgejo/moderation
Moderation
forgejo/privacy
Privacy first
forgejo/release
Release management
forgejo/scaling
Performance and scaling
forgejo/security
Security (please disclose responsibly)
forgejo/ui
User interface
Gain
High
 User research provides indicators that this would be good to have, interested contributors are encouraged to pick this.
Gain
Nice to have
 This is likely worth having, but the assumption is not backed by user research data (it might benefit a small amount of users only.) Unlikely to receive much attention, but feel free to pick.
Gain
Undefined
 Not enough information to assess the request's benefits. This issue may be closed if no gain is established: You can help by giving us more input.
Gain
Very High
 User research indicates that this is an important improvement for Forgejo users. Contributions very welcome!
good first issue
Optimal for first-timers! Make sure to look for further explanations and ask for help if needed. If you want, you can consider the person who added this label as a point of contact.
i18n/backport-stable
This PR needs to be backported to stable branch of Forgejo safely and manually, using a migration script.
impact
large
 Large impact: Potential data loss, many users affected, major degradation in UX.
impact
medium
 Medium impact: Several users affected, degradation in UX, workarounds might be available but inconvenient.
impact
small
 Small impact: No data loss, workarounds might be available, affects few users.
impact
unknown
 Report was not yet triaged to assess impact.
Incompatible license
This pull request contains changes that are not (yet) compatible with the current Forgejo license
issue
closed
 The issue was resolved in the repository of the dependency
issue
do-not-exist-yet
 An issue should be created in the respository of the dependency
issue
open
 An open issue exists in the upstream repository of the dependency
manual test
Pull requests that have been merged with a manual test

Archived

Manually tested during feature freeze
The manual test instructions were followed
OS
FreeBSD
 Specific to the FreeBSD Operating System
OS
Linux
 Specific to (GNU/)Linux Operating Systems
OS
macOS
 Specific to the MacOS Operating System
OS
Windows
 Specific to the Windows Operating System
problem
A user report about a problem. Needs to be triaged to find potential solutions.
QA
regression
found in the version of the milestone and not before
release blocker
Issues that must be fixed before the release can be published
Release Cycle
Feature Freeze
 Only bug fixes with automated tests (except for CSS/JavaScript)
release-blocker
v7.0
 Issues that must be fixed before Forgejo v7.0 can be released 17 April 2024

Archived

release-blocker
v7.0.1
 Issues that must be fixed before Forgejo v7.0.1 can be released

Archived

release-blocker
v7.0.2
 Issues that must be fixed before Forgejo v7.0.2 can be released

Archived

release-blocker
v7.0.3
 Issues that must be fixed before Forgejo v7.0.3 can be released

Archived

release-blocker
v7.0.4
 Issues that must be fixed before Forgejo v7.0.4 can be released

Archived

release-blocker
v8.0.0
 Issues that must be fixed before Forgejo v8.0.0 can be released

Archived

release-blocker/v9.0.0
Issues that must be fixed before Forgejo v9.0.0 can be released

Archived

run-all-playwright-tests
Add this label to a PR to run all playwright tests manually.
run-end-to-end-tests
Trigger additional tests on the PR when it is ready to be merged
stage
2-research
 Needs research to moe on.
stage
3-design
 Needs design to move on
stage
4-implementation
 This is actionable and waits for implementation
test
manual
 manual testing has been documented
test
needed
 test should be added
test
needs-help
 help needed to add a test
test
not-needed
 no additional test is needed
test
present
 test has been added
untested
Pull requests that have been merged with no test and submitted as is to the dependency where they belong

Archived

User research - time-tracker
Time tracking feature for issues and the JS stopwatch.
valuable code
This PR was closed because the implementation is incomplete
worth a release-note
Add this PR to the release notes
User research - Accessibility
Requires input about accessibility features, likely involves user testing.
User research - Blocked
Do not pick as-is! We are happy if you can help, but please coordinate with ongoing redesign in this area.
User research - Community
Community features, such as discovering other people's work or otherwise feeling welcome on a Forgejo instance.
User research - Config (instance)
Instance-wide configuration, authentication and other admin-only needs.
User research - Errors
How to deal with errors in the application and write helpful error messages.
User research - Filters
How filter and search is being worked with.
User research - Future backlog
The issue might be inspiring for future design work.
User research - Git workflow
AGit, fork-based and new Git workflow, PR creation etc
User research - Labels
Active research about Labels
User research - Moderation
Moderation Featuers for Admins are undergoing active User Research
User research - Needs input
Use this label to let the User Research team know their input is requested.
User research - Notifications/Dashboard
Research on how users should know what to do next.
User research - Rendering
Text rendering, markup languages etc
User research - Repo creation
Active research about the New Repo dialog.
User research - Repo units
The repo sections, disabling them and the "Add more" button.
User research - Security
User research - Settings (in-app)
How to structure in-app settings in the future?
No labels
arch
riscv64
backport/v1.19
backport/v1.20
backport/v1.21/forgejo
backport/v10.0/forgejo
backport/v11.0/forgejo
backport/v12.0/forgejo
backport/v13.0/forgejo
backport/v14.0/forgejo
backport/v15.0/forgejo
backport/v16.0/forgejo
backport/v7.0/forgejo
backport/v8.0/forgejo
backport/v9.0/forgejo
breaking
bug
bug
confirmed
bug
duplicate
bug
needs-more-info
bug
new-report
bug
reported-upstream
code/actions
code/api
code/auth
code/auth/faidp
code/auth/farp
code/email
code/federation
code/git
code/migrations
code/packages
code/wiki
database
MySQL
database
PostgreSQL
database
SQLite
dependency-upgrade
dependency
Chi
dependency
Chroma
dependency
F3
dependency
ForgeFed
dependency
garage
dependency
Gitea
dependency
Golang
Discussion
duplicate
enhancement/feature
forgejo/accessibility
forgejo/branding
forgejo/ci
forgejo/commit-graph
forgejo/documentation
forgejo/furnace cleanup
forgejo/i18n
forgejo/interop
forgejo/moderation
forgejo/privacy
forgejo/release
forgejo/scaling
forgejo/security
forgejo/ui
Gain
High
Gain
Nice to have
Gain
Undefined
Gain
Very High
good first issue
i18n/backport-stable
impact
large
impact
medium
impact
small
impact
unknown
Incompatible license
issue
closed
issue
do-not-exist-yet
issue
open
manual test
Manually tested during feature freeze
OS
FreeBSD
OS
Linux
OS
macOS
OS
Windows
problem
QA
regression
release blocker
Release Cycle
Feature Freeze
release-blocker
v7.0
release-blocker
v7.0.1
release-blocker
v7.0.2
release-blocker
v7.0.3
release-blocker
v7.0.4
release-blocker
v8.0.0
release-blocker/v9.0.0
run-all-playwright-tests
run-end-to-end-tests
stage
2-research
stage
3-design
stage
4-implementation
test
manual
test
needed
test
needs-help
test
not-needed
test
present
untested
User research - time-tracker
valuable code
worth a release-note
User research - Accessibility
User research - Blocked
User research - Community
User research - Config (instance)
User research - Errors
User research - Filters
User research - Future backlog
User research - Git workflow
User research - Labels
User research - Moderation
User research - Needs input
User research - Notifications/Dashboard
User research - Rendering
User research - Repo creation
User research - Repo units
User research - Security
User research - Settings (in-app)
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo/forgejo!12976
Reference in a new issue
forgejo/forgejo
No description provided.
Delete branch "birdgoose/forgejo:12975-prevent-url-change-when-watching-or-starring"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?