Needs review
Project:
Popups API (Ajax Dialogs) [D7]
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Aug 2009 at 21:21 UTC
Updated:
25 Oct 2009 at 13:03 UTC
Jump to comment: Most recent file
Comments
Comment #1
v8powerage commentedComment #2
v8powerage commentedComment #3
ufku commentedBUE 2.x worked with Popups 2.x however some buttons reported infinite loop or some other errors. Then i noticed an eval(script) in popups js that evaluates script dataType, which seems to be not necessary as jQuery loads the scripts evaluated. The below patch removes the eval and as far as i tested it works as expected.
ps: Not being familiar with popups API, still i'm not sure if it's the right diagnosis here.
Comment #4
v8powerage commentedHi
With Bueditor 2x and popups 2x it's working (I had 1x Bueditor before), but I'm getting buttons double (look at screenshot attached), perhaps it's because of teaser?
Comment #5
ufku commentedyes they are teaser buttons. In normal view they appear with the teaser textarea when the "split content at cursor" is clicked, however this feature is broken in popups.(Not related to bueditor)
You may exclude edit-js-teaser in bueditor visibility settings.
Comment #6
v8powerage commentedThank You very much, it works (but it's actually edit-teaser-js).