socia-platform / htwplus

The social network and new eLearning system for HTW-Berlin.
http://plus.htw-berlin.de
MIT License
14 stars 8 forks source link

Newsstream-Inhalte filtern #111

Closed fabiankirstein closed 9 years ago

fabiankirstein commented 9 years ago

Filtern der Posts auf dem Newsstream nach:

IvenZ commented 9 years ago

Ich habe mir überlegt das in einem Dropdown mit Multiselect zu machen. http://davidstutz.github.io/bootstrap-multiselect/

Wenn jemand eine schönere Idee hat, dann her damit. Am Wochenende fange ich an :)

fabiankirstein commented 9 years ago

Ganz spontan bin ich gegen Dropdown. Das ist ja ein zusätzlicher Klick. Nervt mich auf anderen Seiten auch oft. Aber würde mich jetzt nicht dagegen wehren.

fabiankirstein commented 9 years ago

Wie wäre es mit horizontal sortierten Buttons, die man einzeln aktivieren kann und damit den Filter einschaltet.

IvenZ commented 9 years ago

Hatte ich zuerst auch im Sinn. Allerdings wird es meiner Meinung nach problematisch, wenn wir doch mal mehr als 4 Filter haben. Auch die Beschreibung der Filter wollte ich nicht auf ein Wort beschränken. Bei einer horizontalen Anordnung kann es also passieren, dass eine gesamte Zeile mit Filter beschrieben ist.

IvenZ commented 9 years ago

@FabiApfelkern Hast du dir das eher so wie bei Google+ vorgestellt?

image

Dann könnte man natürlich die Dropdown weglassen. Ich bin nämlich eher von einem Multiselect ausgegangen.

IvenZ commented 9 years ago

So, ich habe gemerkt, dass ich mit dem aktuellen Stand nicht viel Spielraum für Filter habe. Also habe ich noch mal die Post-Logik aufgeräumt. Die Query setzt sich nun aus atomaren Abfragen zusammen. Das ermöglicht eine präzise Filterung und ist offen für Ergänzungen. Damit wurden auch ein paar Bugs behoben (z.B war es früher möglich, dass man immer noch Posts in einer Gruppe angezeigt bekommt, obwohl man sich nicht mehr in dieser Gruppe befindet. Auf der einen Seite kann man zwar sagen, "klar, ist doch der eigene Post und den darf ich immer sehen". Ich finde allerdings, dass bei einem Post in einer Gruppe, man sein "Post-Recht" der Gruppe überträgt. Und wenn man nicht mehr Mitglied ist, darf man diesen Post eben nicht mehr sehen).

Zusätzlich habe ich Variablennamen und Kommentare schon leicht in Richtung "Kontakte" geändert. Wir wollen ja irgendwann von "Freunden" weg.

Im nächsten Schritt werden dann die Filter bestimmt und implementiert.

IvenZ commented 9 years ago

Ok, der aktuelle Stand ist aufm Develop und kann getestet werden. Ich werde selber noch ausgiebig testen. Wenn bis nächste Woche keine Anmerkungen sind, werde ich es deployen :-)

Anmerkung: Der Favoriten-Filter ist bisher noch nicht so implementiert wie wir ihn haben wollen. Das mache ich dann nach dem Live-Deploy in Ticket #85.

tennox commented 9 years ago

send

-> Design-Vorschlag (Mockup): send

EDIT: Bei Favoriten steht im 'title' "Posts, die du auf deinem Stream verfasst hast". Ist das ein Fehler, oder verstehe ich das nur nicht?

IvenZ commented 9 years ago

Danke fürs anschauen @TeNNoX Ich habe gerade noch mal eine aktuelle Version aufm Dev deployed. Da sind ein Paar der von dir angesprochenen Design-Sachen schon realisiert.

Ich werde morgen noch ein bisschen rumspielen. Allerdings auch aus bekannten Gründen nicht so viel Zeit ins Styling stecken.

Der Favoriten-Filter zeigt dir im Moment alle die von dir geschrieben Posts auf deinem Stream an. Das ist so eine Art "Eigene Posts"-Light. Bei dem werden ja auch Posts aus Gruppen und co. angezeigt.

Der Favoriten-Filter ist im Moment auch noch eine Art Platzhalter für die wirklichen Favoriten. In einem nächsten Schritt soll es möglich sein, Posts zu liken/favorisieren. Diese werden dann unter diesem Filter angezeigt.

Nun verstanden?

tennox commented 9 years ago

Aaaaalles klar :) :+1: Naja, ich finde es verständlich, dass wir erstmal keinen Aufwand in ein redesign stecken wollen, aber für mich sind z.B. solche Sachen wie unsymmetrisches Padding ein Dorn im Auge... So Kleinigkeiten kann man doch machen :P

fabiankirstein commented 9 years ago

Danke Iven! Sieht erstmal gut aus, Kannst du "Die Query setzt sich nun aus atomaren Abfragen zusammen" genauer erklären? Sachen die anders sein sollten:

Sachen die nett wären

IvenZ commented 9 years ago

So, ich habe eure Vorschläge mal aufgegriffen und noch mal geändert. Ich persönlich finde die Button-Gruppe etwas zu "fett". Auch die kleinste Größe wirkt für mich etwas aufdringlich. Die in der Luft hängenden Links finde ich zwar auch nicht optimal, wären aber in der jetzigen Version mein Favorit.

@FabiApfelkern Für die Generierung des Newsstreams sind mehrere WHERE-Clauses in der Abfrage nötig. Jede WHERE-Clause ist vorher definiert und erfüllt genau eine Abfrageeigenschaft. Je nach Filter werden verschiedene WHERE-Clauses zusammengesetzt und bilden nachher das fertige Query.

Die Umsetzung ist im Moment noch nicht so hübsch und muss noch mal überarbeitet werden. Das hat allerdings noch etwas Zeit bis wir im nächsten Schritt irgendwann einen Newsstream zum selber zusammenklicken anbieten.

Da muss ich aber noch mal bissel länger recherchieren wie man sowas macht. Ich denke das macht dann vielleicht auch erst/eher Sinn, wenn wir mit dem Frontend noch mal Richtung SPA umsteigen.

tennox commented 9 years ago

Ich finde die Version 1 deutlich besser... Wie wäre es mit einem farbigen 'Trauerbalken' hier: ? https://cloud.githubusercontent.com/assets/2084639/7688459/d43f0252-fda4-11e4-97c9-5cb10e302cd3.png

Oder wie wäre es mit sowas: unbenannt

fabiankirstein commented 9 years ago

@IvenZ Kannst du das mit den Dropdown mal ausprobieren. Glaube mittlerweile das wäre am Besten.

IvenZ commented 9 years ago

ok, ist deployed. wie stellt ihr euch das mit der Anzeige des aktuellen Filters vor? Ich habe es für den Moment mal so gemacht, dass die Buttonbeschriftung mit dem aktuellen Filter ergänzt wird.

fabiankirstein commented 9 years ago

Ich finde es ganz Klasse so!!

IvenZ commented 9 years ago

So, habe den anderen Filter rausgenommen und den finalen(?) Button eingebaut :) Habe ihn noch mal eine Stufe kleiner gemacht.

ryad-eldajani commented 9 years ago

Ist echt nice geworden.

IvenZ commented 9 years ago

auf live deployed