Upgrade svn 1830 -> 1853

[MicHau]

Ich habe es auch mal durchprobiert. Es passte alles und ich habe es einfach commited.
Also, in der letzten Fassung sollte es nun wieder funktionierten mit dem Standard-Minus.

[XueSheng]

@MicHau
Hab Deinen Änderungsvorschlag mal kurz getestet: Das "-" wird wieder angezeigt, danke! Ob etwas dagegen spricht, müssen andere beurteilen.

[XueSheng]

<break> hab ich gefixt (1854). Ein colspan o.ä.ist nicht notwendig (was hätte der auch bei einem <break> für einen Sinn?)

break geht wieder, danke!
colspan bei break? Hätte ja sein können, dass break nun wie ein tab interpretiert wird und Elemente für den definierten colspan verschiebt.

Kannst Du hier bitte mal checken, ob das mit Revision 1838 auch schon der Fall ist?
Da gab es eben zwischen 1830 und 1838 eine Änderung die evtl. relevant ist - und ab 1839 sind meine Aufräum-Änderungen gekommen...

Mit Rev. 1838 ist das Problem auch vorhanden.

Das ist dann quasi zwingend ein Cache-Problem.
=> Bitte lösche mal den Browser-Cache

Hatte die Option zum Cache Leeren nicht gefunden und dachte daher, dass das Cache Managent evtl. anders funktioniert. Nun gefunden unter "Settings->Privacy->Clear Browsing Data". Problem gelöst, danke!

[MicHau]

Kannst Du hier bitte mal checken, ob das mit Revision 1838 auch schon der Fall ist?
Da gab es eben zwischen 1830 und 1838 eine Änderung die evtl. relevant ist - und ab 1839 sind meine Aufräum-Änderungen gekommen...

Übeltäter ist gefunden: _common.js, Zeile 196

valueElement.empty();

Das hätte aber auch schon vorher schief gehen müssen, ich weiß aber, dass ich das damals gecheckt hatte, ob das Minus erhalten bleibt. Sehr seltsam.

Ich würde das so umschreiben.
Alte Fassung ab Zeile 196:

    valueElement.empty();
    if (undefined !== value) {
      if (('string' == typeof value) || ('number' == typeof value))
        valueElement.append( value );
      else if ('function' === typeof value)
        value( valueElement );
      else {
        for (var i = 0; i < value.length; i++) {
          var thisValue = value[i];
          if (!thisValue) continue;
  
          if( ('string' == typeof thisValue) || ('number' == typeof thisValue) )
            valueElement.append( thisValue );
          else if( 'function' === typeof thisValue )
            thisValue(valueElement);
          else
            valueElement.append($(thisValue).clone());
        }
      }
    }

Neue Fassung:

    valueElement.empty();
    if (undefined !== value) {
      if (('string' == typeof value) || ('number' == typeof value))
        valueElement.append( value );
      else if ('function' === typeof value)
        value( valueElement );
      else {
        for (var i = 0; i < value.length; i++) {
          var thisValue = value[i];
          if (!thisValue) continue;
  
          if( ('string' == typeof thisValue) || ('number' == typeof thisValue) )
            valueElement.append( thisValue );
          else if( 'function' === typeof thisValue )
            thisValue(valueElement);
          else
            valueElement.append($(thisValue).clone());
        }
      }
    }
[B]    else {
      valueElement.append('-');
    }[/B]

Spricht irgendetwas dagegen?

[Chris M.]

Allerdings scheint auch der "break" betroffen zu sein (siehe Anhang). Allerdings kann ich hier im Editor kein Layout hinzufügen. Muss da auch ein definierter colspan rein oder ein anderes Problem?

<break> hab ich gefixt (1854). Ein colspan o.ä.ist nicht notwendig (was hätte der auch bei einem <break> für einen Sinn?)

Die Icons inkl. Mapping sind nun auch wieder da. Die fehlerhafte Darstellung bezieht sich nur auf den Ladevorgang der Visu. Das hängt sicherlich damit zusammen, dass es nun einen default Wert für Mappings gibt. Vorher wurde während dem Laden ein "-" angezeigt. Dadurch, dass das Minus fehlt, passen die Button Größen nicht und icon Positionen sind nicht sichtbar. Da muss ich mal den passenden Thread raussuchen.

Kannst Du hier bitte mal checken, ob das mit Revision 1838 auch schon der Fall ist?
Da gab es eben zwischen 1830 und 1838 eine Änderung die evtl. relevant ist - und ab 1839 sind meine Aufräum-Änderungen gekommen...

Ebenso habe ich das Problem, dass Chrome auf Android die Visu nur noch läd, wenn ein "&forceReload=true" angefügt wird. Ansonsten bleibt er ohne Fehler beim Laden hängen (siehe Screenshot). Wie kann man denn den Ladevorgang mit Chrome auf Android debuggen?

Das ist dann quasi zwingend ein Cache-Problem.
=> Bitte lösche mal den Browser-Cache

[XueSheng]

Die Default-Werte sind vom Minus unabhängig. Es wird initial ein Minus reingeschrieben, erst danach wird nach einem Default-Wert geschaut. Ich habe das Phänomen auch gestern gehabt. Musste Firefox schließen, lokale Daten löschen und neu starten. Dann ging es wieder.

Ich habe es bislang noch nicht geschafft, dass das default "-" beim Laden erscheint (Firefox 23.0). Habe auch Chromium (Ubuntu 13.04) installiert und die Visu geladen. Selbes Phänomen. Auf meinem Android Gerät mit Chrome ebenso. Aus Spaß auch mal mit einem Win7 und Firefox 23.0.1 getestet... auch kein default "-" zu sehen.

Keine Ahnung was ich noch tun kann. Da muss wohl doch noch ein Bug versteckt sein.

Ebenso habe ich das Problem, dass Chrome auf Android die Visu nur noch läd, wenn ein "&forceReload=true" angefügt wird. Ansonsten bleibt er ohne Fehler beim Laden hängen (siehe Screenshot). Wie kann man denn den Ladevorgang mit Chrome auf Android debuggen?

[swiss]

Ach soo Ich habe mich nur gewundert, wesshalb Checkconfig neuerdings mekkert obwohl die Config nicht angefasst wurde. Habe mich gar nie geachtet ob die beiden Einträge etwas bewirken ;-)

[MicHau]

Wiso ist eigentlich seit neustem Mapping und Styling nicht mehr erlaubt beim Multitrigger?

Ich hatte festgestellt, dass das überhaupt nicht verwendet wird im Code und es daher auch aus der XSD entfernt.

[Chris M.]

Das sind die Leiden, wenn man die aktuellste Entwicklerversion einsetzt. Sorry, ist so, wir haben auch immer davor gewarnt.

Die Fehler sollten natürlich alle nicht sein. Das mit der <line> ist zwar ärgerlich aber leider nicht vermeidbar, bzw. dank colspan ist nun auch mehr möglich als vorher...
Die anderen Punkte sehe ich mir an und schreibe hier, wenn ich die behoben habe.

Einen Feature-Freeze haben wir aktuell noch nicht. Der wäre sonst öffentlich angekündigt.
Was wir aber durchaus haben, ist eigentlich eine Phase der Konsolidierung die zum Release hinführen soll.
Mir ist klar, dass meine aktuellen Änderungen dem etwas widersprechen. Andererseits halte ich den Vorteil dieser Änderungen für so groß, dass ich hier das Wagnis eingehe. Und da ich hier aufräume, ist eine stabilere Phase der Entwicklung sehr wertvoll, denn da lassen sich diese Änderungen am saubersten durchführen und testen.

[swiss]

Wiso ist eigentlich seit neustem Mapping und Styling nicht mehr erlaubt beim Multitrigger?

[MicHau]

Allerdings scheint auch der "break" betroffen zu sein (siehe Anhang). Allerdings kann ich hier im Editor kein Layout hinzufügen. Muss da auch ein definierter colspan rein oder ein anderes Problem?

Das ist mir bisher nicht bekannt. Es ist im Code auch kein Layout-Element vorgesehen. Das müsste noch mal überprüft werden.

Das hängt sicherlich damit zusammen, dass es nun einen default Wert für Mappings gibt. Vorher wurde während dem Laden ein "-" angezeigt. Dadurch, dass das Minus fehlt, passen die Button Größen nicht und icon Positionen sind nicht sichtbar.

Die Default-Werte sind vom Minus unabhängig. Es wird initial ein Minus reingeschrieben, erst danach wird nach einem Default-Wert geschaut. Ich habe das Phänomen auch gestern gehabt. Musste Firefox schließen, lokale Daten löschen und neu starten. Dann ging es wieder.

[XueSheng]

Danke für die schnelle Rückmeldung.

line habe ich nun um colspan=0 ergänzt.
Allerdings scheint auch der "break" betroffen zu sein (siehe Anhang). Allerdings kann ich hier im Editor kein Layout hinzufügen. Muss da auch ein definierter colspan rein oder ein anderes Problem?

Die Icons inkl. Mapping sind nun auch wieder da. Die fehlerhafte Darstellung bezieht sich nur auf den Ladevorgang der Visu. Das hängt sicherlich damit zusammen, dass es nun einen default Wert für Mappings gibt. Vorher wurde während dem Laden ein "-" angezeigt. Dadurch, dass das Minus fehlt, passen die Button Größen nicht und icon Positionen sind nicht sichtbar. Da muss ich mal den passenden Thread raussuchen.

[MicHau]

<line>-Elemente benötigen ein <layout colspan="0"/>-child im Navbar, andere <line>-Elemente brauchen auch ein entsprechendes <layout>-Child.
Hier zu finden: https://knx-user-forum.de/333613-post28.html

Da sich die Geister am Editor scheiden, ist das mit den "gravierenden Bugfixes" eher Wunschdenken. Es wird anscheinend noch ein bisschen was geändert werden, wenn ich das richtig interpretiere. Und ein Release ist irgendwie noch nicht richtig in Sicht.

[XueSheng]

Habe eben ein Upgrade von svn 1830 zu 1853 vollzogen. Trotz Leeren des Browser Cache (Firefox 23.0, Ubuntu 13.04) ist nun die komplette Visu "hinüber". Icons fehlen, mapping fehlerhaft, Top Navbar passt nicht mehr, generelles Layout ist verschoben (screenshots vorher/nachher angefügt).

Bevor ich nun mehrere Stunden investiere meine Config anzupassen, wollte ich mal nachhaken, ob die aktuelle svn evtl. grundlegende Fehler enthält, die bekannt sind? Ich dachte es kommen aktuell nur noch gravierende Bugfixes bis zum Release!?

Falls mehr Infos nötig, liefere ich diese gerne nach!