1
1
mirror of https://github.com/go-gitea/gitea synced 2024-11-05 09:44:26 +00:00
gitea/routers/web/repo
Yarden Shoham 937e8b5514
Fix elipsis button not working if the last commit loading is deferred (#29544)
Before this change, if we had more than 200 entries being deferred in
loading, the entire table would get replaced thus losing any event
listeners attached to the elements within the table, such as the elipsis
button and commit list with tippy.

With this change we remove the previous javascript code that replaced
the table and use htmx to replace the table.

htmx attributes added:
- `hx-indicator="tr.notready td.message span"`: attach the loading
spinner to the files whose last commit is still being loaded
- `hx-trigger="load"` trigger the request-replace behavior as soon as
possible
- `hx-swap="morph"`: use the idiomorph morphing algorithm, this is the
thing that makes it so the elipsis button event listener is kept during
the replacement, fixing the bug because we don't actually replace the
table, only modifying it
- `hx-post="{{.LastCommitLoaderURL}}"`: make a post request to this url
to get the table with all of the commit information

As part of this change I removed the handling of partial replacement in
the case we have less than 200 "not ready" files. The first reason is
that I couldn't make htmx replace only a subset of returned elements,
the second reason is that we have a cache implemented in the backend
already so the only cost added is that we query the cache a few times
(which is sure to be populated due to the initial request), and the last
reason is that since the last refactor of this functionality that
removed jQuery we don't properly send the "not ready" entries as the
backend expects `FormData` with `f[]` and we send a JSON with `f` so we
always query for all rows anyway.

# Before

![before](https://github.com/go-gitea/gitea/assets/20454870/482ebfec-66c5-40cc-9c1e-e3b3bfe1bbc1)

# After

![after](https://github.com/go-gitea/gitea/assets/20454870/454c517e-3a4e-4006-a49f-99cc56e0fd60)

---------

Signed-off-by: Yarden Shoham <git@yardenshoham.com>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
2024-03-02 19:31:59 +00:00
..
actions remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
setting Move migration functions to services layer (#29497) 2024-03-01 02:23:00 +00:00
activity.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
attachment.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
blame.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
branch.go remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
cherry_pick.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
code_frequency.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
commit.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
compare.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
contributors.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
download.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
editor_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
editor.go Make PR form use toast to show error message (#29545) 2024-03-02 15:05:07 +00:00
find.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
githttp_test.go Fix http protocol auth (#27875) 2023-11-02 22:14:33 +08:00
githttp.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
helper_test.go Make web context initialize correctly for different cases (#26726) 2023-08-25 19:07:42 +08:00
helper.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_content_history.go Fix issue & comment history bugs (#29525) 2024-03-01 16:46:02 +00:00
issue_dependency.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_label_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_label.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_lock.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_pin.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_stopwatch.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_test.go Implement FSFE REUSE for golang files (#21840) 2022-11-27 18:20:29 +00:00
issue_timetrack.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue_watch.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
issue.go remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
main_test.go make writing main test easier (#27270) 2023-09-28 01:38:53 +00:00
middlewares.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
migrate.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
milestone.go remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
packages.go remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
patch.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
projects_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
projects.go remove util.OptionalBool and related functions (#29513) 2024-03-02 16:42:31 +01:00
pull_review_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
pull_review.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
pull.go Make PR form use toast to show error message (#29545) 2024-03-02 15:05:07 +00:00
recent_commits.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
release_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
release.go Fix a bug returning 404 when display a single tag with no release (#29466) 2024-03-02 14:03:39 +00:00
render.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
repo.go migrate some more "OptionalBool" to "Option[bool]" (#29479) 2024-02-29 18:52:49 +00:00
search.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
topic.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
treelist.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
view_test.go Implement FSFE REUSE for golang files (#21840) 2022-11-27 18:20:29 +00:00
view.go Fix elipsis button not working if the last commit loading is deferred (#29544) 2024-03-02 19:31:59 +00:00
wiki_test.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00
wiki.go Move context from modules to services (#29440) 2024-02-27 08:12:22 +01:00