Add a "Loading..." message to Xinha

[anonymous]

It can take a long time for Xinha to load, it needs a panel in front of it that says it is loading so people don't think it is broken.

[mokhet]

This patch has been tested for IE6 and FF 1.0.4/1.5rc2 only, seems to do the trick, but i'm much too scared to submit it on the repository since it's updating htmlarea.js and htmlarea.css, it's very new (just finished the first working version) and i bet betters devs than me will find a lot of stupid stuff in it or a better way to do it.

Anyway, if this patch do the trick for you mister anonymous, it's good enough :D

[mokhet]

added a config.showLoading variable

[mokhet]

in this second version of the patch i have added a variable in the config object (config.showLoading) set to true by default. Make it false to not show the loading panel message.

[gogo]

Havn't tried it out yet but a couple of suggestions. It should probably be off by default for now, and I don't think you are running the strings through the localiser so they won't be translated.

All you need to do to localise is

HTMLArea._lc("English Goes Here")

[mokhet]

set the config variable false by default

[mokhet]

this third version is setting the config variable false by default as suggested by gogo. The strings were already running throuh the localiser

HTMLArea.prototype.setLoadingMessage = function(string, context)
{
  if (!this.config.showLoading || !document.getElementById("loading_sub_" + this._textArea.name)) { return ; }
  var elt = document.getElementById("loading_sub_" + this._textArea.name);
  elt.innerHTML = HTMLArea._lc(string, context);
};

[anonymous]

I tried the second patch but didn't see the loader... I'll try again with the third patch.

[gocher]

It seams to work! de.js extensions:

  "Loading in progress. Please wait !": "Editor wird geladen. Bitte warten !",
  "Constructing main object": "Hauptteil wird erzeugt",
  "Create Toolbar": "Bearbeitungswerkzeuge werden angelegt",
  "Register panel right": "Erzeugt rechte Leiste",
  "Register panel left": "Erzeugt linke Leiste",
  "Register panel top": "Erzeugt obere Leiste",
  "Register panel bottom": "Erzeugt untere Leiste",

[anonymous]

This is a lot of changes to make. It would be great if this was checked in, especially since the default is "off."

[anonymous]

This greatly improves the usability of Xinha, because the user isn't left wondering if something has broken. "Loading" messages can make people not notice or mind delays as much, and progress status messages often makes users less impatient by giving them an understanding of how much longer it will take, and in this case better yet, what is happening.

[ray]

This should really in the distribution! It works great. Additionally you could make a public function out of it because I assume there are occasions where plugins would show a message to the user.

[mokhet]

the function is in the prototype of HTMLArea and so is public and accessible from plugins.

Watching the last patch make me remember i had started an idea to allow sub messages but i didnt finished.

So i wonder what is the status of this ticket and what gogo and others are thinking about the proposed patch.

And i wonder what action should i take :

1) resolve as wontfix
2) wait for a better solution
3) commit the third patch as it is already (bad idea imo)
4) make a fourth version and submit it on this ticket, waiting for approval
5) or make the fourth version, commit it and close this ticket as fixed

thanks.

[anonymous]

I sure wish you could do version 5, if it means adding it to the core Xinha.

[anonymous]

I mean action 5 (sorry).

[mokhet]

patch updated and commited in changeset:471

[anonymous]

Az oldal betöltés alatt... Kérem várjon!