CometVisu - (interner) Beta-Test

[greentux]

Die Doku soll nur so gut sein, das derjenige, der die Userdoku erstellt, damit was anfangen kann. Irgendwie muss ja ersichtlich sein, was wie geht, ohne den Code verstehen zu müssen.
ICh nehm natürlich auch die Spec ...

[Chris M.]

doxygen wäre auch ne nette Variante.

Es gibt wohl ein Kommentar-Format für JS das sich gut für automatische Dokus verwenden lässt und das - fast noch wichtiger - den JS-Komprimierern erheblich bei der Arbeit hilft.

Leider hatte ich da noch keine Zeit einzusteigen, aber es wäre durchaus Ziel das auch hier einzuführen.

Eine End-User Doku wie hier gerade benötigt, wird da aber wohl nicht herausfallen...

Nachdem ich händisch die xml geschrieben hatte, habe ich dann mal den Editor aufgemacht. Der hat dann aus meiner xml sowas hier gemacht:

 <page name="Overview">
    <line/>
    <text align="center">Welcome to the home!</text>
    <line/>
    <group>

      Temp Aussen
      3/4/51


      Luftfeuchte Aussen
      3/4/52

    </group>

Richtig, vgl. Bug-Report https://sourceforge.net/tracker/?fun...6&atid=1251141 - das ist eines der Dinge die noch einem Release entgegen stehen.

[greentux]

komisches vom Editor

Nachdem ich händisch die xml geschrieben hatte, habe ich dann mal den Editor aufgemacht. Der hat dann aus meiner xml sowas hier gemacht:

 <page name="Overview">
    <line/>
    <text align="center">Welcome to the home!</text>
    <line/>
    <group>

      Temp Aussen
      3/4/51


      Luftfeuchte Aussen
      3/4/52

    </group>
    <info format="%02.1f °C">
      <label>Temp Aussen</label>
      <address transform="DPT:9.001" readonly="true" type="">3/4/51</address>
    </info>
    <info format="%02.1f %rel">
      <label>Luftfeuchte Aussen</label>
      <address transform="DPT:5.001" readonly="true" type="">3/4/52</address>
    </info>

Was bedeuten die Texte gleich nach dem <group> ? Die hat der Editor hinzugefügt. Bei mir ging es gleich mit den info Elementen weiter...

[greentux]

Der Doku fehlen im Moment Einträge zu "group" und zu "format".
Ich würde das gern ergänzen, kann aber wie gesagt nicht so recht im js code durchsteigen.
Das scheint mir ja auch auf mehrere Dateien verteilt. Wenn man mir also irgendwie das extrahieren könnte... doxygen wäre auch ne nette Variante.

[Chris M.]

Stefan, ich habe den Thread gelesen, die Doku im Wiki und die demo.xml Damit hat man ja fast alle Infos. Den JS Code kann ich mir leider nicht so recht anschauen...

Die Doku sollte (hm...) immer dem Stand des Codes entsprechen...

Aber hier kann ich sagen, dass es das noch nicht gibt. Der Grundidee nach hat jedes Widget nämlich nur eine GA, so wie bei anderen KNX-Teilnehmern auch (vgl. "eine Taste").

Klar gibt es ausnahmen wie den ColorChosser, aber da gilt das dann halt auf einem etwas abstrakterem Kontext.

Eine Lösung sind die neu eingeführten Gruppen, die zumindest die Zugehörigkeit der Werte zeigt.
Vom Platzbedarf dürfte dies Dich jedoch nicht zufrieden stellen...

Was ich mir auch schon ein paar mal gedacht habe, wären platzsparende "Status-LEDs"... So dass ich alle Fenster-Stati in zwei Zeilen sehe, o.ä.

Hier wird es v.a. wenn die Text-Visu zur 2D-Visu erweitert wird, weitergehen. Die Text-Visu ist im Grunde nämlich nur ein Proof-of-Concept um möglichst schnell was zeigen zu können und mit minimalem Aufwand eine grundlegende Visu hin zu bekommen.
Bei der 2D (und 3D) Visu wird man dann die Widgets in der Größe verändern und frei platzieren können.

[StefanW]

Aber ja, ich werds in den tracker packen.

Yeah, und schon ist es angelegt

[greentux]

Stefan, ich habe den Thread gelesen, die Doku im Wiki und die demo.xml Damit hat man ja fast alle Infos. Den JS Code kann ich mir leider nicht so recht anschauen... Daher fragte ich erst nach, obs das nicht doch gibt. Am End gibts da ein Attribut und schwups ist das schon da. Aber ja, ich werds in den tracker packen.

[StefanW]

Sind wir nicht derzeit im Feature-Freeze?

und Feature Requests für was neues gehört im Tracker auf SF angelegt.

[greentux]

Gibts das schon oder wärs ein neuer Wunsch:
zwei Werte nebeneinander in einem Info Feld. Also bsp: Temp/Feuchte eng beieinander anzuzeigen...

Gruß

[Chris M.]

Das war ja jetzt eigentlich auch nur als proof-of-concept gedacht, was wäre mit weiteren iframe Attributen, die genauso implementieren oder ein "style" Attribut

Da kann man sicher für beides gute Argumente finden.

Ich tendiere eher zu Einzelattributen als dem generischen Style. Kann die Tendenz aber nicht begründen.

[ctr]

Nö, genau so geht das. Kannst Du einfach im SVN Repository einchecken und jeder wird sich daran erfreuen können

Ok muß ich mal angucken wie das geht, hab zwar nen SF account, aber noch nie etwas eingecheckt ;-)

Das war ja jetzt eigentlich auch nur als proof-of-concept gedacht, was wäre mit weiteren iframe Attributen, die genauso implementieren oder ein "style" Attribut, was sich dann ggf auch mit mehreren Angaben füttern läßt? Wenn ich alle möglich iframe Attribute die ich gerne hätte dann sind das schon Einige, dass liesse sich aber auch als Einzeiler zusammenfassen, erfordert dann aber gültige CSS Syntax vom Nutzer anstatt nur Häkchen setzen und Farben ausfüllen...

[greentux]

Die svn version tut. also alles paletti...

[greentux]

Erster Versuch

Ich habe mal einfach die visu_config.xml angepaßt. Minimal. LEider sind die Änderungen nicht in der Visu sichtbar.
Habe ich irgendwas verpaßt? Da muß doch nix neu gestartet werden etc...

Gruß

[Chris M.]

Spricht etwas gegen diesen Weg? [...]

Nö, genau so geht das. Kannst Du einfach im SVN Repository einchecken und jeder wird sich daran erfreuen können

[ctr]

Ich hätte da noch einen Feature request für iframes:
Könnte man die iframe attribute konfigurierbar machen?
Also z.B:
style
frameborder
allowtransparency
background-color

Spricht etwas gegen diesen Weg? (Ich vermute das muß dann auch noch im xsd angepaßt werden?!)

--- svn/designs/structure_pure.js    2011-06-08 18:42:28.000000000 +0200
+++ customized/designs/structure_pure.js    2011-07-01 15:26:37.000000000 +0200
@@ -611,7 +611,6 @@
       var style = '';
       if( $p.attr('width' ) ) style += 'width:'  + $p.attr('width' ) + ';';
       if( $p.attr('height') ) style += 'height:' + $p.attr('height') + ';';
+      if( $p.attr('background') ) style += 'background:' + $p.attr('background') + ';';
       if( style != '' ) style = 'style="' + style + '"';
       var actor = '<div class="actor"><iframe src="' +$p.attr('src') + '" ' + style + '></iframe></div>';
       ret_val.append( $(actor) ); 
@@ -620,8 +619,7 @@
     attributes: {
       src:    { type: 'uri'   , required: true  },
       width:  { type: 'string', required: false },
+      height: { type: 'string', required: false },
+      background: { type: 'string', required: false }
-      height: { type: 'string', required: false }
     },
     elements: {
       label:  { type: 'string',    required: false, multi: false }
@@ -979,3 +977,4 @@
   
   return value;
 }