Skip to content

Comments

Attempt adding a multi checkbox field #143#176

Closed
Mark-H wants to merge 1 commit intomasterfrom
issue-143
Closed

Attempt adding a multi checkbox field #143#176
Mark-H wants to merge 1 commit intomasterfrom
issue-143

Conversation

@Mark-H
Copy link
Member

@Mark-H Mark-H commented Oct 24, 2019

In progress. Throws a JS error due field.items for some reason.

utilities.js:67 Uncaught TypeError: nodeToRecurse.items.each is not a function
    at constructor.clearDirty (utilities.js:67)
    at constructor.<anonymous> (utilities.js:71)
    at Ext.util.MixedCollection.each (ext-all.js:21)
    at constructor.clearDirty (utilities.js:67)
    at MODx.FormPanel.clearDirty (modx.panel.js:210)
    at new MODx.FormPanel (modx.panel.js:63)
    at Object.create (ext-all.js:21)
    at S.createComponent (ext-all.js:21)
    at S.lookupComponent (ext-all.js:21)
    at S.add (ext-all.js:21)
    at S.<anonymous> (ext-all.js:21)
    at Object.each (ext-base.js:21)
    at S.add (ext-all.js:21)
    at S.initComponent (ext-all.js:21)
    at S.initComponent (ext-all.js:21)
    at S.Ext.Component [as constructor] (ext-all.js:21)
    at S [as constructor] (ext-base.js:21)
    at S [as constructor] (ext-base.js:21)
    at new S (ext-base.js:21)
    at Object.create (ext-all.js:21)
    at MODx.load (modx.js?v=01800b12:85)
    at ClientConfig.page.Home._loadComponents (modx.component.js:68)
    at ClientConfig.page.Home.MODx.Component [as constructor] (modx.component.js:10)
    at new ClientConfig.page.Home (home.js:60)
    at Object.create (ext-all.js:21)
    at MODx.load (modx.js?v=01800b12:85)
    at ?a=home&namespace=clientconfig:159
    at ext-all.js:21
    at b (ext-all.js:21)

#143

@sebastian-marinescu
Copy link
Contributor

sebastian-marinescu commented Mar 21, 2025

Hi Mark, the technology caught up with your idea, and we can finally achieve this!

As I'd really really like to have this feature as soon as humanly possible - can I help?
Should I prepare a new PR based on my radio-settings branch?
Or I can integrate Multi-Checkbox-Field into my Radio-Settings PR, two for one 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants