~abcdw/rde-devel

rde: fonts: Add font-liberation by default. v1 APPLIED

Andrew Tropin: 2
 rde: fonts: Add font-liberation by default.
 rde: fonts: Add font-liberation by default.

 2 files changed, 8 insertions(+), 2 deletions(-)
IIUC, the choice is between usability and customizability / minimalism.

I haven't noticed any major problem with pdfs or sites, but if you do, I
think usability in this case is more important for now.

Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~abcdw/rde-devel/patches/44622/mbox | git am -3
Learn more about email & git

[PATCH] rde: fonts: Add font-liberation by default. Export this patch

--- 
I want to include font-liberation in the list of default fonts in
feature-fonts.  The pros are: proper rendering of some PDFs and some sites
using Times New Roman and other questionable fonts.  The cons are: some sites,
would pick Helevetica (or some other font) if available instead of user's
preffered font.

WDYT?

 src/rde/features/fontutils.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/rde/features/fontutils.scm b/src/rde/features/fontutils.scm
index 5250876e..f0b1c3e5 100644
--- a/src/rde/features/fontutils.scm
+++ b/src/rde/features/fontutils.scm
@@ -88,7 +88,10 @@
          (emacs-fontaine emacs-fontaine)
          (use-serif-for-variable-pitch? #t)
          (extra-fontaine-presets '())
          (extra-font-packages (list font-gnu-unifont)))
          ;; font-liberation needed for proper rendering of sites/pdfs using
          ;; Times, Helvetica, Nimbus.  Use pdffonts from poppler to check
          ;; what fonts are needed for pdf.
          (extra-font-packages (list font-gnu-unifont font-liberation)))
  "Configure fonts.  DEFAULT-FONT-SIZE will be used for making
font-monospace default value, and it will be ignored if
#:font-monospace argument is specified."
-- 
2.41.0

[PATCH v2] rde: fonts: Add font-liberation by default. Export this patch

---
Changes since v1:
Slightly updated comment.

 src/rde/features/fontutils.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/rde/features/fontutils.scm b/src/rde/features/fontutils.scm
index 5250876e..d6d41296 100644
--- a/src/rde/features/fontutils.scm
+++ b/src/rde/features/fontutils.scm
@@ -88,7 +88,10 @@
          (emacs-fontaine emacs-fontaine)
          (use-serif-for-variable-pitch? #t)
          (extra-fontaine-presets '())
          (extra-font-packages (list font-gnu-unifont)))
          ;; font-liberation needed for proper rendering of sites/pdfs using
          ;; Times, Helvetica, Nimbus.  Use pdffonts from poppler package to
          ;; check what fonts are needed for pdf.
          (extra-font-packages (list font-gnu-unifont font-liberation)))
  "Configure fonts.  DEFAULT-FONT-SIZE will be used for making
font-monospace default value, and it will be ignored if
#:font-monospace argument is specified."
-- 
2.41.0
IIUC, the choice is between usability and customizability / minimalism.

I haven't noticed any major problem with pdfs or sites, but if you do, I
think usability in this case is more important for now.