thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

Remove jQuery .attr from the archive download and compare page branch selector (#29918)

Browse source 
- Switched from jQuery `.attr` to plain javascript `.getAttribute`
- Tested the archive download and compare page branch selector
functionality and it works as before

Signed-off-by: Yarden Shoham <git@yardenshoham.com>
(cherry picked from commit dd043854ee8963057daa7b835fc700731ae5fb9e)
This commit is contained in:
Yarden Shoham 2024-03-20 02:04:24 +02:00 committed by Earl Warren
parent 32cef22bfd
commit 872287c380
No known key found for this signature in database
GPG key ID: 0579CB2928A78A00
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
web_src/js/features/repo-common.js
View file

@ -34,7 +34,7 @@ async function getArchive($target, url, first) {
export function initRepoArchiveLinks() { export function initRepoArchiveLinks() {
$('.archive-link').on('click', function (event) { $('.archive-link').on('click', function (event) {
event.preventDefault(); event.preventDefault();
const url = $(this).attr('href'); const url = this.getAttribute('href');
if (!url) return; if (!url) return;
getArchive($(event.target), url, true); getArchive($(event.target), url, true);
}); });
@ -80,10 +80,10 @@ export function initRepoCommonFilterSearchDropdown(selector) {
fullTextSearch: 'exact', fullTextSearch: 'exact',
selectOnKeydown: false, selectOnKeydown: false,
onChange(_text, _value, $choice) { onChange(_text, _value, $choice) {
if ($choice.attr('data-url')) { if ($choice[0].getAttribute('data-url')) {
window.location.href = $choice.attr('data-url'); window.location.href = $choice[0].getAttribute('data-url');
} }
}, },
message: {noResults: $dropdown.attr('data-no-results')}, message: {noResults: $dropdown[0].getAttribute('data-no-results')},
}); });
} }