Needs review
Project:
widgEditor - A WYSIWYG editor
Version:
6.x-1.0
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
10 Jun 2007 at 02:35 UTC
Updated:
29 Jan 2009 at 18:39 UTC
Jump to comment: Most recent file
Comments
Comment #1
Gurpartap Singh commentedIt still remains there because at least it's usable in html view, though not for normal editor. You may want to add some jquery code to remove the grippie when editor is set to active, if you want to turn it off.
Comment #2
alonpeer commentedI've fixed this issue, but not entirely.
I've created jq_iframe.js (added to the module's scripts folder), which is an adapted copy of misc/textarea.js, which allows creation of grippies for iFrames.
widgeditor.module was adapted to load this new .js file, and widgeditor.js was adapted to remove the old grippie, allowing the new grippie to be created.
There's an issue when dragging the grippie upwards. If dragged too fast, the cursor goes into the iFrame, which causes the dragging to stop (since you "leave" the current page).
A light-patch I made for this was enlarging the grippie created for the iFrame (height:40px), so that it takes a really fast dragging to make the cursor go into the iFrame.
Any help with this will be welcome: is there a way to make the function startDrag() add some kind of a mask on top of the iFrame, so that when the cursor scrolls on it, it'll still stay in the current page scope?
In addition to fixing this bug, I've also fixed an IE bug which didn't allow the WYSIWYG to load at all. The problem was the order of loaded scripts. Basically, I moved the call to widgeditor.js's widgInit() function into the jQuery event created in widgeditor.module's widgeditor_add_javascript() (instead of using the JS run()).
Comment #3
alonpeer commentedComment #4
alonpeer commentedMy jq_iframe.js broke the HTML view <-> Plain text view switcher, because of the wrapping DIVs created by the original textarea.js.
I've removed the wrappng DIVs, it seems to work fine on IE7 without it.
Further review will be appreciated.
Comment #5
gagarine commentedtrack
Comment #6
swortis commentedThe HTML view has returned, but the resize handle for HTML View no longer works (in FF at least)