This issue duplicates another issue.
  1. #1
    Sencha Premium Member
    Join Date
    May 2009
    Posts
    156
    Vote Rating
    9
    ZachG will become famous soon enough

      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
    Gainesville, FL
    Posts
    36,522
    Vote Rating
    814
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      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
    ________________
    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 in print!

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

  3. #3
    Sencha Premium Member
    Join Date
    May 2009
    Posts
    156
    Vote Rating
    9
    ZachG will become famous soon enough

      0  

    Default


    Thanks for the heads up.

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

Thread Participants: 1