unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefan@marxist.se>
To: Sebastian Urban <mrsebastianurban@gmail.com>
Cc: 47671@debbugs.gnu.org
Subject: bug#47671: Small tweaks & fixes for Emacs/GNU ELPA websites
Date: Tue, 20 Apr 2021 09:11:00 -0500	[thread overview]
Message-ID: <CADwFkmmE4e-J48Tu_qHLcvAnJH_Nz9aKmcsUiizosGcGVPuJiA@mail.gmail.com> (raw)
In-Reply-To: <b9525124-8650-de65-aae2-3c45c6138ab1@gmail.com> (Sebastian Urban's message of "Fri, 9 Apr 2021 12:55:56 +0200")

tags 47671 fixed
close 47671
thanks

Sebastian Urban <mrsebastianurban@gmail.com> writes:

> First, GNU ELPA website. According to the LAYOUT.CSS
> (http://git.savannah.gnu.org/cgit/emacs/elpa.git/plain/html/layout.css):

Thanks for these fixes, they are now applied.  Please verify that
everything looks okay but note that it might take a day or two for the
websites to update.  See below for details.

> --- old/elpa/html/layout.css	2021-04-08 14:13:47.982155600 +0200
> +++ new/elpa/html/layout.css	2021-04-09 12:27:15.610503900 +0200
> @@ -47,7 +47,7 @@
>  /* End of reset */
>    body {
> -  font-family: "fira sans", sans;
> +  font-family: "Fira Sans", sans-serif;
>    line-height: 1.5em;
>    font-size: 1.2em;
>    color: #333;
> @@ -92,6 +92,7 @@
>  }
>    tt, code {
> +  font-family: "Fira Mono", monospace;
>    background-color: rgb(238, 238, 238);
>    border: 1px solid rgb(187, 187, 187);
>    padding: 0px 6px;
> @@ -99,7 +100,7 @@
>  }
>    pre {
> -  font-family: "Fira Mono";
> +  font-family: "Fira Mono", monospace;
>    font-size: 14px;
>    background: #fafafa;
>    padding: 15px;

Applied.

> --- old/emacs/layout.css	2017-10-13 11:34:01.000000000 +0200
> +++ new/emacs/layout.css	2021-04-08 23:15:06.111355600 +0200
> @@ -59,7 +59,7 @@
>  }
>    pre {
> -  font-family: monospace;
> +  font-family: "Fira Mono", monospace;
>    padding: 10px;
>    background: #efeadc;
>    margin: 20px 0px;
> @@ -71,7 +71,7 @@
>  /* End of reset */
>    body {
> -  font-family: "fira sans", sans;
> +  font-family: "Fira Sans", sans-serif;
>    line-height: 1.5em;
>    font-size: 1.1em;
>    color: #4E4742;
> @@ -135,6 +135,7 @@
>  }
>    tt, code, kbd {
> +  font-family: "Fira Mono", monospace;
>    background: #efeadc;
>    border: 1px solid #ccc;
>    padding: 2px 6px 0px;

Applied.

> Finally, there is an "inline" style for "kbd" inside INDEX.HTML
> (view-source:https://www.gnu.org/software/emacs/tour/index.html):
>
> kbd {
>   font-size: larger;
>   background: #ffe;
> }
>
> It makes "kbd" elements, well, larger. I just think it looks better
> without it, but this is just my opinion.

Yup, applied.

> Third, there are leftovers after bug#43000:
>
> --- old/emacs/emacs.html	2021-03-25 13:18:26.000000000 +0100
> +++ new/emacs/emacs.html	2021-04-08 23:54:08.903893300 +0200
> @@ -231,7 +231,7 @@
>                          <li>New minor mode 'electric-quote-mode' for using curved quotes as you
>                              type.</li>
>                          <li>Character folding support in isearch.el.</li>
> -                        <li>New and improved facilities for inserting Unicode characters: <tt>C-x 8</tt> now has
> +                        <li>New and improved facilities for inserting Unicode characters: <kbd>C-x 8</kbd> now has
>                              shorthands for several chars, such as U+2010 (HYPHEN), U+2011 (NON-BREAKING HYPHEN), and U+2012 (FIGURE DASH).</li>
>  			<p>For more information, read the <a href="news/NEWS.25.1">News</a> file.</p>
>                  </div>

Applied.

> Unfortunately, due to background colour for the "Releases" section,
> visibility is rather bad. Possible solutions:
>
> A. Remove background colour from ".releases" (this one: "background:
> #F0EBDE;") AND set inline style for "kbd" like in the Emacs Tour (this
> also means more consistency for "kbd"), i.e.:
>
> kbd {
>   background: #ffe;
> }

Fixed this in a slightly different way: I styled ".releases kbd" in the
same way as ".releases tt" was previously.

> B. Only set inline style for "kbd" as above.
>
> --- old/emacs/documentation.html	2018-11-23 11:20:01.000000000 +0100
> +++ new/emacs/documentation.html	2021-04-09 00:31:23.789193400 +0200
> @@ -64,7 +64,7 @@
>                          Reference Manual</cite> and several other manuals documenting major
>                          modes and other optional features, can also be read online.  They are
>                          also distributed with Emacs in
> -                        <a href="/software/texinfo/">Info</a> format; type <tt>C-h i</tt> in
> +                        <a href="/software/texinfo/">Info</a> format; type <kbd>C-h i</kbd> in
>                          Emacs to view them.</p>
>                        <ul>
> @@ -95,7 +95,7 @@
>                          appear on the other as well.</p>
>                      <h2>Reporting bugs</h2>
>                      <p>To report bugs, or to contribute fixes and improvements, use the
> -                        built-in Emacs bug reporter (<tt>M-x report-emacs-bug</tt>) or send
> +                        built-in Emacs bug reporter (<kbd>M-x report-emacs-bug</kbd>) or send
>                          email to <a id="YouHelp" href=
>                                      "http://mail.gnu.org/mailman/listinfo/bug-gnu-emacs">bug-gnu-emacs@gnu.org</a>.
>                          You can browse our bug database at
>
> Here, for consistency with the Emacs Tour, inline "kbd" could be added
> as well.

Applied.





  reply	other threads:[~2021-04-20 14:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-09 10:55 bug#47671: Small tweaks & fixes for Emacs/GNU ELPA websites Sebastian Urban
2021-04-20 14:11 ` Stefan Kangas [this message]
2021-04-25 20:41   ` Sebastian Urban
2021-04-26  1:15     ` Stefan Kangas
2021-04-26 14:19       ` Sebastian Urban

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CADwFkmmE4e-J48Tu_qHLcvAnJH_Nz9aKmcsUiizosGcGVPuJiA@mail.gmail.com \
    --to=stefan@marxist.se \
    --cc=47671@debbugs.gnu.org \
    --cc=mrsebastianurban@gmail.com \
    /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.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).