JarvusInnovations / jarvus-ace

An ExtJS wrapper for the ACE editor
7 stars 1 forks source link

Opening Second Ace editor restricts height and width #1

Open bagwanpankaj opened 8 years ago

bagwanpankaj commented 8 years ago

Hi There,

I am using jarvus-ace with ExtJS6, while it works good when only a single/first instance is created. But any subsequent instance has weird CSS style rule imposed. Here is snippet of Styles being applied

element.style {
    margin-top: 0px;
    width: 59795px;
    height: 54px;
    margin-left: -52762px;
}

on element with class .ace_content and applied dynamically. This only happens when second/subsequent instance is created.

Here is Ace config being used

Ext.define('Testt.AceConfiguration', {
  override: 'Jarvus.ace.util.AbstractConfiguration',
  config: {
    options: {
      theme: 'ace/theme/monokai',
      mode: 'ace/mode/javascript',
      fontSize: 18
    }
  }
});

Here is how it is being used

   {
      xtype: 'jarvus-ace-editor',
      reference: 'my_editor',
      flex: 1,
      minHeight: 100,
      labelAlign: 'top',
      fieldLabel: 'Body',
      allowBlank: false,
      subscribe: {
        'Editor': [
            'blur',
            'change'
        ],
        'EditorSession': [
            'change'
        ],
        'Document': [
            'change'
        ]
      }
    }

And there is no JS. error seen in console

bagwanpankaj commented 8 years ago

Is this project maintained actively?

themightychris commented 8 years ago

@bagwanpankaj yep, but it may be a bit before I can dig into this particular issue. Pull requests welcome!

idealogyin commented 8 years ago

Hi There,

I am also facing same issue

themightychris commented 5 years ago

@bagwanpankaj @idealogyin I just picked up the project using this again, and am not seeing the described issue in my own use. Anyone have a patch or a reproducible test case?