rombau / osext2

Online Soccer Extension based on Chrome API
MIT License
1 stars 0 forks source link

Parallel-Betrieb mit GM-Scripts #9

Closed rombau closed 2 years ago

rombau commented 2 years ago

Vielleicht gibt es auch eine Möglichkeit, die Scripts statt in Grease/Tampermonkey in der Extension einzubinden?

rombau commented 2 years ago

Die GM-Scripts dürften direkt in die Seite "injiziert" (in der Konsole wird als Quelle z.b. showteam.php angegeben) und vor den Content Scripts der Erweiterung ausgeführt werden. Somit kommt es zu Problemen, wenn die Erweiterung ein bestimmtes Layout voraus setzt.

Die Erweiterung sollte prüfen, ob Layout-Änderungen durch die GM-Scripts vorgenommen wurden (z.B. der Optionen-Button hinzugefügt wurde).

rombau commented 2 years ago

In den GM-Scripts gibt es kein @run-at, somit laufen die Scripts mit document-end. Die Extension Scripts laufen run_at "document_idle", also nach den GM-Scripts.

Eselce commented 2 years ago

Somit haben sie dasselbe Problem wie konkurrierende GM-Scripts...

Eselce commented 2 years ago

[quote='Michael Bartz','index.php?page=Thread&postID=1243605#post1243605']

Kein direkter Konflikt, aber eine Unschönheit wegen Doppelung in der Anzeige:

# Name Alter Geb. Land U SCH BAK KOB ZWK DEC GES FUQ ERF AGG PAS AUS UEB WID SEL DIS ZUV EIN FixVals Prios Skill Opti tr. Talent Skillschn. Opt.Skill

Skript Extension

Eselce commented 2 years ago

[quote='Sven Loges','index.php?page=Thread&postID=1243606#post1243606']

Danke Dir! Da fällt mir gleich noch ØP (Extension) vs. Prios (Skript) ein.

Ich stelle mir da derzeit vor, daß die doppelte Anzeige einfach erkannt und weggelassen wird... Also kooperativ! Ich versuche das mal im Detail zu klären, wer da auf wen achten muß...

rombau commented 2 years ago

möglicherweise auch ein Problem im Parallel-Betrieb: #26

rombau commented 2 years ago

möglicherweise auch ein Problem im Parallel-Betrieb: #26

Bei Verwendung der "Abkürzungen" im Skript kann die Spielart (z.B. "FSS (H)" statt "Friendly : Auswärts") nicht mehr korrekt initialisiert werden. Hat aber nichts mit #26 zu tun.

rombau commented 2 years ago

die Ziehwarnung aus dem Skript darf von der Erweiterung nicht (mit den anderen unnötigen "Hilfezeilen") gelöscht werden

rombau commented 2 years ago

Das Löschen der Ziehwarnung wird (u.a.) mit https://github.com/rombau/osext2/commit/747541c70b43ceac88e7a4d767c843cb884e5f86 verhindert.

rombau commented 2 years ago

Alle aktuell bekannten Probleme wurden mit #34 gelöst. Sollten noch Fehler auftreten, werden diese mit eigenen Issues behoben.