neradis / index-vivus

student project to present lexical knowledge from XML-transcriptions of dictionaries in an interactive web interface for easier lookup and explorative search
0 stars 0 forks source link

ansprechendere optische Gestaltung #28

Open neradis opened 10 years ago

neradis commented 10 years ago

Das Layout und Styling können und sollten wir durch den Einsatz eines CSS-Frameworks mit überschaubaren Aufwand deutlich verbessern. Ein paar der üblichen Verdächtigen:

@andywer, @chillSen Wenn ihr für euch Einschätzt, dass ihr mehr Zeit spart, das Aussehen ohne Framework zu verbessern, ist das natürlich ebenso akzeptabel. Es sollte
aber dann trotzden eine angenehme, ästhetische Farbkombination, sowie ein ansprechender Satzspiegel und typografische Rythmus vorliegen. Außerdem sollten zur einfachen optischen Orientierung für den Nutzer Seitenelemente stimmig in Boxen gruppiert sein (gern solche mit runden HTML5-Ecken).

@andywer Das ist ein issue, der euch beide betritt, aber ich konnte nur einen von euch direkt als asignee anwählen (hier Simon).

neradis commented 10 years ago

Beim Stand gemäß c9db27990bdb58a0fe8e38a62fb6a5fa85de730f lösen die Callbacks des Volltextfeldes offenbar nicht mehr die korrekten Abfragen aus (GET "/?inputFulltextSearch=der" statt an die korrekte Route für den Volltextsuchservice).

Außerdem ist da kein Schalter mehr für das Umschalten zwischen den Sprachen zu sehen und der Default-Wert für die Sprache ist null, wodurch keine validen Anfragen an den AjaxController enstehen.

neradis commented 10 years ago

Bei c9db27990bdb58a0fe8e38a62fb6a5fa85de730f überlappen auch die Bereiche von Volltextsuche und Stichwortsuche bei einem Viewport kleiner als ca. 1200 px. (Chromium 25, Firefox 20)

andywer commented 10 years ago

@neradis Also ich weiß nicht was bei dir das Problem ist, aber bei mir funktioniert das alles (Stand: https://github.com/neradis/index-vivus/commit/c9db27990bdb58a0fe8e38a62fb6a5fa85de730f).

Zugegeben, die Überschriften der Panels sehen noch sch*\ aus und für die Ergebnisanzeige müssen wir noch einen Throbber anzeigen, damit man sieht dass er was macht bis die Ergebnisse kommen. Aber die Volltextsuche klappt und bei mir überlappt da auch bei kleinen Viewports nichts. Hast du mal den Browsercache geleert?

neradis commented 10 years ago

@andywer Die gestern beschriebenen Probleme haben sich erledigt. Am Browserchache lag es nicht, aber nah daran: Es wurden teilweise veraltete statisch vor kompilierte assets ausgeliefert.

Um zu vermeiden, dass vielelicht noch andere darauf hereinfallen: Kannst du bitte einen kurzen erläuternden Hinweis zur Wikiseite zum Einrichten der Umgebung hinzufügen?

neradis commented 10 years ago

Ansonsten sieht das alles schon sehr viel schöner aus. Nur der Umschalter zwischen den Sprachen ist so wie er sich derzeit verhält und dargestellt wird noch mMn etwas unintutiv/verwirrend.

andywer commented 10 years ago

@neradis Ja, der Schalter muss noch fix gemacht werden. Aber das ist schnell erledigt.

Was genau soll ich denn dazu dokumentieren? Dass man rake assets:precompile ausführen muss wenn man in der production-Umgebung arbeitet?

neradis commented 10 years ago

@andywer

Was genau soll ich denn dazu dokumentieren? Dass man rake assets:precompile ausführen muss wenn man in der production-Umgebung arbeitet?

Zumindest bei der derzeitigen Konfiguration in railsapp/config/environments kann man (wie mir passiert) auch ebenso unter test da auf Probleme (bzw. zumindest unklares Verhalten) stoßen. In meinem Fall hatte ich bereits bei einem früheren commit mal die assets precompiliert und auch unter test hatte rails dann munter weiter veraltetes CSS und JavaScript ausgeliefert, obwohl neuere Quelldateien vorhanden waren.

Vielleicht können wir auch die environment-Konfigurationen für development und test ohne großen Aufwand so anpassen, dass assets pro Serverstert mind. einmal on the fly neu generiert werden? (Für production ist Abhängigkeit von rake assets:precompile natürlich völlig okay.)

andywer commented 10 years ago

@neradis Aber das ist doch quatsch... Für development und test müssen die Assets doch gar nicht vorcompiliert werden. Falls man es aus Versehen doch getan hat, schafft ein rake assets:clean Abhilfe. Das könnte man vllt dokumentieren. ;)

andywer commented 10 years ago

@neradis Der Sprachschalter funktioniert jetzt ein bisschen anders, ich hoffe das ist jetzt so ok. @chillSen Was macht das Template?

neradis commented 10 years ago

@andywer Ja, mMn kann der Schalter so bleiben.