git724/forgejo
1
0
Fork 0
Code Issues Pull requests 3 Projects 1 Releases Packages Wiki Activity

Remove jQuery .attr from the user search box (#29919)

Browse source 
- Switched from jQuery `.attr` to plain javascript `.getAttribute`
- Tested the user search box and it works as before

Signed-off-by: Yarden Shoham <git@yardenshoham.com>
(cherry picked from commit adc61c5d71651acc316bbc3a7fee6f2c0c60b06e)
This commit is contained in:
Yarden Shoham 2024-03-20 02:09:52 +02:00 • committed by Earl Warren
commit e358500519
No known key found for this signature in database
GPG key ID: 0579CB2928A78A00
1 changed files with 6 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

9
web_src/js/features/comp/SearchUserBox.js
View file

@ -5,9 +5,12 @@ const {appSubUrl} = window.config;
const looksLikeEmailAddressCheck = /^\S+@\S+$/; const looksLikeEmailAddressCheck = /^\S+@\S+$/;
export function initCompSearchUserBox() { export function initCompSearchUserBox() {
const $searchUserBox = $('#search-user-box'); const searchUserBox = document.getElementById('search-user-box');
const allowEmailInput = $searchUserBox.attr('data-allow-email') === 'true'; if (!searchUserBox) return;
const allowEmailDescription = $searchUserBox.attr('data-allow-email-description');
const $searchUserBox = $(searchUserBox);
const allowEmailInput = searchUserBox.getAttribute('data-allow-email') === 'true';
const allowEmailDescription = searchUserBox.getAttribute('data-allow-email-description') ?? undefined;
$searchUserBox.search({ $searchUserBox.search({
minCharacters: 2, minCharacters: 2,
apiSettings: { apiSettings: {