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

Re: [tor-bugs] #6355 [Metrics/Atlas]: Distinguish between running relays and non-running relays which last had the Running flag



#6355: Distinguish between running relays and non-running relays which last had
the Running flag
---------------------------+------------------------------
 Reporter:  karsten        |          Owner:  phw
     Type:  enhancement    |         Status:  needs_review
 Priority:  High           |      Milestone:
Component:  Metrics/Atlas  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+------------------------------

Comment (by cypherpunks):

 Replying to [comment:20 RaBe]:
 > Okay, so here's a new patch, that adds an "offline" label, greys out the
 flags, and uses "was" instead of "is" for each tooltip (which works on
 most of the texts - I didn't want us to have to maintain an extra text
 version for every tooltip).
 On hover the line under the nicknames runs till the offline label. The
 following patch fixes that.
 {{{
 diff --git a/templates/search/do.html b/templates/search/do.html
 index 1203947..2466c4e 100644
 --- a/templates/search/do.html
 +++ b/templates/search/do.html
 @@ -71,9 +71,7 @@
  <% _.each(relays, function(relay) { %>
              <tr class="<%= i % 2 == 0 ? 'even' : 'odd' %><%=
 (relay.get('running') === false) ? ' down' : '' %>">
                         <td>
 -            <a href="#details/<%= relay.get('fingerprint') %>">
 -            <%= _.escape(relay.get('nickname')) %>
 -            </a>
 +            <a href="#details/<%= relay.get('fingerprint') %>"><%=
 _.escape(relay.get('nickname')) %></a>
              <% if (relay.get('running') === false) { %><span class="label
 label-important">offline</span><% } %>
              </td>
                         <td><span data-filter="<%= relay.get('bandwidth')
 %>"><%= relay.get('bandwidth_hr') %></td>
 }}}
 The greyed out flags look good IMO. However, the simplistic replacement of
 'is' with 'was' leads to some weird sentences and, like you said, it only
 applies to most of the texts. This makes most sentences use the past tense
 but some use the present tense which is confusing.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/6355#comment:21>
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