Subject: Re: bug#75826: [PATCH] New variable 'bookmark-bmenu-type-column-width'

From: Robert Pluim <[email-masked]>
Date: Sun, 26 Jan 2025 12:30:16 +0100

>>>>> On Sat, 25 Jan 2025 08:45:53 -0500, Ship Mints <[email-masked]> said:

    Ship> *** New variable 'bookmark-bmenu-type-column-width'.
    Ship> This variable allows a user to control the width of the type column
    Ship> on the bookmark menu 'bookmark-bmenu-list'. The default value is 8
    Ship> which is backwards compatible.

    Ship> I marked this new variable as of 31.1 but could also be for 30.2.

If itʼs a defcustom we call it a "user option". Putting on my
wordsmithing hat:

*** New user option 'bookmark-bmenu-type-column-width'.
Specifies the width of the type column in the bookmark menu shown by
'bookmark-bmenu-list'. Default value is 8, which is backwards
compatible.

    Ship> Rationale: In one package I co-maintain, we have custom bookmark handlers
    Ship> with appropriate properties set. The limit of 8 characters for display in
    Ship> the bookmark menu can lead to cumbersome naming for handler abbreviations.

    Ship> -Stephane
    Ship> From b4cc1e055755ceaaa348776f1c2fe40b67f02511 Mon Sep 17 00:00:00 2001
    Ship> From: shipmints <[email-masked]>
    Ship> Date: Sat, 25 Jan 2025 08:36:12 -0500
    Ship> Subject: [PATCH] Add new custom variable bookmark-bmenu-type-column-width

    Ship> This variable allows a user to control the width of the type column
    Ship> on the bookmark menu 'bookmark-bmenu-list'. The default value is 8
    Ship> which is backwards compatible.

    Ship> * lisp/bookmark.el (bookmark-bmenu-mode):
    Ship> Honor bookmark-bmenu-type-column-width when creating the
    Ship> tabulated list.

The commit message should mention the new defcustom.

Robert
--