Success! Looks like we've fixed this one. According to our records the fix was applied for DSGNR-1403 in Architect 2.1.
  1. #1
    Sencha User
    Join Date
    Jan 2012
    Posts
    5
    Vote Rating
    0
    sssml2011 is on a distinguished road

      0  

    Default "defaultMargins" is set to a string instead of an object.

    "defaultMargins" is set to a string instead of an object.


    I attempted to set the defaultMargins property for a Container to {top:0, left:0, bottom:0, right:5} and Designer set it to the string '{top:0, left:0, bottom:0, right:5}' in the generated code. This caused the contained items to not display. I removed the defaultMargins property value and the items display, obviously without the margins.

  2. #2
    Sencha - Architect Dev Team aconran's Avatar
    Join Date
    Mar 2007
    Posts
    8,185
    Vote Rating
    63
    aconran is just really nice aconran is just really nice aconran is just really nice aconran is just really nice aconran is just really nice

      0  

    Default


    It does convert it to a string for me, but it also works properly in object notation.

    Regardless, we will look into this bug.
    Aaron Conran
    @aconran
    Sencha Architect Development Team

  3. #3
    Sencha User
    Join Date
    Jan 2012
    Posts
    5
    Vote Rating
    0
    sssml2011 is on a distinguished road

      0  

    Default


    I am not sure what you mean by "but it also works properly in object notation." When the defaultMargins property is set to a string, as Designer is doing, regardless of whether the string contains object notation, the margins are not applied.

    The other thing I noticed is that defaultMargins does not work when set to a CSS margin string, such as "0 0 0 5", even though the Ext JS documentation says it can be set this way. This appears to either be a bug in Ext JS, or incorrect documentation.

  4. #4
    Sencha - Architect Dev Team dfoelber's Avatar
    Join Date
    Sep 2011
    Location
    Rockville, MD
    Posts
    93
    Vote Rating
    10
    dfoelber will become famous soon enough

      1  

    Default


    This issue has been fixed and will be included in the next update.

    Thanks!
    David Foelber
    @davidfoelber
    Sencha Architect Development Team

  5. #5
    Ext JS Premium Member
    Join Date
    Oct 2009
    Location
    Hamilton ON Canada
    Posts
    24
    Vote Rating
    0
    fargs is on a distinguished road

      0  

    Default


    I am using build 442 but the release notes have only been updated to 439. This issue still exists in 442. Adding release dates to the changelog would be useful as well.

    Thanks,

  6. #6
    Sencha - Architect Dev Team aconran's Avatar
    Join Date
    Mar 2007
    Posts
    8,185
    Vote Rating
    63
    aconran is just really nice aconran is just really nice aconran is just really nice aconran is just really nice aconran is just really nice

      0  

    Default


    We released Sencha Architect 2.1 this morning and this issue is resolved.
    www.sencha.com/blog/sencha-architect-2-1

    Please check out the changelog:
    http://docs.sencha.com/architect/2/#!/guide/changelog
    Aaron Conran
    @aconran
    Sencha Architect Development Team