tinyMCE editor no longer shown for Text Boxes on profile form

Bug #529929 reported by Andrew Nicols
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Undecided
Evan Goldenberg

Bug Description

On Master:

If you Create/Edit a view and add a Text Box, the text box is no longer a
wysiwyg editor but is a text box instead.
This appears to be caused by commit
c27a1236f2f70ebd207b25dae53ef115f61156d9 on Mon 15th Feb which adjusts the
javascript to only give the first tinyMCE on the page focus.

The wysiwyg editor does still show up on pages like the 'Edit Profile'
page, where it is the only tinymce editor on the page. I would guess that
the tinyMCE editor is not focussed, and therefore not populated.

Reverting the commit does fix it, but obviously there was a very good
reason to implement it and I'm not privy to it so producing a fix which
satisfies all situations is a little tricky.

Andrew

Evan Goldenberg (naveg)
Changed in mahara:
assignee: nobody → Evan Goldenberg (naveg)
Revision history for this message
Evan Goldenberg (naveg) wrote :

I implemented a quick fix for this in 0dacf2cc018ba23cd89876d5f83b720a58e74ae4. Might not be the best fix, but for now it'll work. It means that a wysiwyg editor in a block instance config form will not obtain focus automatically.

The problem has to do with the fact that block instance config forms are dynamically inserted into the page after it has already loaded, so the javascript variable specifying which wysiwyg editor to focus was undefined. The fix simply checks first.

Changed in mahara:
status: New → Fix Committed
milestone: none → 1.3.0
Evan Goldenberg (naveg)
Changed in mahara:
milestone: 1.3.0 → 1.2.4
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.