[skinlcarsng] segfault in displaymenu.c (gelöst)

Zitat von rell

Andere Frage: Wie offen bist du für pull requests?

Da bin ich sehr offen. Da ich selber nicht der große Designer bin, kann es ja nur besser werden.

Grüße

kamel5

Zitat von rell

Aber wie oben beschrieben, würde ich dahingehend im Plugin nichts ändern

OK, der Einheitlichkeit halber würde ich das aber trotzdem machen, da bei den anderen Texten diese Abfrage auch schon vorhanden ist.

Grüße

kamel5

Zitat von rell

Habe mal den fix ins Ausgabeplugin eingecheckt.

Danke für die Infos. Habe es nun in meinen Plugins auch nachgezogen

Zitat von rell

n cOglCmdDrawText könntest du der Vollständighalber vorab bei !symbols[0] aussteigen, denn in dem Fall geht er später eh nicht in die for-Schleife und du sparst dir die ganzen Bind()...

Gute Idee, werde ich einbauen.

rell ,

Zitat von rell

vor allem DrawRaster ist sehr nützlich.

Das habe ich von einem Deiner Beiträge abgeschaut.

Zitat von rell

wäre evtl. auch eine kleine Optimierung

Das schaue ich mir an.

Zitat von rell

Außerdem ist mir aufgefallen, dass der Text-Hintergrund der Kanalnummer der 2. Timers etwas über den Zwischenraum zeichnet.

Ich dachte eigentlich, das hätte ich gefixt. Ist vielleicht beim rebase verloren gegangen. Ich schaue nochmal.

Ansonsten will ich vor dem nächsten Release noch einige Dinge im Menü gerade ziehen.

Wenn Du mir noch mal genau schreibst, was Du hier meinst:

Zitat von rell

- Menu-Ansicht oben um ein kleines Rechteck niedriger

- Buttons auf lcars-Standardverhältnis/-größe und wo anders hin, damit

- das das Videofenster oben rechts Platz hat, ohne dass die obere Leiste so hart geteilt wird

könnte ich mir das auch noch ansehen. Vor allem das Erste und Dritte. Wenn man oben ein Rechteck weg lässt, wird ja auch das Videofenster kleiner und das wollte ich eigentlich nicht.

Und wenn Du noch zusätzliche Themes hast, nehme ich auch gerne mit auf.

Grüße

kamel5

Zitat von rell

Ich persönlich finde das Videofenster zu groß

Zitat von rell

Ich fänds cooler, wenn die obere (blaue) horizontale Leiste nicht vor dem Videofenster nach oben geht, sondern gerade durchgeht und das Video einfach oben rechts drin sitzt. Das "Live" würde ich auch direkt ins Videofenster zeichnen. Evtl. könnte man es auch seitlich noch einfassen. Da müsste ich mal schauen, was es da im Original-lcars so gibt.

OK, das Videofenster könnte ich in 2 Größen konfigurierbar machen, und bei kleinem Video die Leiste nach rechts durchziehen.

Zitat von rell

Siehe hier. Die Stelle würde schon passen, nur das Verhältnis der Ellipse passt nicht zun den Buttons, wie sie der Lcars-Standard definiert.

Ja, an manchen Stellen bin ich ein wenig vom Standard abgewichen, die Ellipse kann ich aber anpassen.

Zitat von rell

Das wird erst beim Menü richtig interessant

Das habe ich auch schon gedacht, na mal sehen, was da raus kommt.

Grüße

kamel5

OK,

ich habe jetzt soweit alles auf einen Stand gebracht, wo ich denke, das das so eine Weile Bestand haben könnte.

Unter anderem auch ein kleineres Videofenster, kofigurierbar im Setup. Das Raster lässt sich jetzt über Make und eine Setup-Option ein-und ausschalten. Und im Menü habe ich das Raster etwas situationsabhängig gemacht , da ist es etwas übersichtlicher.

Du kannst ja mal schauen, ob es mit dem kleinen Videofenster jetzt soweit OK ist.

Ich werde auch noch ein paar Tests machen und wenn ich nichts weiter finde, dann in absehbarer Zeit als Release freigeben.

Grüße

kamel5

Schön, das es gefällt.

Zitat von rell

- Im replay fehlt oben ein Balken, wo die Lautstärkeregelung kommt, ist das Absicht?

Ja, das war im Moment Absicht, ich wollte so wenig wie möglich von Videofenster verdecken. Ich kann da aber auch einen Balken hin machen, vielleicht ist da auch ein halb hoher Balken (wie in der Kanalansicht) OK?

Zitat von rell

- Was auf meiner Liste noch steht, ist, dass ich persönlich gerne die Timer rechts und did EPG Info links hätte. Ich habe mich immer noch nicht dran gewöhnt. Das wäre wahrscheinlich vom Design relativ einfach möglich, aber die Beschreibung der nachfolgenden Sendung hätte links keinen Platz. Wäre mir nicht wichtig, da ich dann eh in tvguide gehe..

Ja, das wird etwas schwierig, da ist nicht viel Platz. Am Anfang habe ich auch darüber nachgedacht, das dann aber verworfen. Ich würde das erst einmal so lassen...

Zitat von rell

- Wenn einer Timer aufnimmt, zeigt es bei mir

Rec: #(null) - 19: an. Da ist der Platz dann aus und mit #(null) passt was nicht.

Das muss ich mal beobachten, ob sich da wieder was eingeschlichen hat, denn das ging ja schon mal.

Zitat von rell

- Lautstärkeleiste: Ich würde keine Einzelblocks setzen, sondern den Balken ohne Zwischenräume zeichnen

Das habe ich vom Original-LCARS übernommen, das kann ich aber auch konfigurierbar machen.

Grüße

kamel5

Zitat von rell

Ich habe angefangen, das etwas umzugestalten. Wäre per Setup konfigurierbar. Ich kann dir den Patch schicken, wenns fertig ist.

Das schaue ich mir dann gerne an.

Zitat von rell

Das wäre ein Patch dafür.

Ich hatte da auch schon eine einfache Lösung gefunden und dann gleich noch einen Farbverlauf mit eingebaut .

Ist jetzt im git.

Grüße

kamel5

Zitat von rell

- Wenn einer Timer aufnimmt, zeigt es bei mir

Rec: #(null) - 19: an. Da ist der Platz dann aus und mit #(null) passt was nicht.

Das habe ich mal eine Weile beobachtet und konnte es bei mir nicht feststellen.

Im Prinzip kann das aber vorkommen, wenn es keine Devicenummer gibt. Warum es aber kein Device oder keine Devicenummer gibt, ist mir nicht ganz klar, denn ohne Device kann es ja auch keine Aufnahme geben.

Ich habe das jetzt so umgebaut, das ein "?" ausgegeben wird, wenn es keine Devicenummer gibt.

git ist aktualisiert.

Grüße

kamel5

Zitat von rell

Vielleicht liegts daran, dass die Aufnahmen hier vom Server gemacht werden und nicht die Devices vom Client benutzt werden...

Remotetimer hatte ich bisher noch gar nicht im Fokus, das erklärt es aber. Und dann ist es auch bestimmt besser, die Devicenummer bei Remotetimern ganz weg zu lassen.

Grüße

kamel5

Ich habe den develop Branch nochmal aktualisiert (doch noch ein paar Bugs gefunden) .

Die Timeranzeige ist jetzt soweit geändert, das im Falle von Remotetimern bei vorhandenen wareagleicons ein Symbol angezeigt wird.

Falls sich momentan keine Unstimmigkeiten mehr finden, werde ich gegen Ende der Woche ein neues Release machen. Deine Änderungen an der Kanalanzeige können dann in die nächste develop Version einfließen.

Grüße

kamel5