Skip to content

WebUI: prefer for loop over Array.forEach method#23231

Merged
Chocobo1 merged 1 commit intoqbittorrent:masterfrom
Chocobo1:webui_foreach2
Sep 7, 2025
Merged

WebUI: prefer for loop over Array.forEach method#23231
Chocobo1 merged 1 commit intoqbittorrent:masterfrom
Chocobo1:webui_foreach2

Conversation

@Chocobo1
Copy link
Member

@Chocobo1 Chocobo1 commented Sep 5, 2025

These were missed in 6ac0c5a. Also refactor the code a bit.

These were missed in 6ac0c5a.
Also refactor the code a bit.
@Chocobo1 Chocobo1 added this to the 5.2 milestone Sep 5, 2025
@Chocobo1 Chocobo1 added WebUI WebUI-related issues/changes Code cleanup Clean up the code while preserving the same outcome labels Sep 5, 2025
@Chocobo1 Chocobo1 requested a review from a team September 5, 2025 13:30
@Chocobo1 Chocobo1 merged commit 5c0010a into qbittorrent:master Sep 7, 2025
15 checks passed
@Chocobo1 Chocobo1 deleted the webui_foreach2 branch September 7, 2025 08:16
Chocobo1 added a commit to Chocobo1/qBittorrent that referenced this pull request Sep 12, 2025
Chocobo1 added a commit that referenced this pull request Sep 14, 2025
* WebUI: prefer `classList.toggle()` over other pattern
  Addresses: #23231 (comment)
* WebUI: prefer using built-in objects Constructor
* WebUI: combine function calls

PR #23261.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code cleanup Clean up the code while preserving the same outcome WebUI WebUI-related issues/changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants