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 menu entries sorted by name #663

Merged
merged 11 commits into from Jul 30, 2019

Conversation

@jp112sdl
Copy link
Contributor

commented Jul 12, 2019

@jens-maus

This comment has been minimized.

Copy link
Owner

commented Jul 12, 2019

Danke. Dein Patch macht aber zuviele newline Änderungen wie es aussieht. bitte die newline Art (Windows vs. Unix) beibehalten und versuchen das die patches so klein wie möglich bleiben weil es sonst zu Konflikten kommt in Zukunft.

@jp112sdl

This comment has been minimized.

Copy link
Contributor Author

commented Jul 12, 2019

Verstehe die Problematik.
Ich versuche es noch mal mit einem anderen Editor.

@jens-maus

This comment has been minimized.

Copy link
Owner

commented Jul 13, 2019

Please sign-off here that these icons are also the other changes are definitly able to be published under Apache-2.0 and also reference the sources you obtained these icons from.

@jp112sdl

This comment has been minimized.

Copy link
Contributor Author

commented Jul 13, 2019

The source reference / license reference is included in /www/config/help.cgi

I will remove the icons from flaticon.com and use all of icons8.de

Do you think https://icons8.de/license is compatible to Apache-2.0 ?

removed flaticon icons
only use icons from icons8.de
@jp112sdl

This comment has been minimized.

Copy link
Contributor Author

commented Jul 13, 2019

I have removed the icons from flaticon.com.
All icons are now from icons8.de

jp112sdl added 3 commits Jul 13, 2019
Merge branch 'master' of https://github.com/jens-maus/RaspberryMatic
* 'master' of https://github.com/jens-maus/RaspberryMatic:
  Update README.md
  Update README.md
  Update README.md
  Update README.md
  lets publish 3.47.10.20190713
  final updates for 3.47.10.20190713 release.
  updated linux kernel for Tinkerboard to 4.19.58 to fix some last minute issues.
jens-maus added 4 commits Jul 30, 2019
moved all submenu png icons out of the patches folder tree to the
overlay because a patch cannot contain binary files.

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

@ptweety

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

Hallo @jp112sdl und @jens-maus,

ich habe mal überlegt, ob man diesen PR auch anders implementieren könnte. Im Prinzip haben wir derzeit in der WebUI ein Menü, welches aus einer Tabelle statisch aufgebaut wird. Um die jeweilige Reihenfolge zu ändern, ist es erforderlich, dass man die jeweilige config ändert (admin.js, user.js).

Alternativ könnte man aber auch die config belassen, wie sie ist und stattdessen Flexibilität ins Menü selber reinbringen. Konkret also die Tabelle des Submenu nur mit div Elementen darstellen und die Sortierung dann später mit der css-eigenschaft order basteln, wie man möchte.

Das könnte dann sogar jeder User individuell für sich selber per bekannter color.map Tricks umgestalten.

Aktuell implementiert:

Bildschirmfoto 2019-09-11 um 22 25 32

Veränderung des Menüs:

Bildschirmfoto 2019-09-11 um 22 41 55

Danach dann die individuelle Sortierung:

Bildschirmfoto 2019-09-11 um 22 47 47

@jp112sdl

This comment has been minimized.

Copy link
Contributor Author

commented Sep 12, 2019

Das könnte dann sogar jeder User individuell für sich selber per bekannter color.map Tricks umgestalten.

Das wäre richtig genial!

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