Filebrowser fix

[seife]

Zwei Kleinigkeiten:
1. - wenn man die Sortierreihenfolge ändert (mit "?"), dann wird das unten bei der Tastenbeschreibung nicht angezeigt.
2. - Bei der Beschreibung der Taste "?" wird immer die nächste Sortiermethode angezeigt (die, die aktiv wird, wenn man "?" drückt). Das mag zwar "korrekt" sein, aber es ist unintuitiv (zumindest für meine Frau und mich ist es interessanter zu sehen, wie jetzt gerade sortiert wird).

Code: Alles auswählen

Index: apps/tuxbox/neutrino/src/gui/filebrowser.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/filebrowser.cpp,v
retrieving revision 1.114
diff -u -p -r1.114 filebrowser.cpp
--- a/apps/tuxbox/neutrino/src/gui/filebrowser.cpp      24 May 2007 19:12:33 -0000      1.114
+++ b/apps/tuxbox/neutrino/src/gui/filebrowser.cpp      30 Sep 2007 09:03:25 -0000
@@ -839,7 +839,7 @@ bool CFileBrowser::exec(const char * con
                }
                else if (msg==CRCInput::RC_help)
                {
-                       if (g_settings.filebrowser_sortmethod >= 4)
+                       if (g_settings.filebrowser_sortmethod >= FILEBROWSER_NUMBER_OF_SORT_VARIANTS)
                                g_settings.filebrowser_sortmethod = 0;
                        else
                                g_settings.filebrowser_sortmethod++;
@@ -847,6 +847,7 @@ bool CFileBrowser::exec(const char * con
                        sort(filelist.begin(), filelist.end(), sortBy[g_settings.filebrowser_sortmethod]);

                        paint();
+                       paintFoot();
                }
                else if (CRCInput::isNumeric(msg))
                {
@@ -1149,7 +1150,7 @@ void CFileBrowser::paintFoot()

                //?-Button
                frameBuffer->paintIcon(NEUTRINO_ICON_BUTTON_HELP, x + (1 * dx), by2 - 3);
-               g_Font[SNeutrinoSettings::FONT_TYPE_INFOBAR_SMALL]->RenderString(x + 35 + (1 * dx), ty2, dx - 35, g_Locale->getText(sortByNames[(g_settings.filebrowser_sortmethod + 1) % FILEBROWSER_NUMBER_OF_SORT_VARIANTS]), COL_INFOBAR, 0, true); // UTF-8
+               g_Font[SNeutrinoSettings::FONT_TYPE_INFOBAR_SMALL]->RenderString(x + 35 + (1 * dx), ty2, dx - 35, g_Locale->getText(sortByNames[g_settings.filebrowser_sortmethod]), COL_INFOBAR, 0, true); // UTF-8

                //Mute-Button
                if (strncmp(Path.c_str(), VLC_URI, strlen(VLC_URI)) != 0) { //Not in vlc mode

[Steffi]

Danke seife, das hat mich schon lange gestört.

[PauleFoul]

Ein Danke auch von mir!

Ich hoffe nur das sich jemand findet der Deine ganzen Patche ins CVS eincheckt...

[seife]

Vielleicht hole ich mir doch mal einen CVS-account ;-)

[PauleFoul]

Vielleicht hole ich mir doch mal einen CVS-account ;-)

Jo, ich denke das wäre echt besser!!

[Steffi]

@seife:
Es gibt mittlerweile nicht mehr allzu viele Devs, die noch Interesse an der Weiterentwicklung von Neutrino haben. Mir fallen auf Anhieb nur noch Günther, dbt und Houdini ein. Deshalb wäre es wirklich schön, wenn du auch einen Account bekämst, denn deine diversen Patches sind durch die Bank sinnvoll und hilfreich. Ich versuche diese Patches umgehend bei mir anzuwenden, aber allmählich verliert man dabei den Überblick.

Steffi

[seife]

..., denn deine diversen Patches sind durch die Bank sinnvoll und hilfreich.

Bisher sind sie das. Aber wer weiß, was mir erst einfällt, wenn ich DIE MACHT habe... ;-)

[eule]

Bezüglich der Macht wirst du sicher schon diese Nachricht erblickt haben. Also Mail an derget und Sie wird dir gegeben.

wer einen cvs Account möchte
weil er denkt das er sich im stande fühlt dem Project
wertvollen Code beizusteuern
der sende mir dafür bitte eine Mail ...
da ich das Forum nur noch selten besuche und da ja sonst auch nicht wirklich viele Admins zu erreichen sind ...

derget

wer es nicht weiss : mein nick at noernet.de

[Steffi]

Zurück zum Topic.

Irgend etwas stimmt noch nicht.

5x hintereinander "?" --> Segmentation fault und Neutrino startet neu

[seife]

Ups. Moment, ich muß mal ein image für die dream bauen (meine Frau belegt grad die dbox mit Fußball :-)

[seife]

Ähm. Ja. Peinlich.
Ersetze mal

Code: Alles auswählen

    if (g_settings.filebrowser_sortmethod >= FILEBROWSER_NUMBER_OF_SORT_VARIANTS)
        g_settings.filebrowser_sortmethod = 0; 
    else
        g_settings.filebrowser_sortmethod++;

durch

Code: Alles auswählen

    if (++g_settings.filebrowser_sortmethod >= FILEBROWSER_NUMBER_OF_SORT_VARIANTS)
        g_settings.filebrowser_sortmethod = 0;

(den else-Zweig weglassen).

Das passiert, wenn man erst kompiliert und probiert, dann nochmal "verschönert", kompiliert aber nicht mehr probiert ;-)

[Steffi]

Jetzt klappt es.

Übrigens: Deutschland ist Weltmeister!

[seife]

super, dann kann ich endlich ein neues Image flashen ;-))

[doktorknow]

Moin moin,
könntest Du bei der Gelegenheit auch mal auf den "Filter an" "Filter aus" im Filebrowser schauen, der normalerweise mit der blauen Taste gesetzt wird?
Im Filebrowser ändern sich zwar die angezeigten Files, es bleibt im unteren Bereich aber immer "Filter aus" eingeblendet.

MfG
doktorknow

[seife]

Ja, das ist einfach :-)

Code: Alles auswählen

diff -u -p -r1.114 filebrowser.cpp
--- a/neutrino/src/gui/filebrowser.cpp  24 May 2007 19:12:33 -0000      1.114
+++ b/neutrino/src/gui/filebrowser.cpp  30 Sep 2007 15:24:37 -0000
@@ -761,6 +761,7 @@ bool CFileBrowser::exec(const char * con
                        if(Filter != NULL)
                        {
                                use_filter = !use_filter;
+                               paintFoot();
                                ChangeDir(Path);
                        }
                }

Edit: oops, zuwenig Kontext. Das ist im "if ( msg == CRCInput::RC_blue )"-Zweig.

[doktorknow]

Man, Du räumst echt auf!!! RESPEKT! und danke!
MfG
doktorknow