#+title: FFS code review with Protesilaos
#+date: 2026-05-07 Thu 22:10:33 -0400
#+options: num:nil toc:nil
#+macro: kbd @@html:$1@@
#+html_head:
In the recent weeks I've been engaging [[https://protesilaos.com/coach/][Prot as an Emacs coach]] to help
with doing review passes over my upcoming [[https://git.kelar.org/~bandali/ffs][~ffs~]] package as I work on
polishing and documenting it in preparation for offering it for
inclusion in GNU ELPA.
Yesterday we had our second session focused on ~ffs~, which I recorded
and share publicly with everyone with Prot's permission, so that
others can also benefit from Prot's insights and experience as we
discuss various aspects of Emacs package development with the concrete
example of ~ffs~.
Here is the video recording of our session:
#+attr_html: :preload metadata :controls controls :width 720
#+attr_html: :src https://archive.org/download/ffs-code-review-prot-20260506/ffs-code-review-prot-20260506.mp4
#+attr_html: :poster ffs-code-review-prot-poster.jpg
#+begin_video
#+html:
Sorry, this embedded video will not work,
because your web browser does not support HTML5 video.\\
[[https://archive.org/download/ffs-code-review-prot-20260506/ffs-code-review-prot-20260506.mp4][[ please watch the video in your favourite streaming media player ]]]
#+end_video
You can [[https://archive.org/details/ffs-code-review-prot-20260506][view]] or [[https://archive.org/download/ffs-code-review-prot-20260506/ffs-code-review-prot-20260506.mkv][download the full-resolution video]] from the Internet
Archive.
I addressed most of Prot's feedback about ~ffs~ from our first
session, and I'll be working on the changes we discussed in this
session in the next days.
In the last third of the video we switched topics to discuss a few
Emacs-related tangents including adding a 'padding' effect for the
mode line and its constructs, and distilling and separating the
easily-reusable package-like parts of one's Emacs configuration from
the actual configuration of those parts (e.g. the distinction of
=prot-lisp= and =prot-emacs-modules= in [[https://protesilaos.com/emacs/dotemacs][Prot's Emacs configuration]]).
For mode line padding, here is the snippet I'm using with Prot's
~doric-themes~:
#+begin_src elisp
(doric-themes-with-colors
(custom-set-faces
`(mode-line
((t :box (:line-width 6 :color ,bg-shadow-intense))))
`(mode-line-inactive
((t :box (:line-width 6 :color ,bg-shadow-subtle))))
`(mode-line-highlight
((t :box (:color ,bg-shadow-intense))))))
#+end_src
Take care, and so long for now.
# Local Variables:
# org-html-postamble-format: (("en" "
#