Reported in #1204786: [D6] Bug in custom JavaScript configuration - editor not showing in some cases by GreatStarMaster:

Му custom javascript configuration is
config.extraCss = "body {background: #FFFFFF; background-image: none;}";
Editor not showing. Perhaps it is because of the semicolon in the style.

Comments

dczepierga’s picture

Status: Active » Fixed

Changes commited to GIT.

Pls update to last DEV version.

Greetings

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

mkesicki’s picture

Title: [D6] Bug in custom javascript configuration - semicolon problem » [D6] Bug in custom JavaScript configuration - semicolon problem
anrikun’s picture

Status: Closed (fixed) » Active

I'm reopening this issue:
In 6.x-1.11, semicolons seem required in custom javascript configuration.
So current description is incorrect.

mkesicki’s picture

Status: Active » Postponed (maintainer needs more info)

@anrikun please ,
write steps to reproduce your issue and write in more details what problem do you have?

anrikun’s picture

Sorry, I thought I was clear enough :-)

Description for field Custom javascript configuration currently displays the following as examples:

config.fontSize_sizes = '16/16px;24/24px;48/48px;'
config.font_names = 'Arial;Times New Roman;Verdana'

Respecting these examples, I have typed:
config.bodyClass = 'myclass1 myclass2'
But it didn't worked (no error but my bodyClass setting was ignored), so I tried to add a semicolon:
config.bodyClass = 'myclass1 myclass2';
And it worked.

So I guess semicolons are needed here, and that examples should be changed into:

config.fontSize_sizes = '16/16px;24/24px;48/48px;';
config.font_names = 'Arial;Times New Roman;Verdana';
mkesicki’s picture

Status: Postponed (maintainer needs more info) » Active

@anrikun ,
thank you for noticing this.
We try fix this as soon as possible.

mkesicki’s picture

Status: Active » Fixed

I committed changes to GIT.

mkesicki’s picture

Status: Fixed » Closed (fixed)
anrikun’s picture

Thanks for this quick commit!

mkesicki’s picture

Title: [D6] Bug in custom JavaScript configuration - semicolon problem » [D6] Fix description to "Custom JavaScript configuration" option