1. #561
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    [IMPLEMENTED]


    Class Ext.Panel

    floating should be indicated as Mixed instead of Boolean. You can also specify an object which will then be used as a config object for Ext.Layer.

    floating : Boolean Mixed

    True to float the panel (absolute position it with automatic shimming and shadow), false to display it inline where it is rendered (defaults to false). Note that by default, setting floating to true will cause the panel to display at negative offsets so that it is hidden -- because the panel is absolute positioned, the position must be set explicitly after render (e.g., myPanel.setPosition(100,100). Also, when floating a panel you should always assign a fixed width, otherwise it will be auto width and will expand to fill to the right edge of the viewport.
    Perhaps:

    floating : Boolean Mixed

    True to float the panel (absolute position it with automatic shimming and shadow), false to display it inline where it is rendered (defaults to false). Note that by default, Setting floating to true will create an Ext.Layer for this panel and display the panel at negative offsets so that it is hidden -- because the panel is absolute positioned, the position must be set explicitly after render (e.g., myPanel.setPosition(100,100). Also, when floating a panel you should always assign a fixed width, otherwise it will be auto width and will expand to fill to the right edge of the viewport. This property may also be configured with an object to be used as the configuration object for the Ext.Layer (create link to Ext.Layer) that will be created.

  2. #562
    Ext User jmcneese's Avatar
    Join Date
    Feb 2008
    Posts
    25
    Vote Rating
    0
    jmcneese is on a distinguished road

      0  

    Default


    [closed]

    in reference to http://extjs.com/forum/showthread.php?t=59757:

    in Ext.grid.GridPanel, the recent inclusion of the config property 'fieldLabel' states that when the grid is contained by a component using FormLayout, this property will instruct the container to render the grid as a form field, with the fieldLabel appearing to the side as other form fields are rendered.

    there should be:

    1. something in the text for this config property stating that it also requires the 'isFormField' configure option to be set to true
    2. the config property 'isFormField' should be documented, as it is required to allow 'fieldLabel' to be used.

  3. #563
    Sencha - Community Support Team Condor's Avatar
    Join Date
    Mar 2007
    Location
    The Netherlands
    Posts
    24,246
    Vote Rating
    94
    Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of

      0  

    Default


    [closed]

    Quote Originally Posted by jmcneese View Post
    in reference to http://extjs.com/forum/showthread.php?t=59757:

    in Ext.grid.GridPanel, the recent inclusion of the config property 'fieldLabel' states that when the grid is contained by a component using FormLayout, this property will instruct the container to render the grid as a form field, with the fieldLabel appearing to the side as other form fields are rendered.

    there should be:

    1. something in the text for this config property stating that it also requires the 'isFormField' configure option to be set to true
    2. the config property 'isFormField' should be documented, as it is required to allow 'fieldLabel' to be used.
    It's much more complex than simply including isFormField:true! A BasicForm expects all kinds of methods to be present for a Component with isFormField:true.
    So simply adding isFormField:true and a fieldLabel to a GridPanel will cause javascript errors in form processing.

    ps. In Ext 3.0 simply adding a fieldLabel is enough, no need to set isFormField:true.

  4. #564
    Ext User
    Join Date
    Nov 2008
    Posts
    3
    Vote Rating
    0
    jililv is on a distinguished road

      0  

    Smile Method annotation error for Ext.Container.doLayout( [Boolean shallow] )

    Method annotation error for Ext.Container.doLayout( [Boolean shallow] )


    Hi All

    I am new to ExtJS and I am not sure if the bug has been reported here or not. Below is what I find.

    The current annotation for Ext.Container.doLayout( [Boolean shallow] ) in Ext 2.2 Api Documentation is shown below:

    doLayout( [Boolean shallow] ) : void
    Force this container's layout to be recalculated. A call to this function is required after adding a new component to an already rendered container, or possibly after changing sizing/position properties of child components.

    Parameters:

    shallow : Boolean
    (optional) True to only calc the layout of this component, and let child components auto calc layouts as required (defaults to false, which calls doLayout recursively for each subcontainer)

    Returns:
    void

    I think the annotation of the True & false vaule for the shallow parameter is reversed. It should be changed as follows:

    shallow : Boolean
    (optional) false to only calc the layout of this component, and let child components auto calc layouts as required (defaults to true, which calls doLayout recursively for each subcontainer)

    Here is the source code in the class Ext.Container:

    doLayout : function(shallow){
    if(this.rendered && this.layout){
    this.layout.layout();
    }
    if(shallow !== false && this.items){
    var cs = this.items.items;
    for(var i = 0, len = cs.length; i < len; i++) {
    var c = cs;
    if(c.doLayout){
    c.doLayout();
    }
    }
    }
    },

    You can see only when shallow == true makes the method start the recursion to subcontainers.

    [i]And the word "shallow" also confuse me a little bit. I think using "deep" may be more appropriate if what I'd written above is correct.


    Thanks!



  5. #565
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,004
    Vote Rating
    650
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


    I think it's fine, the intent is clear and it's explained well.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

  6. #566

  7. #567
    Sencha User
    Join Date
    Mar 2007
    Posts
    7,854
    Vote Rating
    4
    tryanDLS is on a distinguished road

      0  

    Default


    You can see only when shallow == true makes the method start the recursion to subcontainers.
    Yes, it's correct as written. You misunderstand the difference betweeen
    Code:
    shallow !== false
    and
    Code:
    shallow == true

  8. #568
    Ext User
    Join Date
    Nov 2008
    Posts
    3
    Vote Rating
    0
    jililv is on a distinguished road

      0  

    Smile Annotation of Ext.Container.doLayout()

    Annotation of Ext.Container.doLayout()


    Quote Originally Posted by evant View Post
    I think it's fine, the intent is clear and it's explained well.
    Hi evant

    Do you think the API's annotation need to be changed or not? Sorry my English is not very good and I can not tell from your reply if you agree with me or not.

    Hi All

    Let me explain my opinion in another way.

    When I call doLayout(true) or doLayout() on my toplevel container, such as an Ext.Window, I find that it will recursively layout the subcontainers within that Ext.Window object. Contrarily, when I call doLayout(false) on my toplevel container, the subcontainers will not be refreshed. This is what I find through practice.

    But when I first use the doLayout method, after reading the current annotation of this method in the Api Documentation makes me think that calling doLayout(false) on the toplevel container will recursively layout the subcontainers which I finally find is not correct through practice.

    I am not sure if you guys have the same experience or not?

    Thanks!

  9. #569
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    Annotation of Ext.Container.doLayout()

    I think the docs need touch up. I agree that the default does a recursive layout.

    If shallow =
    1. true.
      Code:
      (true !== false) => true, therefore does recursive layout
    2. false.
      Code:
      (false !== false) => false, therefore layout only this container, no children
    3. undefined (the default).
      Code:
      (undefined !== false) => true, therefore does recursive layout

    Suggested revision:
    Code:
    (optional) Specify false to only calc the layout of this component, and let child components auto calc layouts as required.  The default is true, which calls doLayout recursively for each subcontainer item.
    Not a doc change, but I'd also agree that "shallow" is contradictory to the behavior.

  10. #570
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default [implemented]

    [implemented]


    [implemented]

    The argument type is not declared in Ext.Panel for toolTemplate.

    This crashes oxymoron's js doc tool, so may well do the same to ExtJS's version.

    Code:
        /**
         * @cfg toolTemplate
         * @type {Ext.Template/Ext.XTemplate}
    Specifying the type fixes it:
    Code:
        /**
         * @cfg {Object} toolTemplate
         * @type {Ext.Template/Ext.XTemplate}

Thread Participants: 182

  1. JeffHowden (3 Posts)
  2. Animal (64 Posts)
  3. tryanDLS (10 Posts)
  4. christocracy (1 Post)
  5. jarrod (2 Posts)
  6. jay@moduscreate.com (21 Posts)
  7. JTM (1 Post)
  8. vtswingkid (1 Post)
  9. young_matthewd (3 Posts)
  10. sjivan (1 Post)
  11. yogurtearl (1 Post)
  12. Condor (34 Posts)
  13. mdissel (2 Posts)
  14. steve.neill (4 Posts)
  15. mikegiddens (3 Posts)
  16. rmesser (1 Post)
  17. jweber (1 Post)
  18. aconran (8 Posts)
  19. vmorale4 (13 Posts)
  20. tof (4 Posts)
  21. stever (1 Post)
  22. berend (3 Posts)
  23. Jul (16 Posts)
  24. bone (2 Posts)
  25. timb (1 Post)
  26. punkrider (2 Posts)
  27. dolittle (4 Posts)
  28. papasi (1 Post)
  29. kimu (1 Post)
  30. willydee (2 Posts)
  31. dotnetCarpenter (1 Post)
  32. steven (3 Posts)
  33. DeyV (1 Post)
  34. cdomigan (1 Post)
  35. JorisA (2 Posts)
  36. MaxT (3 Posts)
  37. jsakalos (2 Posts)
  38. tellioglu (2 Posts)
  39. architect (1 Post)
  40. ksachdeva (1 Post)
  41. PFM (1 Post)
  42. evant (4 Posts)
  43. thesilentman (4 Posts)
  44. para (2 Posts)
  45. sj137 (6 Posts)
  46. J.C. Bize (3 Posts)
  47. Jonno70 (1 Post)
  48. splintor (9 Posts)
  49. mribbons (1 Post)
  50. danh2000 (1 Post)
  51. loeppky (2 Posts)
  52. mdm-adph (3 Posts)
  53. NBRed5 (1 Post)
  54. meej (1 Post)
  55. tobiu (1 Post)
  56. patspam (1 Post)
  57. gimbles (2 Posts)
  58. jan1234 (2 Posts)
  59. Ytorres (1 Post)
  60. aparajita (4 Posts)
  61. carol.ext (2 Posts)
  62. end-user (3 Posts)
  63. nicora (1 Post)
  64. devnull (4 Posts)
  65. AdamDawes (1 Post)
  66. mjlecomte (92 Posts)
  67. BlackICE1979 (1 Post)
  68. vincentc (1 Post)
  69. rednix (3 Posts)
  70. hansellh (1 Post)
  71. TaterSalad (1 Post)
  72. aptana_michelle (1 Post)
  73. Trixor (3 Posts)
  74. Iveco (1 Post)
  75. mscdex (4 Posts)
  76. watrboy00 (8 Posts)
  77. pic_Nick (1 Post)
  78. tohagan (1 Post)
  79. gimler (1 Post)
  80. smcguire36 (1 Post)
  81. ziesemer (2 Posts)
  82. balou (1 Post)
  83. yarmy (1 Post)
  84. jerrybrown5 (1 Post)
  85. abent (1 Post)
  86. lvanderree (4 Posts)
  87. Greenosity (1 Post)
  88. tjcrowder (18 Posts)
  89. treffer (2 Posts)
  90. SeaSharp (3 Posts)
  91. dhurlburtusa (1 Post)
  92. neshaug (2 Posts)
  93. tony_k (1 Post)
  94. Jim McCabe (1 Post)
  95. dyndan (1 Post)
  96. Hemlock (1 Post)
  97. runnersixty (1 Post)
  98. stoober (1 Post)
  99. battisti (1 Post)
  100. slobo (1 Post)
  101. plan-r (2 Posts)
  102. joeaudette (1 Post)
  103. paradawks (1 Post)
  104. saJoshua (7 Posts)
  105. t34 (6 Posts)
  106. markpele (1 Post)
  107. Gordon76 (1 Post)
  108. mankz (3 Posts)
  109. cblin (1 Post)
  110. jasonbronson (3 Posts)
  111. stebru (1 Post)
  112. Enes (1 Post)
  113. jburnhams (1 Post)
  114. david.james (1 Post)
  115. krystian (1 Post)
  116. erlando (1 Post)
  117. asteidle (1 Post)
  118. iamnoah (1 Post)
  119. molasses (1 Post)
  120. jonn (3 Posts)
  121. jhurst (1 Post)
  122. Starfall (14 Posts)
  123. creusois (1 Post)
  124. silvereen (1 Post)
  125. vilivili (2 Posts)
  126. jezmck (1 Post)
  127. nbourdeau (1 Post)
  128. YarivYaari (1 Post)
  129. luv2hike (1 Post)
  130. mrileyaz (1 Post)
  131. jmcneese (1 Post)
  132. treqx (1 Post)
  133. willdudziak (1 Post)
  134. NOSLOW (3 Posts)
  135. tomcheng76 (2 Posts)
  136. jove4015 (1 Post)
  137. xpurpur (1 Post)
  138. mknopp (1 Post)
  139. aurelien (2 Posts)
  140. nctag (5 Posts)
  141. pbuyle (1 Post)
  142. keckeroo (2 Posts)
  143. vcastel (1 Post)
  144. switch (1 Post)
  145. dcesarnascimento (1 Post)
  146. mirko (1 Post)
  147. mdkalman (1 Post)
  148. nb (1 Post)
  149. makana (2 Posts)
  150. fins (1 Post)
  151. NoahK17 (1 Post)
  152. Answerfor (1 Post)
  153. russn (1 Post)
  154. accilies (1 Post)
  155. SamuraiJack1 (1 Post)
  156. Eric24 (2 Posts)
  157. Funcracker (5 Posts)
  158. oxyum (1 Post)
  159. SoreGums (1 Post)
  160. Berni (1 Post)
  161. photomo (1 Post)
  162. nicktak (1 Post)
  163. don_luiz (2 Posts)
  164. wwarby (7 Posts)
  165. hAmpzter (1 Post)
  166. masuran (1 Post)
  167. ben_dog (3 Posts)
  168. thboileau (4 Posts)
  169. kds.chris (1 Post)
  170. salewski (1 Post)
  171. Mjollnir26 (1 Post)
  172. mikamikaze (1 Post)
  173. dreas (2 Posts)
  174. mschwartz (1 Post)
  175. jililv (2 Posts)
  176. TinKim (1 Post)
  177. zgrose (1 Post)
  178. martinjrb (1 Post)
  179. flinz (2 Posts)
  180. PierceSD (1 Post)
  181. dlgoodchild (1 Post)
  182. brian.moeskau (12 Posts)