Hi, On Wed, Nov 4, 2009 at 3:22 PM, Marius Dumitru Florea < [email protected]> wrote:
> Hi Guillaume, > > Guillaume Lerouge wrote: > > Hi Marius, > > > > On Wed, Nov 4, 2009 at 2:20 PM, Marius Dumitru Florea < > > [email protected]> wrote: > > > >> Embed HTML elements pose some problems: > >> > >> * they are loaded asynchronously and any attempt to access even the > >> simplest DOM property like nodeType through JavaScript raises an > >> exception if the object is not fully loaded (closed to > >> https://bugzilla.mozilla.org/show_bug.cgi?id=331672 ). > >> > >> * they can't be fully controlled from JavaScript. The specific plugin > >> that handles each of them decides the display and behavior (e.g. > >> https://bugzilla.mozilla.org/show_bug.cgi?id=250855 a flash movie might > >> auto start in edit mode and, unless you want to see a movie while > >> editing, that's not nice). > >> > >> * they aren't reloaded after they are detached and reattached to the DOM > >> tree using JavaScript which means that any DOM manipulation involving > >> one of the ancestors of a embed node can lead to the disappearance of > >> the embedded object. Sadly, appendChild, insertBefore and replaceChild > >> detach the target node before attaching it to the new location. As a > >> consequence there's no way to move an embed node inside the DOM tree > >> without detaching it. > >> > >> So I propose we replace the embed HTML elements with static image > >> placeholders just after the edited document finishes loading. The > >> placeholder will have the same size as the embedded object in view mode. > >> > >> Note, ckeditor already does this. You can check by copying the HTML from > >> http://jira.xwiki.org/jira/browse/XWIKI-3975 to the source of > >> http://ckeditor.com/demo . > >> > >> I'm +1. > >> > > > > Sounds good. It will also make the page easier to edit since it will be > less > > heavy. > > > > Only question: where do we display text for that image? The information > text > > we put with the image has to be easy to internationalize. > > Indeed that's something to think about. One option is to use a generic > icon that stands for "Embedded Object" in the center of the placeholder > (which also has a border) and use a translatable text as the tool tip of > the placeholder. WDYT? > Maybe we could use a DIV + text in the DIV + the background-image CSS property so that the text can be internationalized easily yet is diaplayed above the image. WDYT? Guillaume CKeditor uses an image that contains some text (e.g. "FLASH") but that's > not i18n friendly. > > Marius > > > > > Otherwise +1 too. > > > > Guillaume > > > > > >> Thanks, > >> Marius > >> _______________________________________________ > >> devs mailing list > >> [email protected] > >> http://lists.xwiki.org/mailman/listinfo/devs > >> > > > > > > > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > -- Guillaume Lerouge Product Manager - XWiki SAS Skype: wikibc Twitter: glerouge http://guillaumelerouge.com/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
