From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4/4] docs/website: cosmetic changes
Date: Sat, 5 Dec 2015 18:21:12 +0100 [thread overview]
Message-ID: <56631D08.2050105@mind.be> (raw)
In-Reply-To: <1449213674-11333-5-git-send-email-angelo.compagnucci@gmail.com>
On 04-12-15 08:21, Angelo Compagnucci wrote:
> * Smaller images in download and docs pages
> * Fixes to links in docs
> * Minor fixings in index
This should probably be three patches, maybe even more. Also, preferably say
something more specific than "Minor fixes".
>
> Signed-off-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
> ---
> docs/website/css/main.css | 15 ++++++++++-----
> docs/website/docs.html | 43 ++++++++++++++++++++++++-------------------
> docs/website/download.html | 17 ++++++++---------
> docs/website/header.html | 12 ++++++------
> 4 files changed, 48 insertions(+), 39 deletions(-)
>
> diff --git a/docs/website/css/main.css b/docs/website/css/main.css
> index db9c4c9..57009cf 100644
> --- a/docs/website/css/main.css
> +++ b/docs/website/css/main.css
> @@ -207,16 +207,21 @@ i {
> .flip-container {
> perspective: 1000;
> }
> - /* flip the pane when hovered */
> - .flip-container:hover .flipper, .flip-container.hover .flipper {
> - transform: rotateY(180deg);
> - }
> +/* flip the pane when hovered */
> +.flip-container:hover .flipper, .flip-container.hover .flipper {
> + transform: rotateY(180deg);
> +}
Whitespace change -> separate patch.
>
> -.flip-container, .front, .back {
> +.flip-container, .flip-container .front, .flip-container .back {
I'm not a CSS expert (to say the least), but isn't this equivalent to
.flip-container {
because the two other specifiers are a subset of the first one?
> width: 180px;
> height: 180px;
> }
>
> +.flip-container-small, .flip-container-small .front, .flip-container-small .back {
Same here.
Also, wouldn't it be better to add .flip-container-small to the other two
.flip-container specifications, so that you can remove the flip-container class
from the classes in the HTML files?
> + width: 120px;
> + height: 120px;
> +}
> +
> /* flip speed goes here */
> .flipper {
> transition: 0.6s;
> diff --git a/docs/website/docs.html b/docs/website/docs.html
> index 4b86383..f2a8597 100644
> --- a/docs/website/docs.html
> +++ b/docs/website/docs.html
> @@ -10,60 +10,65 @@
> change/extend/fix things. It is available as:</p>
> <div class="row mt centered">
> <div class="col-sm-4">
> - <div class="flip-container center-block"
> + <div class="flip-container flip-container-small center-block"
> ontouchstart="this.classList.toggle('hover');">
> <div class="flipper">
> <div class="front">
> <a href="/manual.html">
> - <img src="images/browser.png" width="180" alt="" />
> + <img src="images/browser.png" alt="" />
> </a>
> </div>
> <div class="back">
> <a href="/manual.html">
> - <img src="images/browser.png" width="180" alt="" />
> + <img src="images/browser.png" alt="" />
> </a>
> </div>
> </div>
> </div>
> - <h3>HTML
> - <br />
> - <a href="/manual.html">Stable</a>|
> + <h3>HTML</h3>
> + <h4><a href="/manual.html">Stable</a>|
> <a href="http://nightly.buildroot.org/manual.html">
> - Nightly</a></h3>
> + Nightly</a></h4>
> </div>
> <div class="col-sm-4">
> - <div class="flip-container center-block"
> + <div class="flip-container flip-container-small center-block"
> ontouchstart="this.classList.toggle('hover');">
> <div class="flipper">
> <div class="front">
> - <img src="images/pdf-icon.png" width="180" alt="" />
> + <a href="downloads/manual/manual.pdf">
> + <img src="images/pdf-icon.png" alt="" />
> + </a>
> </div>
> <div class="back">
> - <img src="images/pdf-icon.png" width="180" alt="" />
> + <a href="downloads/manual/manual.pdf">
> + <img src="images/pdf-icon.png" alt="" />
> + </a>
> </div>
> </div>
> </div>
> - <h3>PDF
> - <br />
> - <a href="/downloads/manual/manual.pdf">Stable</a>|
> + <h3>PDF</h3>
> + <h4><a href="/downloads/manual/manual.pdf">Stable</a>|
> <a href="http://nightly.buildroot.org/manual.pdf">
> Nightly</a></h3>
> </div>
> <div class="col-sm-4">
> - <div class="flip-container center-block"
> + <div class="flip-container flip-container-small center-block"
> ontouchstart="this.classList.toggle('hover');">
> <div class="flipper">
> <div class="front">
> - <img src="images/text.png" width="180" alt="" />
> + <a href="downloads/manual/manual.text">
> + <img src="images/text.png" alt="" />
> + </a>
> </div>
> <div class="back">
> - <img src="images/text.png" width="180" alt="" />
> + <a href="downloads/manual/manual.text">
> + <img src="images/text.png" alt="" />
> + </a>
> </div>
> </div>
> </div>
> - <h3>ASCII
> - <br />
> - <a href="/downloads/manual/manual.text">Stable</a>|
> + <h3>ASCII</h3>
> + <h4><a href="/downloads/manual/manual.text">Stable</a>|
> <a href="http://nightly.buildroot.org/manual.text">
> Nightly</a></h3>
> </div>
> diff --git a/docs/website/download.html b/docs/website/download.html
> index 4ff397d..ea77b25 100644
> --- a/docs/website/download.html
> +++ b/docs/website/download.html
> @@ -9,17 +9,17 @@
> <b>2015.11</b></h3>
> <div class="row mt centered">
> <div class="col-sm-6">
> - <div class="flip-container center-block"
> + <div class="flip-container flip-container-small center-block"
> ontouchstart="this.classList.toggle('hover');">
> <div class="flipper">
> <div class="front">
> <a href="/downloads/buildroot-2015.11.tar.gz">
> - <img src="images/zip.png" width="180" alt="" />
> + <img src="images/zip.png" alt="" />
> </a>
> </div>
> <div class="back">
> <a href="/downloads/buildroot-2015.11.tar.gz">
> - <img src="images/zip.png" width="180" alt="" />
> + <img src="images/zip.png" alt="" />
> </a>
> </div>
> </div>
> @@ -34,17 +34,17 @@
> </p>
> </div>
> <div class="col-sm-6">
> - <div class="flip-container center-block"
> + <div class="flip-container flip-container-small center-block"
> ontouchstart="this.classList.toggle('hover');">
> <div class="flipper">
> <div class="front">
> <a href="/downloads/buildroot-2015.11.tar.bz2">
> - <img src="images/package.png" width="180" alt="" />
> + <img src="images/package.png" alt="" />
> </a>
> </div>
> <div class="back">
> <a href="/downloads/buildroot-2015.11.tar.bz2">
> - <img src="images/package.png" width="180" alt="" />
> + <img src="images/package.png" alt="" />
> </a>
> </div>
> </div>
> @@ -153,9 +153,8 @@
> can update your source tree at any time so it is in sync
> with the latest and greatest by entering your buildroot
> directory and running the command:</p>
> - <pre>
> - git pull
> -
> +<pre>
> +git pull
This bit should have been part of the tidy patch - and tidy shouldn't add
indentation here...
> </pre>
> <p>Because you've only been granted anonymous access to
> the tree, you won't be able to push your changes to the
> diff --git a/docs/website/header.html b/docs/website/header.html
> index 69f4203..c348ebb 100644
> --- a/docs/website/header.html
> +++ b/docs/website/header.html
> @@ -36,31 +36,31 @@
> <ul class="nav navbar-nav navbar-right">
> <li>
> <a href="/news.html">
> - <span class="glyphicon glyphicon-star"></span>News</a>
> + <span class="glyphicon glyphicon-star"></span> News</a>
I'm not sure I agree with this. Currently the icon and the text are nice and
tight together, adding a space here makes the icon come closer to the text left
of it.
Regards,
Arnout
> </li>
> <li>
> <a href="/docs.html">
> <span class="glyphicon glyphicon-book">
> - </span>Documentation</a>
> + </span> Documentation</a>
> </li>
> <li>
> <a href="/support.html">
> <span class="glyphicon glyphicon-question-sign">
> - </span>Support</a>
> + </span> Support</a>
> </li>
> <li>
> <a href="/contribute.html">
> - <span class="glyphicon glyphicon-share"></span>Contribute</a>
> + <span class="glyphicon glyphicon-share"></span> Contribute</a>
> </li>
> <li>
> <a href="/sponsors.html">
> - <span class="glyphicon glyphicon-usd"></span>Sponsors</a>
> + <span class="glyphicon glyphicon-usd"></span> Sponsors</a>
> </li>
> <li>
> <a id="menubutton" class="btn btn-success btn-shrink"
> href="/download.html">
> <span class="glyphicon glyphicon-cloud-download"></span>
> - <span class="text-shrink">Download</span>
> + <span class="text-shrink"> Download</span>
> </a>
> </li>
> </ul>
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
next prev parent reply other threads:[~2015-12-05 17:21 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-04 7:21 [Buildroot] [PATCH 0/4] website caring Angelo Compagnucci
2015-12-04 7:21 ` [Buildroot] [PATCH 1/4] docs/website: fixing html tidy errors and warnings Angelo Compagnucci
2015-12-05 16:41 ` Arnout Vandecappelle
2015-12-04 7:21 ` [Buildroot] [PATCH 2/4] docs/website: html cleaning and indentation fixing Angelo Compagnucci
2015-12-05 16:51 ` Arnout Vandecappelle
2015-12-05 16:52 ` Arnout Vandecappelle
2015-12-05 16:56 ` Arnout Vandecappelle
2015-12-04 7:21 ` [Buildroot] [PATCH 3/4] docs/website: resizing images Angelo Compagnucci
2015-12-04 7:21 ` [Buildroot] [PATCH 4/4] docs/website: cosmetic changes Angelo Compagnucci
2015-12-05 17:21 ` Arnout Vandecappelle [this message]
2015-12-05 17:21 ` [Buildroot] [PATCH 0/4] website caring Arnout Vandecappelle
2015-12-07 21:00 ` Angelo Compagnucci
2015-12-17 12:32 ` Thomas Petazzoni
2015-12-17 23:06 ` Arnout Vandecappelle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56631D08.2050105@mind.be \
--to=arnout@mind.be \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.