Skip to content

Commit f174236

Browse files
committed
dashboard/app: test form get
1 parent fd497a2 commit f174236

File tree

2 files changed

+5
-23
lines changed

2 files changed

+5
-23
lines changed

dashboard/app/static/coverage.js

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -17,24 +17,6 @@ function initUpdateForm(){
1717
$('#target-subsystem').val(curUrlParams.get('subsystem'));
1818
$('#target-manager').val(curUrlParams.get('manager'));
1919
$("#only-unique").prop("checked", curUrlParams.get('subsystem') == "1");
20-
21-
$(document).ready(function() {
22-
$("#updateButton").click(function(event) {
23-
event.preventDefault(); // don't submit the form
24-
25-
var period = $("#target-period").val();
26-
var subsystem = $("#target-subsystem").val();
27-
var manager = $("#target-manager").val();
28-
var onlyUnique = $("#only-unique").prop("checked") ? "1" : "0";
29-
30-
var newLocation = new URL(window.location.href);
31-
newLocation.searchParams.set("period", period);
32-
newLocation.searchParams.set("subsystem", subsystem);
33-
newLocation.searchParams.set("manager", manager);
34-
newLocation.searchParams.set("unique-only", onlyUnique);
35-
window.location.href = newLocation.toString();
36-
});
37-
});
3820
}
3921

4022
$(document).ready(initUpdateForm);

pkg/cover/templates/heatmap.html

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@
105105

106106
{{ define "body" }}
107107
<div style="display:inline-block">
108-
<form>
108+
<form method="get">
109109
<div style="display:inline-block; vertical-align: top">
110110
<label for="target-period">Periods:</label>
111111
<br>
@@ -116,26 +116,26 @@
116116
<label for="only-unique">Only unique:</label>
117117
</div>
118118
<div style="display:inline-block; vertical-align: top">
119-
<select id="target-period">
119+
<select id="target-period" name="period">
120120
<option value="month">Month</option>
121121
<option value="day">Day</option>
122122
</select>
123123
<br>
124-
<select id="target-subsystem">
124+
<select id="target-subsystem" name="subsystem">
125125
<option value="">*</option>
126126
{{ range $ss := .Subsystems }}
127127
<option value="{{ $ss }}">{{ $ss }}</option>
128128
{{ end }}
129129
</select>
130130
<br>
131-
<select id="target-manager">
131+
<select id="target-manager" name="manager">
132132
<option value="">*</option>
133133
{{ range $manager := .Managers }}
134134
<option value="{{ $manager }}">{{ $manager }}</option>
135135
{{ end }}
136136
</select>
137137
<br>
138-
<input type="checkbox" id="only-unique" disabled>
138+
<input type="checkbox" id="only-unique" name="unique-only" disabled>
139139
</div>
140140
<br>
141141
<button id="updateButton">Update</button>

0 commit comments

Comments
 (0)