CMSMS v1.10 issue with MicroTiny

[spcherub]

There's a small problem with MicroTiny bundled with CMSMS 1.10. If more than 1 custom styles are setup in the MicroTiny configuration, the first style selection always ends up appending a comma "," after the class name.

Here are the steps to replicate this problem:
- Go to Extensions > MicroTiny WYSIWYG Editor > Settings
- In the CSS Styles field, enter the follow or something similar:

Code: Select all

mystyle=mystyle1
mystyle1=mystyle2
mystyle2=blah

- Save Settings
- Highlight a word or two and apply the first style ("mystyle" in my example above).
- Now view the HTML and you will see that the class name has a comma in it like below:

Code: Select all

<span class="mystyle,">Lorem</span>

Please let me know if there is some place else I should log this as a bug.

Thanks,
Sanjay

[KO]

I confirm this same thing. Basically the editor help/info says

"The styles can either be just the class name, or a classname with a new name to show. Must be sepereated by either commas or newlines. "

But the newlines wont work ... so this is a bug or that part of microtiny instruction is not correct. You just need to to put all on same line and be sure there is no line change anywhere as those are not cleared properly.

Code: Select all

mystyle=mystyle1,mystyle1=mystyle2,mystyle2=blah

BTW: those styles are not read from stylesheet like in normal TinyMCE advanced module.

[spcherub]

I think Calguy confirmed this bug in a separate thread and mentioned that the fix will be part of the next patch.

S

[KO]

I just updated to latest 1.10.1 and "styles not read from stylesheet" is fixed on that (which is great but I still see those commas after every linechange if CSS-stylenames are separated with new line. This is a minor issue as things work by not using them but could cause confusion for some people not knowing that.

So probably this bug should be filed ...
http://dev.cmsmadesimple.org/bug/list/651
... so devs know about it. Do you want to do it or should I?

[spcherub]

PLease go ahead and do it. Thanks!

[KO]

Bug filed!

http://dev.cmsmadesimple.org/bug/view/7120