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

UI style tweaks #479

Merged
merged 2 commits into from Sep 20, 2016
Merged

UI style tweaks #479

merged 2 commits into from Sep 20, 2016

Conversation

andig
Copy link
Contributor

@andig andig commented Sep 20, 2016

  • better default plot size
  • button bar rightmost button alignment fixed
  • removed "Jetzt" to avoid undesired breaking
  • smaller buttons in main view, aligned with left margin
  • less padding
  • plot overlay vertically centered

@andig andig merged commit 6505c1a into volkszaehler:master Sep 20, 2016
@andig andig deleted the uistyle branch September 20, 2016 16:33
@J-A-U
Copy link
Collaborator

J-A-U commented Sep 27, 2016

•better default plot size

Hm, der Graph wirkt doch arg gedrungen.

Ich hab den Eindruck man kann weniger erkennen, bin mir aber unschlüssig ob es nur noch etwas ungewohnt ist oder vorher wirklich besser war.

@andig
Copy link
Contributor Author

andig commented Sep 27, 2016

Hast Du einen Screenshot?

@J-A-U
Copy link
Collaborator

J-A-U commented Sep 27, 2016

Der Graph hatte nur ~2/3 der gewohnten Höhe.
vz_graph_zu_schmal
Ich hab gerade nochmal (!) den Browsercache geleert und es sieht wieder wie gewohnt aus.

@J-A-U
Copy link
Collaborator

J-A-U commented Sep 29, 2016

Der Fehler ist wieder da. Sowohl bei Opera 40.0 als auch IE11.
Kein Browsercache geleert oder Änderungen am VZ gemacht.

@andig
Copy link
Contributor Author

andig commented Sep 29, 2016

Der Fehler ist wieder da.

Das liegt daran dass wir versuchen allen Content- Plot und Tabelle- auf die Seite zu bekommen. Dafür wird das Chart bis Minimum - 300 Pixel - verkleinert.

Du kannst in der options.js den Minimalwert vz.options.plot.minHeight anpassen. Zusätzlich könnten wir darüber diskutieren ob der Minimalwert von 300 zu klein ist?

Ein Fehler ist es aber nicht ;)

@J-A-U
Copy link
Collaborator

J-A-U commented Sep 29, 2016

Its not a bug, its a feature. :D
Komisch war halt das mal so und mal so dargestellt wurde...

Bei mir ist mit der PV und der Abgastemperatur die Spreizung recht groß. In der Tagesansicht ist die Grundlast dann fast nicht mehr zu erkennen. Von daher wäre ein größerer Chart schon nicht verkehrt.

Ich find aber den Parameter nicht. Du meintest doch
https://github.com/volkszaehler/volkszaehler.org/blob/master/htdocs/frontend/javascripts/options.js ?

@andig
Copy link
Contributor Author

andig commented Sep 29, 2016

Ich find aber den Parameter nicht. Du meintest doch

Genau. Ist noch nciht drin, kannst Du Dir aber ab der entsprechenden Stelle einfach eintragen.

Bei mir ist mit der PV und der Abgastemperatur die Spreizung recht groß.

Wenns hilft leg Dir die beiden doch einfach auf unterschiedliche Achsen? Zur Not 1x Temp links und 1x Temp rechts um sie auseinander zu halten ;)

@TylonHH
Copy link

TylonHH commented Oct 3, 2016

Also ich würde es besser finden wenn nur der Graph die maximale Bildschirmhöhe einnimmt und zur Tabelle runter gescrollt werden kann.
Warum? Weil ich mir das Frontend hauptsächlich auf'm Handy (landscape) anschaue und vorrangig Tendenzen und Ausschläge sehen möchte. Auf die Tabelle schaue ich ja nur einmal, wegen den Werten.

@andig
Copy link
Contributor Author

andig commented Oct 3, 2016

Bitte Diskussionen auf ML an einer Stelle führen sonst wirds unübersichtlich.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants