Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

show favorite list on startpage dropdown menu sorted by name #665

Conversation

@jp112sdl
Copy link
Contributor

commented Jul 14, 2019

On the start page
Bildschirmfoto 2019-07-14 um 13 08 04
, the combobox with favorites is currently listed how they where added or modified.
And if you modify a favorite afterwards, the list is re-ordered.
Bildschirmfoto 2019-07-14 um 12 44 44

With this patch, the list is always ordered by name.
Bildschirmfoto 2019-07-14 um 12 45 37

@jp112sdl

This comment has been minimized.

Copy link
Contributor Author

commented Jul 14, 2019

Consider: When the webui is loaded, the favorite of the first position is shown!

The user can now influence directly, which favorite is shown by default. i.e. first letter is A or a 0; so that it is on the top position after sorting

{
Write( '<option>${noFav}</option>' );
}
+ oFavoriteList.SortByName();

This comment has been minimized.

Copy link
@jens-maus

jens-maus Jul 14, 2019

Owner

Anstatt das SortBaName() bei jedem Aufruf von startpage.htm aufzurufen sollte es IMHO effektiver/ressourcenschonender sein das SortByName besser in den Routinen aufzurufen nach de Editieren des Namens bzw. anlegen eines neuen Favoriten.

This comment has been minimized.

Copy link
@jp112sdl

jp112sdl Jul 14, 2019

Author Contributor

Da sehe ich leider nicht so recht durch.
Da werden die IDs umgeshiftet; der zuletzt editierte wandert nach ganz unten, nachdem vorher auch eine Kopie angelegt wurde.
Müsste wohl irgendwo hier passieren: /www/rega/esp/favorites.fn#L233
Ich schaue es mir die Tage noch mal an 👍

This comment has been minimized.

Copy link
@jp112sdl

jp112sdl Jul 14, 2019

Author Contributor

Fazit nach einer Stunde intensiver Beschäftigung mit der SaveFavoriteList Methode:
Ich blick da absolut nicht durch 😣
Es mangelt schon am Verständnis undokumentierten verwendeten internen Methoden:

      integer iOrigID = oCopy.EnCopyID();
      object oOrig = dom.GetObject( iOrigID );
      if( oOrig )
      {
        oCopy.EnumCopyTo( oOrig.ID() );
        dom.DeleteObject( oCopy.ID() );
        oOrig.EnCopyID( ID_ERROR );
      }

Aber auch mit viel Debug-Output bekomme ich es nicht auf die Reihe, an irgendeiner Stelle die Sortierung einzubauen. 😞

This comment has been minimized.

Copy link
@jens-maus

jens-maus Jul 30, 2019

Owner

Also ich hab vorerst auch kapituliert. An den Stellen werden anscheinend Objekte hin/her-kopiert um irgendwie eine neue Favoritenliste zu erzeugen falls Objekte entfernt werden. Da schein deine ursprüngliche Änderung mit einfach SortByName() doch die effektivere zu sein.

renamed 0061-WebUI-Fix-SortFavoritesComboboxOnStartpageByName to
0062-WebUI-Fix-SortFavoritesCombobox

@jens-maus jens-maus merged commit f8e13af into jens-maus:master Jul 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.