all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Mauro Aranda <maurooaranda@gmail.com>
To: Al Haji-Ali <abdo.haji.ali@gmail.com>, 75171@debbugs.gnu.org
Subject: bug#75171: 30.0.50; Checklist widget inside a group does not initialize correctly
Date: Fri, 3 Jan 2025 10:05:26 -0300	[thread overview]
Message-ID: <94fb0321-f49b-4cbc-84a5-9696f9366cd3@gmail.com> (raw)
In-Reply-To: <m234i0q3df.fsf@gmail.com>

Al Haji-Ali <abdo.haji.ali@gmail.com> writes:

 > On 02/01/2025, Mauro Aranda wrote:
 >
 >> Al Haji-Ali <abdo.haji.ali@gmail.com> writes:
 >> There's no explicit :value for the group widget, so its value is nil,
 >> and then the Widget library tries to create all four children with a nil
 >> value.  The radio-button-choice widget differs from the checklist widget
 >> in that the former tries to be created with a selected choice, and tries
 >> harder than the checklist widget, which is fine with a value of nil.
 >>
 >> That way, you see that the specified value is obeyed in
 >> radio-button-choice but not in the checklist widget.
 >>
 >> FTR, I'm not saying this is 100% correct.  I'm just trying to explain
 >> how it works now.
 >
 > Thanks, this makes sense. Did I miss an explanation of this issue
 > somewhere in the docs?

I don't think so.  The manual is still missing good explanations.

 >> This is intended behavior, AFAIU.  The parent can override values for
 >> the children, and that way we can recreate widgets with new values just
 >> by changing the values of the parent.
 >
 > I was aware of this behaviour for the group, but I didn't twig the
 > consequence on a checklist.
 >
 > I have to say that I find it a bit counter-intuitive. Also the fact
 > that there is no way to set the value of a checklist without repeating
 > it in the group is somewhat awkward (certainly it will complicate my
 > implementation)

Note that you don't necessarily need to repeat it in the checklist
widget.  In my example, you only need to pass the :value in group:

(defun widget-test ()
   (let ((items '((item :format "%[One%] " :value 1)
                  (item :format "%[Two%] " :value 2)
                  (item :format "%[Three%] " :value 3))))
     (widget-create
      'group
      :value '(2 (1 2))
      (append '(radio-button-choice
                :format "Not inline radio:\n%v\n")
              items)
      (append '(checklist
                :format "Not inline checks:\n%v\n")
              items)))
   (widget-setup))

When created, radio-button-choice will get its :value set to 2 and
checklist will get it set to (1 2).

 > and a bit inconsistent since setting the value of the
 > checklist after the group creation would override the value of the
 > group. A fix, or a some way, that would prevents the group from
 > overriding the values of the children would be useful, IMO.

 > With that being said, this issue should probably be closed as "not a 
bug".

I'd like to take some time to study this and see if there's a way to
improve the situation.






      reply	other threads:[~2025-01-03 13:05 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-29  8:33 bug#75171: 30.0.50; Checklist widget inside a group does not initialize correctly Al Haji-Ali
2025-01-02 13:56 ` Mauro Aranda
2025-01-02 22:04   ` Al Haji-Ali
2025-01-03 13:05     ` Mauro Aranda [this message]

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=94fb0321-f49b-4cbc-84a5-9696f9366cd3@gmail.com \
    --to=maurooaranda@gmail.com \
    --cc=75171@debbugs.gnu.org \
    --cc=abdo.haji.ali@gmail.com \
    /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.