[Patch] Dynamische Einpassung und Anordnung der Icons

[GetAway]

Der Blaue war doch heller.

new_icons_col.rar

[dbt]

...sonst geht ein Stück zu viel klassisches Neutrino verloren.

Naja...[sacken lass]... ich hab auch eigene im Image, siehe hier weil die alten zum einen nun wirklich schon angestaubt sind und so piepsig wie die sind, auch schon mal zum "suchen" animieren, was nun wirklich nichts neues mehr ist, auch wenn die Diskussion schon mal mit den Kugeln war, sag ich das immer noch. Was die ewiggestrigen angeht, kann man ja die alten wie gehabt einspielen... nur um den Spieß mal umzudrehen.

Edit:
Andererseits ist bei den Farbbuttons an einigen Stellen noch hardcodedes Zeug drin, was nicht förderlich ist, sage nur mal Infoviewer, GetAways Icons haben optisch nicht die gleiche Größe wie die Numbuttons, was so noch nicht ganz optimal ist.

Edit: probiert mal diese: colorbuttons.zip

[GetAway]

Die Farbbuttons waren eigentlich immer etwas kleiner als die NumButtons.
Ist halt Geschmackssache. So groß wirken sie sehr klotzig. Der Blaue müsste
als einziger das hellere Blau bekommen. Hatte mich da auch vertan.

Edit:
Eventlist rework.

Edit: Patch entfernt, ist im CVS

[mb405]

@dbt
deine colorbuttons sind perfekt. größe und farbe passen

[mb405]

kann man das nicht auch noch ändern ?

Code: Alles auswählen

--- eventlist_old.cpp	2010-07-03 10:19:32.000000000 +0200
+++ eventlist.cpp	2010-07-03 10:24:25.000000000 +0200
@@ -1044,26 +1044,6 @@
 			parent->hide();
 		showMenu();
 	}
-	else if(actionkey =="2")
-	{
-		/*
-		if(*m_search_list == EventList::SEARCH_LIST_CHANNEL)
-		{
-			mf[1]->setActive(true);
-			m_search_channelname = g_Zapit->getChannelName(*m_search_channel_id);;
-		}
-		else if(*m_search_list == EventList::SEARCH_LIST_BOUQUET)
-		{
-			mf[1]->setActive(true);
-			m_search_channelname = bouquetList->Bouquets[*m_search_bouquet_id]->channelList->getName();
-		}
-		else if(*m_search_list == EventList::SEARCH_LIST_ALL)
-		{
-			mf[1]->setActive(false);
-			m_search_channelname = "";
-		}
-		*/
-	}	
 	else if(actionkey =="3")
 	{
 		// get channel id / bouquet id
@@ -1085,8 +1065,7 @@
 		}
 		else if(*m_search_list == EventList::SEARCH_LIST_BOUQUET)
 		{
-			int nNewBouquet;
-			nNewBouquet = bouquetList->show();
+			int nNewBouquet = bouquetList->show();
 			//printf("new_bouquet_id %d\n",nNewBouquet);
 			if (nNewBouquet > -1)
 			{

ist ja unbenutzter code und das nNewBouquet is ja schon mal im letzten checkin geändert worden

[dbt]

Habe grade gesehen, dass die Positionen der Icons hier auch noch nicht passen. Dank GetAways kleineren Numbuttons sieht man das sehr schön.

[dbt]

kann man das nicht auch noch ändern ?


ist ja unbenutzter code und das nNewBouquet is ja schon mal im letzten checkin geändert worden

Der wird sicher zu etwas gut gewesen sein, was aber scheinbar tatsächlich nicht gebraucht wird, aber den Grund dafür, warum das dann drin ist wäre schon interessant zu wissen Könnte aber sein, dass das woanders noch abgerufen wird....
Edit: erledigt

[GetAway]

Habe grade gesehen, dass die Positionen der Icons hier auch noch nicht passen. Dank GetAways kleineren Numbuttons sieht man das sehr schön.

Edit: Patch entfernt, ist im CVS.

[dbt]

erledigt...
was immernoch etwas stört, sind die festen Werte ICON_HEIGHT, ICON_LARGE_WIDTH, so auch im infoviewer. Das müsste man evtl. auch mal anfassen.

[GetAway]

Beim Infoviewer war eine größere Umstrukturierung notwendig, damit flog auch jede Menge
doppelter Code raus. Zusätzlich war eine kleine Änderung in den Movieplayern notwendig.
Also, schön testen.

Patch entfernt.

PS:

was immernoch etwas stört, sind die festen Werte ICON_HEIGHT, ICON_LARGE_WIDTH, so auch im infoviewer. Das müsste man evtl. auch mal anfassen.

Die InfoIcons werden dann Teil 2

[Gaucho316]

... Zusätzlich war eine kleine Änderung in den Movieplayern notwendig. ...

Ich habe mir mal deine Änderungen für die beiden Movieplayer angesehen. Die Parameter "text_button_green" und "text_button_red" der Funktion "showMovieTitle" darfst du nicht so mir nichts dir nichts rausschmeißen. Die haben ihre Berechtigung. Die Belegung der Rot- und Grün-Taste ist jeweils unterschiedlich im TS-Teil des MP1, im VLC-Teil des MP1 und im MP2. Außerdem wird "g_InfoViewer->showMovieTitle(...)" im MP1 an zwei Stellen aufgerufen. Du hast aber nur eine geändert.

[GetAway]

Ja, die 2. Stelle habe ich übersehen. Habe nur mit mp2 getestet. Ich hatte mich schon gewundert wieso da
eine locale übergeben wird, die doch von überall abrufbar ist. Da werde ich mir noch mal etwas überlegen.
2. Status oder Modusübergabe wäre einfacher.

Nutzt den MP1 denn noch jemand ??

[dwilx]

Nutzt den MP1 denn noch jemand ??

Im JTG ist er nach wie vor drin, soviel ich weis.

[Gaucho316]

Nutzt den MP1 denn noch jemand ??

Ja, ich. Und im CVS ist er ja auch Standard. Vielleicht sollte ich mir den MP2 aber wirklich mal einbauen. Merkt sich denn der MP2 im Zusammenspiel mit dem Moviebrowser die letzte Abspielposition und kann dann von dort wieder starten? Ich meine, dass das nicht geht. Das ist nämlich für mich das KO-Kriterium.

2. Status oder Modusübergabe wäre einfacher.

Da hast du Recht. Vielleicht ist es am besten, per ifdef MP1 und MP2 auseinanderzuhalten und den Modus des MP1 als Parameter zu übergeben.

[GetAway]

Hier noch die msgbox, nicht zu verwechseln mit der messagebox.
buttons_msgbox.diff

@Gaucho316
Bookmarks im MP2 muss man immer noch explizit setzen, auch in Verbindung mit dem MB.

[mb405]

jo der mp2 fasziniert mich auch garnicht. ich nutze den mp1 und bin voll zufrieden damit.
solange man die beiden nicht gleichzeitig nutzen kann, nutz ich weiter den mp1

[GetAway]

... Zusätzlich war eine kleine Änderung in den Movieplayern notwendig. ...

Ich habe mir mal deine Änderungen für die beiden Movieplayer angesehen. Die Parameter "text_button_green" und "text_button_red" der Funktion "showMovieTitle" darfst du nicht so mir nichts dir nichts rausschmeißen. Die haben ihre Berechtigung. Die Belegung der Rot- und Grün-Taste ist jeweils unterschiedlich im TS-Teil des MP1, im VLC-Teil des MP1 und im MP2. Außerdem wird "g_InfoViewer->showMovieTitle(...)" im MP1 an zwei Stellen aufgerufen. Du hast aber nur eine geändert.

Patch aktualisiert: infoviewer_iconbuttons_rework_v2.diff

Getestet mit MP1 und 2.

[dwilx]

solange man die beiden nicht gleichzeitig nutzen kann, nutz ich weiter den mp1

Das wird wohl auch nie passieren, nachdem sich Seife solche Mühen gemacht, den MP2 von den MP1-Relikten zu befreien...

[Gaucho316]

Ich habe auch mal über die neue Variante drüber geguckt. So ganz passt dein Patch immer noch nicht. Folgendes wird als Buttontext gebraucht:

MP1 TS:

• Rot: LOCALE_TIMERLIST_PLUGIN
• Grün: LOCALE_INFOVIEWER_LANGUAGES

MP1 VLC:

• Rot: LOCALE_AUDIOPLAYER_STOP
• Grün: LOCALE_MOVIEPLAYER_VLCHELP2

MP2:

• Rot: LOCALE_MOVIEPLAYER_TSHELP17
• Grün: LOCALE_INFOVIEWER_LANGUAGES

Was in deinem Code noch fehlt, ist LOCALE_TIMERLIST_PLUGIN für die rote Taste im MP1-TS-Modus. Deshalb ja mein Vorschlag mit ifdef.

Außerdem könntest du "enum { FILE_MODE, VLC_MODE };" mit in "infoviewer.h" aufnehmen. Dann musst du im Infoviewer nicht plötzlich mit Zahlen arbeiten und der Code wird meiner Meinung nach an der Stelle übersichtlicher. So hab ich es ja auch mit "enum { NO_AC3, AC3_AVAILABLE, AC3_ACTIVE };" gemacht.

Mehr kann ich jetzt erst einmal nicht dazu sagen, ohne es tatsächlich getestet zu haben.

[PauleFoul]

@GetAway

Mensch das ist ja wirklich spitze was Du hier optimierst

Da muss ich gleich mal ein aktuelles Image auf die Boxen flashen...


Kann man Deine Optimierungen eigentlich auch ins SVN einfließen lassen??



Gruß
____Paule

[dbt]

Hier noch die msgbox, nicht zu verwechseln mit der messagebox.
buttons_msgbox.diff

erledigt

Auch die neuen Coloricons sind jetzt drin, die machen sich in Verbindung mit den neuen Numbuttons auch recht gut und das Neutrino look&feel leidet auch nicht drunter und wie gesagt, wenn jemand die Alten haben will, kann er die ja leicht customized einbauen oder nachträglich installieren siehe: Neutrino:Customizing#eigene_Icons.

Patch aktualisiert: infoviewer_iconbuttons_rework_v2.diff

Getestet mit MP1 und 2.

Das nenn' ich mal schön aufgeräumt Da sind aber mehr Tests erforderlich...aber sieht schon mal gut aus

[dbt]

@GetAway

Mensch das ist ja wirklich spitze was Du hier optimierst

Da muss ich gleich mal ein aktuelles Image auf die Boxen flashen...


Kann man Deine Optimierungen eigentlich auch ins SVN einfließen lassen??



Gruß
____Paule

Nicht ganz so, aber vom Prinzip her schon...

[GetAway]

Was in deinem Code noch fehlt, ist LOCALE_TIMERLIST_PLUGIN für die rote Taste im MP1-TS-Modus. Deshalb ja mein Vorschlag mit ifdef.

Außerdem könntest du "enum { FILE_MODE, VLC_MODE };" mit in "infoviewer.h" aufnehmen. Dann musst du im Infoviewer nicht plötzlich mit Zahlen arbeiten und der Code wird meiner Meinung nach an der Stelle übersichtlicher. So hab ich es ja auch mit "enum { NO_AC3, AC3_AVAILABLE, AC3_ACTIVE };" gemacht.

Mehr kann ich jetzt erst einmal nicht dazu sagen, ohne es tatsächlich getestet zu haben.

Das war Absicht und gehört mit zu der Aufräumaktion. Warum sollte man 2 fast gleiche Captions für die
gleiche Aktion (Plugin <=> starte Plugin), die auf dem Button liegt, verwenden? Die 2. Variante erschien
mir logischer, deswegen habe ich die für beide Player genommen und die andere weggelassen.
Die Aufzählungskonstante habe ich mal in die infoviewer.h mit übernommen. Von weiteren ifdef's
halte ich nicht so viel.

infoviewer_iconbuttons_rework_v3.diff

[Gaucho316]

Ach Mist, ich hatte noch im Hinterkopf, dass man im MP2 keine Plugins starten kann. Dass das geändert wurde, ist an mir vorbei gegangen. Dann braucht man natürlich kein ifdef. Ich hätte mir LOCALE_MOVIEPLAYER_TSHELP17 vielleicht doch vor meinem Posting angucken sollen, bevor ich so klug daher poste. Tut mir echt Leid.

Warum hast du jetzt eigentlich FILE_MODE aus dem enum rausgeschmissen? Das hättest du doch gut in "infoviewer.h" für "showMovieTitle(..., const int playmode = FILE_MODE);" und "void showButton(..., const int mode = FILE_MODE) const;" nutzen können.

Aber ansonsten sag ich auch mal für die viele Arbeit, die du dir machst.

[seife]

Welches Plugin ist im Movieplayer denn Benutzbar / sinnvoll? Ich kann das irgendwie nicht erkennen