Beiträge von Thunder

Zitat von Micky3248

Da ich die Updatefunktion von 102.15 auf 115.2 in Thunderbird verwendet habe

Bevor Diskussionen entstehen: Ja, das "manuell getriggerte" Update per Hilfe → Über Thunderbird ist seit gestern seitens MZLA aktiviert worden.

Zitat von mxr

das von sztbm beschriebene Verhalten kann ich nicht bestätigen, bei mir klappt beides ("Allow HTML Temp" und "Spam Scores") zusammen bei einem ersten Test problemlos.

Bei mir auch

Zitat von sztbm

wenn ich das Add On "Spam Score" aktiviere

Meinst Du dieses hier?

Zitat von Sepp_I

einem <pre> vorne und einem </pre> hinten

Die gehören natürlich entfernt. Das kommt manchmal davon, wenn man per Copy&Paste arbeitet.

Zitat von mxr

muss ich noch in den Debug

Ich hätte da auf jeden Fall nochmal rein geschaut - schon aus Neugierde, ob da jetzt die Einträge erscheinen, die gewollt sind.

Offensichtlich hat sich Thunderbird mal wieder an seinem Cache verschluckt. Ein Programmstart mit der Option -purgecaches hätte vermutlich ebenso das Problem behoben.

Deinstalliere das Add-on mal, beende Thunderbird, starte Thunderbird wieder neu und installiere das Add-on neu. Dann gehe in die Einstellungen des Add-ons und aktiviere dessen Debug-Modus.

In der Konsole müsste es bei (pur) "Debug" ungefähr so aussehen:

Zitat von mxr

gerade fällt mir auf, dass bei zwei von fünf Konten bei vielen eMails (nicht bei allen!) *kein* "Antworten"-Button mehr angezeigt wird - kA warum das so ist, kann das ggf zusammenhängen, oder soll ich ein neues Thema aufmachen?

Mit rechter Maustaste kommt 'Antworten', aber nicht in der Headeranzeige... Teils liegen die Konten auf demselben Server, daran kann's also nicht liegen...

Lass das erstmal beiseite. Da kann man später drauf zurück kommen - evtl. dann tatsächlich in einem separaten Thema.

Mir fehlt noch ein Screenshot nach befolgen meiner Anweisungen, wenn der "Debug-Modus" in den Einstellungen meines Add-ons aktiviert wurde.

1.) Welche anderen Add-ons hast Du installiert?

2.) Schaue bitte mal in die Fehler-Konsole des Thunderbird. Benutze dort das Papierkorb-Icon, um erstmal alle Einträge in der Konsole zu löschen. Dann wechsle bitte mal den ausgewählten/angezeigten Ordner der Mails und klicke eine E-Mail an, damit sie angezeigt wird. Was ist jetzt in der Fehler-Konsole zu sehen? Screenshot bitte.

3.) Aktiviere in den Einstellungen des Add-ons mal den "Debug-Modus" und wiederholte das Prozeder aus Nummer 2)

Zitat von schlingo

der es eigentlich wissen müsste

...der es vermutlich eigentlich besser wissen müsste...

Man kann von Downgrades nur grundlegend abraten. Jedenfalls sollte niemand so naiv sein, zu glauben, dass auf lange Sicht alles gut ist, nur weil innerhalb der ersten Stunden und Tage nach einem Downgrade nichts besonderes auffällt.

Okay, Version 9.0.0 wartet jetzt auf ATN auf Überprüfung und Freigabe.

Wenn John aus Kanada zurück ist, kann man vielleicht noch die Art und Weise, wie der Remote Content erlaubt wird, verbessern. Außerdem werde ich noch ein paar Funktionen im Code konsolidieren, die momentan durch das neue Konzept auf 2 Dateien verteilt und vom Code her quasi doppelt sind (womit auch Fehler eher auftreten und schwieriger zu finden sein könnten).

Zitat von jobisoft

Ja, die gesamte mail3pane ist doch jetzt anders. Dein target ist nicht mehr in messenger.xhtml sondern in about:message. Ich denke du musst

let window = event.target.ownerGlobal.top;

nehmen, um an das tabmail zu kommen.

Wenn man sich auskennt, kann es so einfach sein - vielen Dank! Das war es.

Zitat von jobisoft

Welchen benutzt du (der nicht geht)?

Du hattest dieses Konstrukt mit mir geschaffen:

Zitat von Thunder

die manipulierte remoteContentBar wieder eingebaut werden muss, und da brauche ich nun Hilfe.

Ich habe die Manipulation und den click listener einbauen können. Mein Code ist bestimmt verbesserungsfähig

Es bleibt noch ein Problem damit zu beheben. in der experimentellen API bekomme ich derzeit nicht mehr die korrekte tab.id mit Hilfe des Codes, den wir bisher verwendet hatten. Komischer Weise wird kein Fehler geworfen, sondern man bekommt jeweils eine falsche tab.id. Leider ist die tab.id aber zentral wichtig für die (weitere) Funktion des ganzen Add-on-Codes und bspw. den Reload des korrekten Tabs etc.

Zitat von jobisoft

Evtl schonmal releasen mit Hinweis auf feature-incomplete?

Das werde ich wohl so machen, um gleich auch mehr Feedback zu erhalten.

Zitat von Thunder

Mist, es gibt einen weiteren Bug durch die Funktion getButtonPolicy. Diese berücksichtigt nämlich nicht evtl. gedrückte Modifier Keys, sondern liest einfach stupide die Options von AHT aus. Das muss ich auch noch fixen.

Behoben

Zitat von Thunder

Wenn die Default Policy identisch mit der Button Policy ist, dann wir die Mail nicht neu geladen und zusätzlich versagt deswegen dann auch der onMessageDisplayed Listener

Behoben

Somit bin ich jetzt erneut an einem Punkt, von dem ich glaube, dass jetzt "nur" noch die manipulierte remoteContentBar wieder eingebaut werden muss, und da brauche ich nun Hilfe.

Derzeitiger Stand:

Weiterer entdeckter Bug:

Wenn die Default Policy identisch mit der Button Policy ist, dann wir die Mail nicht neu geladen und zusätzlich versagt deswegen dann auch der onMessageDisplayed Listener, der zwingend den obigen Block auf false zurück stellen müsste und den Tab wieder zu den Prefs reconnecten müsste. An der Stelle rächt sich wahrscheinlich, dass aus meiner messageContentPolicy das Feature des (erzwungenen) Reload heraus gelöscht wurde.

Wann kommt es vor, dass man bei identischer Policy die Mail per Button neu laden will? Das ist notwendig, wenn die Mail per Button bspw. als HTML geladen wurde und man dann per Shift+Button wieder zu Reintext zurück will. In diesem Fall wäre die Default Policy = plaintext und die Button Policy plötzlich auch = plaintext (was im derzeitigen Stand auf GitLab schon für sich versagt, weil die Modifier Keys da vollkommen übergangen werden). Die Policy kann aber auch nahezu identisch sein, wenn man von HTML per Klick zu HTML+RemoteContent gelangen will. Da kommt es dann drauf an, wie genau man die Policies im Code vergleicht. Ich habe hier nämlich leider erst jetzt entdeckt, dass man Objekte gar nicht so einfach in JS vergleichen kann. jobisoft hatte für "isCurrentPolicy" ursprünglich auch nur den Teil "msgBodyAs" aus dem Objekt verglichen. Ich habe dies naiv und primitiv auf das ganze Objekt ausgeweitet, was beim Vergleich mittels "==" aber immer "false" ergibt. Je nach Situation in der Logik des Codes muss ich da also auch nochmal ran.

Mein alter Code erschien kompliziert, aber der war für die vielen möglichen Fallstricke durchaus ausgereift und strukturiert

Mist, es gibt einen weiteren Bug durch die Funktion getButtonPolicy. Diese berücksichtigt nämlich nicht evtl. gedrückte Modifier Keys, sondern liest einfach stupide die Options von AHT aus. Das muss ich auch noch fixen.

jobisoft

Dieser commit macht folgendes:

1.) Dein ignorePolicyChange wird so abgeändert, dass es die Tabs tatsächlich erst wieder reconnected, wenn AHT mit seinen messagePolicy updates fertig ist. Somit wird die per Klick mit HMTL geladene Mail nun auch erfolgreich angezeigt, wenn weitere AHT-Optionen aktiviert sind (Remote Content und/oder Inline Attachments). Es erübrigt sich auch mein comment out, welcher die Tabs ja dauerhaft von den default prefs entkoppelt hatte. Ich denke, dass müsste Dir so gefallen.

2.) Ich habe Deinen obigen Vorschlag so eingebaut, der die Button-Aktion blockiert, solange die erste Aktion nicht vollständig abgeschlossen ist. Deine Logik habe ich umgekehrt, weil es mir günstiger erschien. Ich denke eigentlich, dass man gar kein Array mit den verschiedenen Tabs nutzen müsste. Vermutlich würde ein globaler Block ausreichen oder sogar noch "sicherer" sein. Aber das mit dem Tab-Array scheint mir okay zu sein.

Somit steht jetzt nur noch das Wiedereinbauen der RemoteContentBar-Manipulation aus, wenn ich nicht irre. Wobei auch die Verwendung von setMsgHdrPropertyAndReload("remoteContentPolicy", kAllowRemoteContent); steht noch aus.