Checkbox value is saved as 1, not false as desired

$form['toolbar_top'] = [ '#type' => 'checkbox', '#title' => $this->t('Toolbar at top'), '#description' => $this->t('Make toolbar stay at the top when scrolling.'), '#default_value' => $config->get('toolbar_top'), ];

$ drush config:get admin_toolbar_tools.settings _core: default_config_hash: WgdZsrd_5w9jlmcHV4R9dD2tG9OZEkYo4I_O8h7Gq8Q max_bundle_number: 20 hoverintent_functionality: true show_local_tasks: false

Comments & Activities

Issue created by @ressa
Status changed to Needs review 3 months ago6:32am 30 May 2024

Comment 3 months ago →

🇮🇳India dev16.addweb

Thanks @ressa, I reviewed Both config (hoverintent_functionality and how_local_tasks), Then I
define type: boolean to admin_toolbar.schema.yml, as below

admin_toolbar.settings:
  type: config_object
  label: 'Admin Toolbar settings'
  mapping:
    menu_depth:
      type: integer
      label: 'Depth of displayed menu'
    toolbar_top:
      type: boolean
      label: 'Toolbar at top'

Now values in checkboxe is saved as true or false

drush cget admin_toolbar.settings
_core:
  default_config_hash: jvTSppzcgH5wnzBhX5xnAExcp2I1CzkQ_aky65XNfYI
menu_depth: 4
toolbar_top: true

Status changed to Active 3 months ago10:15am 30 May 2024
Comment 3 months ago →
🇺🇸United States cilefen
There isn’t a merge request to review.
Status changed to Closed: won't fix 3 months ago12:12pm 30 May 2024
Comment 3 months ago →
🇮🇳India Prashant.c Dharamshala
This issue is related to the contributed module https://www.drupal.org/project/admin_toolbar → and not Drupal core, hence IMHO this could be closed.
Comment 3 months ago →
🇩🇰Denmark ressa Copenhagen
Thank you everyone for fast replies, and resolving this issue so quickly. I agree, this should be fixed in the Admin Toolbar issue.

Checkbox value is saved as 1, not false as desired

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments & Activities