[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #21175 [Metrics/Metrics website]: Line breaks separate checkboxes from labels in userstats-bridge-transport.html



#21175: Line breaks separate checkboxes from labels in userstats-bridge-
transport.html
-------------------------------------+------------------------------
 Reporter:  dcf                      |          Owner:  metrics-team
     Type:  defect                   |         Status:  new
 Priority:  Medium                   |      Milestone:
Component:  Metrics/Metrics website  |        Version:
 Severity:  Normal                   |     Resolution:
 Keywords:                           |  Actual Points:
Parent ID:                           |         Points:
 Reviewer:                           |        Sponsor:
-------------------------------------+------------------------------
Description changed by dcf:

Old description:

> The checkboxes and labels wrap such that the checkbox sometimes is left
> at the end of a line, separated from its label:
>
> [[Image(transport-wrap.png)]]
>
> This is the way it's currently formatted in website/web/WEB-
> INF/graph.jsp:
> {{{
> <input type="checkbox" name="transport" value="${row[0]}"
> id="transport_${row[0]}"${row[1]}>
> <label class="checkbox-label" for="transport_${row[0]}">${row[2]}</label>
> }}}
>
> I did some experiments in the DOM editor and this fixes the problem and
> looks good with the current CSS (make `input` a child of `label` rather
> than use a `for` attribute):
> {{{
> <label class="checkbox-label">
>   <input type="checkbox" name="transport" value="${row[0]}"
> id="transport_${row[0]}"${row[1]}>
> </label>
> }}}
>
> (Actually, the percentile checkboxes already work that way:)
> {{{
> <label class="checkbox-label">
>   <input type="checkbox" name="n" value="${row[0]}"${row[1]}> ${row[0]}
> </label>
> }}}

New description:

 The checkboxes and labels wrap such that the checkbox sometimes is left at
 the end of a line, separated from its label:

 [[Image(transport-wrap.png)]]

 This is the way it's currently formatted in website/web/WEB-INF/graph.jsp:
 {{{
 <input type="checkbox" name="transport" value="${row[0]}"
 id="transport_${row[0]}"${row[1]}>
 <label class="checkbox-label" for="transport_${row[0]}">${row[2]}</label>
 }}}

 I did some experiments in the DOM editor and this fixes the problem and
 looks good with the current CSS (make `input` a child of `label` rather
 than use a `for` attribute):
 {{{
 <label class="checkbox-label">
   <input type="checkbox" name="transport" value="${row[0]}"
 id="transport_${row[0]}"${row[1]}> ${row[2]}
 </label>
 }}}

 (Actually, the percentile checkboxes already work that way:)
 {{{
 <label class="checkbox-label">
   <input type="checkbox" name="n" value="${row[0]}"${row[1]}> ${row[0]}
 </label>
 }}}

--

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21175#comment:1>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs