[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