This issue duplicates another issue.
  1. #1
    Sencha User
    Join Date
    May 2009
    Posts
    121
    Vote Rating
    7
    ZachG is on a distinguished road

      0  

    Default [4.1.2] Checkbox in Menu Item

    [4.1.2] Checkbox in Menu Item


    REQUIRED INFORMATION

    Ext version tested:
    • Ext 4.1.2.381
    • Build date: 2012-09-07 16:54:29 (4861809b76e43ce65f9f8bfed1f258813adb26de)
    Browser versions tested against:
    • Chrome
    Description:
    • Ext.menu.CheckItem has checked and group properties that make a group of them act like a radio button.
    • In ExtJS < 4.1.2, the one that's checked would just display a circle where the icon normally goes.
    • In 4.1.2, we're seeing the checked one also displays a checkbox on the right side of the menu.
    • There needs to be some way to turn this off.
    Steps to reproduce the problem:
    • Go to the Feed Viewer example.
    • Run the code below.
    • Click on the My Menu button to show the menu.
    The result that was expected:
    • It would work like in ExtJS < 4.1.2
    The result that occurs instead:
    • Nope
    Test Case:


    Code:
    Ext.onReady(function()    {
        Ext.create("Ext.window.Window", {
            title:"ExtJS 4.1.1 (Correct)",
            width:500,
            height:500,
            layout:"fit",
            dockedItems:[
                {
                    xtype:"toolbar",
                    dock:"top",
                    items:[
                        {
                            text:"My Menu",
                            menu:{
                                defaults:{
                                    checked:false,
                                    group:"mygroup"
                                },
                                items:[
                                    {
                                        text:"Not Checked 1"
                                    },
                                    {
                                        text:"I AM CHECKED",
                                        checked:true
                                    },
                                    {
                                        text:"Not Checked 2"
                                    },
                                    {
                                        text:"Not Checked 3"
                                    }
                                ]
                            }
                        }
                    ]
                }
            ]
        }).show();
    });





    HELPFUL INFORMATION




    Screenshot or Video:
    • attached
    Debugging already done:
    • Ext.menu.CheckItem.prototype.renderTpl is where the checkbox is coming from. The rightCheckbox​ variable is true.
    Attached Images

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    St. Louis, MO
    Posts
    33,641
    Vote Rating
    434
    mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of

      0  

    Default


    We already have an internal bug reported that was prompted from a support ticket in our support portal: EXTJSIV-7265

    This will be fixed for 4.1.3, there is a merge pending to fix this.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    http://www.JSONPLint.com - Source to lint your JSONP!

    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is almost in print!

    When posting code, please use BBCode's CODE tags.

  3. #3
    Sencha User
    Join Date
    May 2009
    Posts
    121
    Vote Rating
    7
    ZachG is on a distinguished road

      0  

    Default


    Thanks for the heads up.

    For now, we're overriding Ext.menu.CheckItem.prototype.renderTpl to not include the right checkbox piece.