Remove jQuery .attr from the branch/tag selector (#30010)

yardenshoham · web-flow · commit bc92478575d9 · 2024-03-22T23:26:56.000Z
- Switched from jQuery `.attr` to plain javascript `.setAttribute`
- Tested the cherry-pick from the branch/tag selector and it works as
before

Signed-off-by: Yarden Shoham &lt;git@yardenshoham.com&gt;
diff --git a/web_src/js/components/RepoBranchTagSelector.vue b/web_src/js/components/RepoBranchTagSelector.vue
@@ -83,7 +83,7 @@ const sfc = {
         this.isViewBranch = false;
         this.$refs.dropdownRefName.textContent = item.name;
         if (this.setAction) {
-          $(`#${this.branchForm}`).attr('action', url);
+          document.getElementById(this.branchForm)?.setAttribute('action', url);
         } else {
           $(`#${this.branchForm} input[name="refURL"]`).val(url);
         }

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ const sfc = {`
`83`	`83`	`this.isViewBranch = false;`
`84`	`84`	`this.$refs.dropdownRefName.textContent = item.name;`
`85`	`85`	`if (this.setAction) {`
`86`		- $(`#${this.branchForm}`).attr('action', url);
	`86`	`+ document.getElementById(this.branchForm)?.setAttribute('action', url);`
`87`	`87`	`} else {`
`88`	`88`	$(`#${this.branchForm} input[name="refURL"]`).val(url);
`89`	`89`	`}`