Mailing List Archive

[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load.
// Add your reply above here
==================================================
Aaron Fuleki updated this ticket at November 15th, 2010 @ 07:45 PM

Xinha editing windows appear broken (i.e. partially loaded) and inoperable on many browsers, but only if their containing element is collapsed in the UI when the browser first loads the page. The WYSIWYG window becomes operable if you do anything to trigger the UI javascript to reload (e.g., click "Save and Stay", delete some element with the AJAX-y delete button, etc.).

This affects the latest Xinha (currently v0.96.1, as well as v0.95), on all modern browsers, and IE9. Collapsed WYSIWTG elements open up and work fine on IE8. This may mean the issue stems from some IE workaround in bric's AJAX UI code (or it could be coincidental).

**Browsers tested that show the issue:**

* Safari 5 (Mac/Win)
* Firefox 3.6 (Mac/Win)
* Chrome 7 (Mac/Win)
* IE9 preview 6 (Win)

**Unaffected Browsers:**

* IE 8 (Win)

**Steps to reproduce:**

* Open an affected browser.
* Open/Create any story with a WYWSIWYG-enabled element or field.
* Collapse the WYSIWYG element (click the triangle to the left of the container name).
* Save the story with the element collapsed.
* Edit the story again.
* Click the arrow to expand the element.
* Notice that the WYSIWYG appears to load, but looks off (edges missing, the footer runs across the middle, etc.), and you can't click in it to edit anything.
* Do something to force the UI javascript to refresh (click "Save and Stay", delete some other element in the story, etc.).
* Notice that any WYSIWYG elements that were open before the refresh now operate as expected.
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
phillipadsmith updated this ticket at November 22nd, 2010 @ 05:53 PM

Related Xinha bug: http://bricolage.lighthouseapp.com/projects/29601/tickets/180-xinha-create-link-button-does-not-work-in-bric-2
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
theory updated this ticket at November 22nd, 2010 @ 08:06 PM
* Milestone changed from "" to "2.0.1"

--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
phillipadsmith updated this ticket at November 22nd, 2010 @ 09:07 PM

Confirmed. And the solution appears to be straightforward (but is beyond my tuits today): http://www.xinha.org/punbb/viewtopic.php?id=2809
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
Aaron Fuleki updated this ticket at December 2nd, 2010 @ 09:34 PM

Thanks for finding that!

Instead of bootstrapping Xinha at page load time, I'd recommend the following workaround:

1. When the story profile page loads, record the existence and state of all text areas earmarked for Xinha-fication, somewhere within bric's UI JavaScript scope.
1. Iterate through all of the Xinha-eligible text areas.
1. If a given text area is visible, load Xinha within it, and note that in the state variable.
1. If a given text area is __not__ visible, add a click event to its show/hide link, so that Xinha is applied the first time the user opens it (then the event removes itself).

So the three possible states of a Xinha-eligible element are:

1. Hidden, never shown, Xinha never loaded (action: load Xinha the first time the user expands the element).
1. Visible, Xinha never loaded (action: load Xinha at page load/ready time).
1. Hidden __or__ visble, Xinha loaded (action: none).

I think this would work better than the suggestions on the Xinha discussion forums (forcing them all to be visible at load time), because it avoids making the page jump around. Also, bric's current UI JavaScript remembers the previous collapsed/expanded state of an element, which we don't want to bork up.
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
theory updated this ticket at December 8th, 2010 @ 11:56 PM
* Assigned user changed from "Bugs (at bricolage)" to "theory"
* State changed from "new" to "open"
This issue is a giant PITA. I wish that Xinha would fix the underlying issue, whatever it is. I'm having a hell of a time making this work.
--------------------------------------------------------------------------------

Assigned: theory
State: open
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
theory updated this ticket at December 9th, 2010 @ 03:55 AM

Anyone using htmlArea, FCKEditor, or JS-Quicktags with 2.0?
--------------------------------------------------------------------------------

Assigned: theory
State: resolved
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
theory updated this ticket at December 9th, 2010 @ 04:13 AM

FCKeditor and JS-Quicktags continue to work fine, by my quick test. htmlArea is a bit funky, though. Anyone use it?
--------------------------------------------------------------------------------

Assigned: theory
State: resolved
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #217] Xinha WYSIWYG fields broken when collapsed on load. [ In reply to ]
// Add your reply above here
==================================================
battery CEO updated this ticket at December 10th, 2010 @ 09:21 AM

<a href="http://www.gpbattery.co.uk/"><h1>digital camera battery</h1></a>
<a href="http://www.gpbattery.co.uk/htc-p3300-pda-battery-lous.htm">htc p3300 battery</a>
<a href="http://www.gpbattery.co.uk/htc-tytn-pda-battery-lous.htm">htc tytn Battery</a>
<a href="http://www.gpbattery.co.uk/fujifilm-np-45-digital-camera-battery-lous.htm">np45 battery charger</a>
<a href="http://www.gpbattery.co.uk/htc-touch-diamond-pda-battery-lous.htm">htc touch diamond battery</a>
<a href="http://www.gpbattery.co.uk/fujifilm-finepix-f50fd-battery-charger-lous.htm">fujifilm finepix f50fd Battery</a>
<a href="http://www.gpbattery.co.uk/htc-s730-pda-battery-lous.htm">htc s730 battery</a>
<a href="http://www.gpbattery.co.uk/fujifilm-finepix-f200exr-battery-charger-lous.htm">fujifilm finepix f200exr Battery</a>
<a href="http://www.gpbattery.co.uk/htc-s620-pda-battery-lous.htm">htc s620 battery</a>
<a href="http://www.gpbattery.co.uk/htc-hermes-pda-battery-lous.htm">htc hermes battery</a>
<a href="http://www.gpbattery.co.uk/htc-p6500-pda-battery-lous.htm">htc p6500 battery </a>
<a href="http://www.gpbattery.co.uk/i-mate-jasjar-pda-battery-lous.htm">i-mate jasjar battery</a>
<a href="http://www.gpbattery.co.uk/htc-kais160-pda-battery-lous.htm">htc kais160 battery</a>
<a href="http://www.gpbattery.co.uk/fujifilm-np-40-battery-charger-lous.htm">fujifilm np-40 Battery</a>
<a href="http://www.gpbattery.co.uk/htc-wizard-pda-battery-lous.htm">htc wizard battery</a>
<a href="http://www.gpbattery.co.uk/htc-touch-dual-pda-battery-lous.htm">htc touch dual battery</a>

<a href="http://www.newbatterystore.com/"><h1>Battery on our store</h1></a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-samsung-vp-d250i.htm">samsung vp-d250i battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-olympus-x-790.htm">olympus x-790 battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-olympus-fe-340.htm"> olympus fe-340 battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-jvc-bn-vf823u.htm">jvc bn-vf823u battery </a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-panasonic-nv-gs230.htm">panasonic nv-gs120 battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-panasonic-lumix-dmc-fx60a.htm">panasonic lumix dmc-fx60a battery</a>
<a href="http://www.newbatterystore.com/iwop-camcorder-battery-jvc-gz-hd7.htm">jvc gz-hd7 battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-panasonic-lumix-dmc-fx180.htm">panasonic lumix dmc-fx180 battery</a>
<a href="http://www.newbatterystore.com/iwop-camcorder-battery-jvc-bn-v507b.htm">jvc bn-v507b battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-panasonic-nv-gs17ef-s.htm">panasonic nv-gs17ef-s battery</a>
<a href="http://www.newbatterystore.com/iwop-camcorder-battery-jvc-gr-d370ah.htm">jvc gr-d370ah battery</a>
<a href="http://www.newbatterystore.com/iwop-camcorder-battery-jvc-gr-ax970u.htm">jvc gr-ax970u camcorder battery</a>
<a href="http://www.newbatterystore.com/iwop-camcorder-battery-jvc-gz-ms130.htm">jvc gz-ms130 battery</a>
<a href="http://www.newbatterystore.com/iwop-digital-camera-battery-canon-nb-6l.htm">canon nb-6l battery</a>
<a href="http://www.newbatterystore.com/iwop-battery-charger-jvc-gz-mg130.htm">jvc gz-mg130 battery</a>
--------------------------------------------------------------------------------

Assigned: theory
State: resolved
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/217-xinha-wysiwyg-fields-broken-when-collapsed-on-load/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com