From: Sebastian Urban <mrsebastianurban@gmail.com>
To: 47671@debbugs.gnu.org
Subject: bug#47671: Small tweaks & fixes for Emacs/GNU ELPA websites
Date: Fri, 9 Apr 2021 12:55:56 +0200 [thread overview]
Message-ID: <b9525124-8650-de65-aae2-3c45c6138ab1@gmail.com> (raw)
Hello!
First, GNU ELPA website. According to the LAYOUT.CSS
(http://git.savannah.gnu.org/cgit/emacs/elpa.git/plain/html/layout.css):
body {
font-family: "fira sans", sans;
(...)
}
pre {
font-family: "Fira Mono";
(...)
}
Website should set Fira fonts, but if external fonts are not allowed,
it loads the default font (usually a serif font, e.g. Times New Roman
on Windows) for everything, because for example Firefox expects
"sans-serif" instead "sans", while "pre" section is missing
"monospace". Additionally, ".tt, code" don't have defined fonts, hence
on the main page "Fira Sans" is used (for keys and commands) instead of
monospace font.
The DIFF:
--- 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;
Second, "A Guided Tour of Emacs" website. According to the LAYOUT.CSS
(https://www.gnu.org/software/emacs/layout.css):
pre {
font-family: monospace;
(...)
font-size: 1.2em;
}
This is used in the "Download" section, actually. It's missing "Fira
Mono", and I don't think font-size is needed there at all.
body {
font-family: "fira sans", sans;
(...)
}
Basically the same problem, as above (1st point).
tt, code, kbd {
background: #efeadc;
border: 1px solid #ccc;
padding: 2px 6px 0px;
border-radius: 4px;
/* color: #ddd; */
}
No font-family defined, commands are printed with "Fira Sans"/"the
default font" depending on the permission.
The DIFF:
--- 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;
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.
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>
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;
}
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.
--
S. U.
next reply other threads:[~2021-04-09 10:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-09 10:55 Sebastian Urban [this message]
2021-04-20 14:11 ` bug#47671: Small tweaks & fixes for Emacs/GNU ELPA websites Stefan Kangas
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b9525124-8650-de65-aae2-3c45c6138ab1@gmail.com \
--to=mrsebastianurban@gmail.com \
--cc=47671@debbugs.gnu.org \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
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.