1
0
Fork 0
forgejo/web_src/js/modules/fomantic
silverwind a01387f5b1
Add form field id generation, remove duplicated ids (#30546)
Fixes: https://github.com/go-gitea/gitea/issues/30384

On repo settings page, there id `repo_name` was used 5 times on the same
page, some in modal and such. I think we are better off just
auto-generating these IDs in the future so that labels link up with
their form element.

Ideally this id generation would be done in backend in a subtemplate,
but seeing that we already have similar JS patches for checkboxes, I
took the easy path for now.

I also checked that these `#repo_name` were not in use in JS and the
only case where this id appears in JS is on the migration page where
it's still there.

---------

Co-authored-by: Giteabot <teabot@gitea.io>
(cherry picked from commit d4ec6b3d16496ce3b479d5a08f79823122dc2b7b)

Conflicts:
	- templates/repo/settings/options.tmpl
	  Conflict resolved by manually removing all `id` and `for`
	  attributes from elements that had `repo_name` as their id.
2024-04-22 08:09:00 +02:00
..
api.js Move some JS code from fomantic.js to standalone files (#27994) 2023-11-12 07:15:00 +00:00
aria.md Remove fomantic checkbox module (#30162) 2024-04-07 11:05:37 +02:00
base.js Add form field id generation, remove duplicated ids (#30546) 2024-04-22 08:09:00 +02:00
checkbox.js Add form field id generation, remove duplicated ids (#30546) 2024-04-22 08:09:00 +02:00
dropdown.js Remove jQuery class from the comment context menu (#30179) 2024-04-07 15:40:31 +02:00
form.js Add form field id generation, remove duplicated ids (#30546) 2024-04-22 08:09:00 +02:00
modal.js Remove jQuery .attr from the Fomantic modal cancel buttons (#30113) 2024-03-30 07:17:30 +01:00
transition.js Move some JS code from fomantic.js to standalone files (#27994) 2023-11-12 07:15:00 +00:00