Labels |
Added:
?
|
We had this reported before but it was in the template editor. Look for the PR that fixed it there and apply here
@brianteeman pull request #30241?
Yes thats the one
Looks like indeed this solves the issue
diff --git a/build/media_source/plg_editors_codemirror/js/joomla-editor-codemirror.w-c.es6.js b/build/media_source/plg_editors_codemirror/js/joomla-editor-codemirror.w-c.es6.js
index ecef691..e99ae93 100644
--- a/build/media_source/plg_editors_codemirror/js/joomla-editor-codemirror.w-c.es6.js
+++ b/build/media_source/plg_editors_codemirror/js/joomla-editor-codemirror.w-c.es6.js
@@ -42,4 +42,10 @@
const that = this;
+ // Check if instance exists to avoid duplication on resize
+ if (this.instance !== '') {
+ Joomla.editors.instances[this.element.id] = this.instance;
+ return;
+ }
+
// For mode autoloading.
window.CodeMirror.modeURL = this.getAttribute('mod-path');
That's not the correct solution since the problem is not in CodeMirror, a similar issue exists with tinyMCE #33101
The problem is that tabs are actually moving DOM elements when switching from tabs to accordion therefore the proper solution would be to fix the tabs to not move the DOM elements.
a similar issue exists with tinyMCE
We had this reported before but it was in the template editor.
The problem with the template editor, when using out of the box configuration (CodeMirror) still remains today unresolved.
using out of the box configuration
What do you mean here? Can you provide an example?
Simple.
Yeah, that's from tabs and the stupid mode change to accordion. I shouldn't have pushed that stupid code...
So... What now?
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-07-17 20:49:02 |
Closed_By | ⇒ | Quy |
Confirm Issue, test on nightly build.