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

Re: [tor-relays] Simplifying ExoneraTor



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 16/07/15 23:16, Joshua Lee Tucker wrote:
> Hi guys,

Hello Joshua,

> I've made some simple (improvements?) changes to the ExoneraTor
> page - I've basically made the "about" section a sticky footer (no
> JS) and spaced the content a little better. I've tested it across
> Firefox/Chrome/Safari/iOS and it looks good on all of them.

It does look great!

Can I ask you to explain the changes in more detail by going through
my questions in the diff below?  I'd want to know what things are
changed and why.  Did I mention that I'm not good at web development? :)

> Maybe we could also add some information about the expected IP and
> date formats in the whitespace, too.

Let's talk about that.  I'm also not sure if putting examples into the
two input text boxes is such a good idea.  I heard that it can confuse
people by thinking there's already an IP address entered, so why would
they have to put in another one, which is even more problematic on
browsers with date selector.  It also hides the fact that ExoneraTor
supports searches by IPv6 addresses.  Maybe we can put in different
placeholders and add some good explanations below the form.  I still
think that less is more with respect to text length, but if we can
come up with some smart text there, that might do it.

> The updated version can be found at:
> http://tucker.wales/tor/exonerator/
> 
> Please let me know what you think.

It's great!  Thanks for working on this!

So, here's the diff with my questions:

@@ -6,6 +6,7 @@
 <meta name="viewport" content="width=device-width, initial-scale=1">
 <title>ExoneraTor</title>
 <link rel="stylesheet" href="css/bootstrap.min.css">
+<link rel="stylesheet" href="css/styles.css">
 <link href="images/favicon.ico" type="image/x-icon" rel="icon">
 </head>
 <body>

Is it good practice to use a custom .css file in combination with
Bootstrap, or should we try to only use Bootstrap's classes,
properties, etc. to come up with the approximate design that we want?
 Or, would it be possible to stick with Bootstrap-only styles for now
until there's no other way than using our own style file to do the
thing we want?


@@ -17,10 +18,15 @@
 </div><!-- page-header -->
 </div><!-- col -->
 </div><!-- row -->
- -
 <div class="row">
 <div class="col-xs-12">
 <div class="text-center">
+<div class="row vbottom15">
+<div class="col-xs-12">
+<h4>Enter the details of the relay that you would like to check
below:</h4>
+</div>
+</div>
+<div class="row"></div>
 <form class="form-inline">
 <div class="form-group">
 <label for="inputIp" class="control-label">IP address</label>

What's the purpose of that last row there that doesn't come with any
columns?


@@ -35,6 +41,9 @@
 </div><!-- text-center -->
 </div><!-- col -->
 </div><!-- row -->
+</div><!-- container -->
+<footer>
+<div class="container">
 <div class="row">
 <div class="col-xs-6">
 <h3>About Tor</h3>

I guess the <footer> is what moves the footer to the bottom, together
with our styles.css, right?  Is that a common HTML thing, a
Bootstrap-specific thing, a custom tag that only works with our
styles.css, or how does this work?  Is this the most Bootstrap-y way
to do it?  (As you notice, I'm trying to stick to the defaults as long
as possible.)


@@ -54,7 +63,9 @@
 <p class="text-center small">"Tor" and the "Onion Logo" are <a
href="https://www.torproject.org/docs/trademark-faq.html.en";>registered
trademarks</a>
of The Tor Project, Inc.</p>
 </div><!-- col -->
 </div><!-- row -->
- -</div><!-- container -->
+</div>
+</footer>
 </body>
 </html>

Makes sense.  By the way, is it good practice to add these comments,
or is that just something that old people do who first learned HTML in
the late 90's and whose favorite HTML editor is vim?

Thanks a lot!

All the best,
Karsten


> Kind Regards,
> 
> Joshua Lee Tucker @tuckerwales
> 
> 
> 
> _______________________________________________ tor-relays mailing
> list tor-relays@xxxxxxxxxxxxxxxxxxxx 
> https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-relays
> 

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: GPGTools - http://gpgtools.org

iQEcBAEBAgAGBQJVqK5kAAoJEJD5dJfVqbCruzIIAM4FIR724JAOzpxqvSa8pg0v
KLOFd4IepvS8sCdp5Qz54nMBLE17peXO3joNN3yydXJveftqCf/gTgxjlACjSkNA
Ugp1r9gHr+Oldg7wocacySk6tGddCvb8rS27Zho9nOhXJTMawiwAe/wFOhsl/ZXv
G7svWNFFBBm6+2FjANis7uelGIfsCRQsBlDgku4XqvhwTlcV3S9Cta7SrO2RE6Er
iWkzZi1fbfJUsMI+qS0iYx0fWd1P9B+pDBF6N09w4HppSjk3eKncKkaG2SY6LiW9
TLItFL79Ka7elQNB3rxAhr0a4QdmzPzp2zgrew3gztSRqzU1c9pXpbs7qaUHc/k=
=6YUL
-----END PGP SIGNATURE-----
_______________________________________________
tor-relays mailing list
tor-relays@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-relays