[PATCH 2/3] fff-web: add web interface package

Tobias Klaus tk+ff at meskal.net
Mi Dez 30 10:29:27 CET 2015


Hallo,

ich hab mich  mal durchgeklickt. Ich finde die Weboberfläche schon recht 
gelungen. Schön schlicht und übersichtlich.

Einzelne Anmerkungen sind dann unten.



Am Sonntag, 13. Dezember 2015, 22:16:10 schrieb Tim Niemeyer:
> Signed-off-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> ---
> 
>  src/packages/fff/fff-web/Makefile                  |  42 +
>  src/packages/fff/fff-web/files/etc/config/uhttpd   |  35 +
Soweit ich das verstanden habe sollten wir in Paketen keine /etc/config 
Dateien ausliefern. Wir wissen ja nie, ob wir das letzte oder das erst Paket 
sind. uci-Einstellungen sollten daher wohl durch ein uci-defaults Skript 
getätitgt werden. Das wäre für mich kein Grund diesen Patch nicht 
einzuspielen, aber ich denke langfristig sollten wir das ändern.

>  src/packages/fff/fff-web/files/etc/httpd.conf      |   1 +
>  .../fff/fff-web/files/www/public/cgi-bin/status    |  63 ++
>  .../files/www/public/css/grids-responsive-min.css  | 491 ++++++++++++
>  .../fff/fff-web/files/www/public/css/pure-min.css  | 892
> +++++++++++++++++++++ .../fff/fff-web/files/www/public/css/side-menu.css |
> 276 +++++++
>  .../fff/fff-web/files/www/public/css/style.css     |  78 ++
>  .../fff/fff-web/files/www/public/index.html        |  13 +
>  .../fff/fff-web/files/www/ssl/cgi-bin/home         |  71 ++
>  .../fff/fff-web/files/www/ssl/cgi-bin/misc         |  33 +
>  .../fff/fff-web/files/www/ssl/cgi-bin/password     |  16 +
>  .../fff/fff-web/files/www/ssl/cgi-bin/settings     |  16 +
>  .../fff/fff-web/files/www/ssl/cgi-bin/upgrade      |  36 +
>  src/packages/fff/fff-web/files/www/ssl/home.html   |  54 ++
>  src/packages/fff/fff-web/files/www/ssl/home.js     |  46 ++
>  src/packages/fff/fff-web/files/www/ssl/index.html  | 124 +++
>  .../fff/fff-web/files/www/ssl/password.html        |  28 +
src/packages/fff/fff-web $ grep -a charset files/www/ssl/password.html 
<meta charset="utf-8"/>
src/packages/fff/fff-web $ file files/www/ssl/password.html
files/www/ssl/password.html: HTML document, ISO-8859 text

Das führt dann natürlich zu Darstellungsfehlern. Kann durchaus sein, dass 
kmail da mal wieder was am encoding verkackt hat, aber zumindestens derjenige 
der den Patch dann pusht, sollte darauf achten.

Insgesamt, müssen wir meiner Meinung nach noch einiges - nicht in diesem Patch 
- erledigen:
* "Massenkompatiblen" Zugang auf das webinterface
* Graphische Auswahl der Geokoordinaten


Da ich den Patch hauptsächlich als Arbeitsgrundlage dafür sehe, würde ich ihn 
auch so applien, damit dann viele damit spielen und Erweiterungen schicken 
können.
Deshalb: 
Reviewed-by: Tobias Klaus <tk+ff at meskal.net

Grüße
Tobias

PS: Um auf das Webinterface zu kommen, verwende ich bisher das Portforwarding 
von ssh:
ssh -L 10443:localhost:443 -L 1080:localhost:80 root at ipv6%device

Damit kann man dann unter http://localhost:80 https://localhost:10443 die 
öffentliche und die administrationsseite ansehen.


-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 819 bytes
Beschreibung: This is a digitally signed message part.
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20151230/8c16f6fe/attachment-0002.sig>


Mehr Informationen über die Mailingliste franken-dev