Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext GWT Premium Member takayser's Avatar
    Join Date
    Oct 2008
    Location
    Lucerne, Switzerland
    Posts
    188
    Vote Rating
    0
    takayser is an unknown quantity at this point

      0  

    Default [FIXED] [2.0 RC1] Desktop - addTool(menuItem) broken

    [FIXED] [2.0 RC1] Desktop - addTool(menuItem) broken


    If you open the startmenu from desktop and then put another menuItem in the startmenu, reopen the startmenu, then the application crashes. This only happens if you have opened the startmenu once before and if you use addTool(), for add() it works as expected.

    I wrote a testcase for that, run it and click continuous on the start menu button to reproduce the issue.

    Code:
    public void onModuleLoad() {
    
          final Desktop desktop = new Desktop();
    
          Info.display("empty startmenu", "works");
          Timer timer1 = new Timer() {
             @Override
             public void run() {
                Info.display("add(menuItem);", "works too!");
                desktop.getStartMenu().add(new MenuItem("that works"));
             }
          };
    
          timer1.schedule(4000);
          Timer timer2 = new Timer() {
             @Override
             public void run() {
                Info.display("addTool(menuItem);", "crashes");
                desktop.getStartMenu().addTool(new MenuItem("that works"));
             }
          };
          timer2.schedule(8000);
    }

  2. #2
    Software Architect
    Join Date
    Sep 2007
    Posts
    13,971
    Vote Rating
    132
    sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light

      0  

    Default


    The current design of addTool is prerender. I will add this ifnormation to the javadocs.

  3. #3
    Software Architect
    Join Date
    Sep 2007
    Posts
    13,971
    Vote Rating
    132
    sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light sven is a glorious beacon of light

      0  

    Default


    Added that featrue to SVN.

Thread Participants: 1