madeyourday / contao-rocksolid-custom-elements

RockSolid Custom Elements Contao Extension
http://rocksolidthemes.com/de/contao/plugins/custom-content-elements
MIT License
48 stars 12 forks source link

Error since 3.5.6 with elements #60

Closed m-knorr closed 8 years ago

m-knorr commented 8 years ago

We recognized a problem with some of our ce-elements created with rsce!

The save process doesn't works due to a javascript error.

An invalid form control with name='rsce_field_items__rsce_dummy__headline' is not focusable.

hope the error is traceable!

ausi commented 8 years ago

Did you use the latest version (1.3.7) of the extension?

m-knorr commented 8 years ago

Yes. We use the current version of composer-client, custom-elements and contao. :(

ausi commented 8 years ago

Please clear the browser cache and check if that fixes the error. If the error still appears, please send us the rsceconfig.php file.

ausi commented 8 years ago

Did clearing the browser cache fix the issue?

madbonkey commented 8 years ago

I'm having the same issue with madeyourday/contao-rocksolid-custom-elements@v1.3.9 and contao/core@3.5.6.

Clearing the browser cache did not resolve the problem.

Edit: This is not necessarily related to any rocksolid extension. I'm starting to get this error for regular contao content elements.

Edit 2: This seems to be a browser issue. I found http://stackoverflow.com/a/28340579 and http://stackoverflow.com/a/30785150 on this. I'm getting the error for regular "Text" content elements that are newly created as well. The problem seems to be the "required" attribute on the hidden <textarea> that holds TinyMCE's HTML content. Initially, this field is blank, required and hidden, and when the HTML5 form validation kicks in, it squawks about the focus thing. Maybe this helps, I'll dig deeper.

ausi commented 8 years ago

Are you using a modified tinyMCE.php file? If so, please perform the changes of Contao 3.5.6: https://github.com/contao/core/commit/c16e537f33d51767b773416094a8c3f0bd9b704f

See contao/core#8131

madbonkey commented 8 years ago

I'll try that later today, thanks. Seems like that would fix it.

ausi commented 8 years ago

@m-knorr, @NicolasSchneider does this issue still appear?

madbonkey commented 8 years ago

@ausi It doesn't any more on clean installations for me.

m-knorr commented 8 years ago

thanks!