From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: [PATCH] emacs: Add a name/version/synopsis heading in package-info buffers Date: Wed, 29 Oct 2014 17:25:33 +0100 Message-ID: <87d29akflu.fsf@gnu.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:49331) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjW3p-0003Wx-Tm for guix-devel@gnu.org; Wed, 29 Oct 2014 12:25:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XjW3g-0004LB-Be for guix-devel@gnu.org; Wed, 29 Oct 2014 12:25:25 -0400 Received: from hera.aquilenet.fr ([2a01:474::1]:44787) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjW3f-0004KS-Qh for guix-devel@gnu.org; Wed, 29 Oct 2014 12:25:16 -0400 List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Alex Kost Cc: Guix-devel --=-=-= Content-Type: text/plain Hello, What would think of a patch like this: --=-=-= Content-Type: text/x-patch Content-Disposition: inline diff --git a/emacs/guix-info.el b/emacs/guix-info.el index 551d79a..c3499e4e 100644 --- a/emacs/guix-info.el +++ b/emacs/guix-info.el @@ -198,13 +198,27 @@ ENTRIES should have a form of `guix-entries'." entries guix-info-delimiter)) +(defun guix-info-insert-heading (entry) + "Insert the heading for ENTRY, showing its name, version, and +synopsis." + (guix-format-insert (concat (guix-get-key-val entry 'name) + " " (guix-get-key-val entry 'version) + "\n") + 'guix-package-info-heading) + (guix-format-insert (guix-get-key-val entry 'synopsis) + 'guix-package-info-synopsis) + (guix-format-insert "\n\n")) + (defun guix-info-insert-entry (entry entry-type &optional indent-level) "Insert ENTRY of ENTRY-TYPE into the current info buffer. If INDENT-LEVEL is non-nil, indent displayed information by this number of `guix-info-indent' spaces." (let ((region-beg (point))) + (guix-info-insert-heading entry) (mapc (lambda (param) - (guix-info-insert-param param entry entry-type)) + ;; The name, version, and synopsis are already displayed. + (unless (memq param '(name version synopsis)) + (guix-info-insert-param param entry entry-type))) (guix-info-get-displayed-params entry-type)) (when indent-level (indent-rigidly region-beg (point) @@ -376,6 +390,12 @@ See `insert-text-button' for the meaning of PROPERTIES." (guix-define-buffer-type info package :required (id installed non-unique)) +(defface guix-package-info-heading + '((((type tty pc) (class color)) :weight bold) + (t :height 1.6 :weight bold :inherit variable-pitch)) + "Face for package name and version headings." + :group 'guix-package-info) + (defface guix-package-info-name '((t :inherit font-lock-keyword-face)) "Face used for a name of a package." @@ -392,7 +412,8 @@ See `insert-text-button' for the meaning of PROPERTIES." :group 'guix-package-info) (defface guix-package-info-synopsis - '((t :inherit font-lock-doc-face)) + '((((type tty pc) (class color)) :weight bold) + (t :height 1.1 :weight bold :inherit variable-pitch)) "Face used for a synopsis of a package." :group 'guix-package-info) --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable It changes the rendering of package-info buffers such that, instead of displaying something like: Name : coreutils Version : 8.23 Synopsis : Core GNU utilities (file, text, shell) ... it first displays the important info in a visible heading, and then goes on with the other fields unchanged: coreutils 8.23 Core GNU utilities (file, text, shell) Outputs: out Delete ... On X11, the package name and version are displayed in a large variable-width font, and the synopsis is displayed in a slightly smaller font. Ludo=E2=80=99. --=-=-=--