PDA

View Full Version : [CLOSED][3.1svn] RowEditor appears blank if bufferResize is set to false



meroy
25 Jan 2010, 6:23 PM
In examples/grid/row-editor.html, I added the following after ext-all.js:



<!-- GC --><!-- LIBS -->
<script type="text/javascript" src="../../adapter/ext/ext-base.js"></script>
<script type="text/javascript" src="../../ext-all.js"></script>

<script type="text/javascript">
Ext.Container.prototype.bufferResize = false;
</script>

<script type="text/javascript" src="gen-names.js"></script>
<script type="text/javascript" src="../ux/RowEditor.js"></script>
<script type="text/javascript" src="row-editor.js"></script>



Refresh your browser and double click on a row.
The row-editor panel appears under IE 6, but is blank.
For other broswers, such as FF, Safari -- only the buttons appear, but no form panel.

Thanks,

meroy
25 Jan 2010, 8:06 PM
The example works with code extracted from 5931 (left <) and fails with the (right >) 5932+.



diff -r svn.ext-5931/src/widgets/layout/BoxLayout.js svn.ext-5937/src/widgets/layout/BoxLayout.js
79c79
< onLayout : function(ct, target){
---
> renderAll : function(ct, target){
86c86,92
< this.renderAll(ct, this.innerCt);
---
> Ext.layout.BoxLayout.superclass.renderAll.call(this, ct, this.innerCt);
> },
>
> onLayout : function(ct, target){
> if(!this.innerCt){
> this.renderAll(ct, target);
> }
diff -r svn.ext-5931/src/widgets/layout/ColumnLayout.js svn.ext-5937/src/widgets/layout/ColumnLayout.js
87,90c87
< // private
< onLayout : function(ct, target){
< var cs = ct.items.items, len = cs.length, c, i;
<
---
> renderAll : function(ct, target) {
97c94,103
< this.renderAll(ct, this.innerCt);
---
> Ext.layout.ColumnLayout.superclass.renderAll.call(this, ct, this.innerCt);
> },
>
> // private
> onLayout : function(ct, target){
> var cs = ct.items.items, len = cs.length, c, i;
>
> if(!this.innerCt){
> this.renderAll(ct, target);
> }

meroy
25 Jan 2010, 8:21 PM
It's with BoxLayout.js.

Jamie Avins
25 Jan 2010, 9:13 PM
That parameter is not optional.

meroy
25 Jan 2010, 9:40 PM
What I'm saying is that for apps with bufferResize set to false after including ext-all.js and running RowEditor will fail with the current build -- all browsers.

The API says that bufferResize can be set to false.

Jamie, it was just a FYI and I submitted a bug report.

Disregard this, if you think this won't be a problem out there.

Thanks.