Plots2: Implementieren eines neuen Designs fĂŒr die Tags-Seite (muss aufgebrochen werden)

Erstellt am 15. MĂ€rz 2019  Â·  46Kommentare  Â·  Quelle: publiclab/plots2

Aufbauend auf #4922 sollten wir daran arbeiten, die Seite unter https://publiclab.org/tags nach diesem Design neu aufzubauen:

Screen Shot 2019-03-15 at 12 45 20 PM

Hier wird darĂŒber diskutiert: https://publiclab.org/notes/edie_blues/01-09-2019/the-tags-page-mockup-let-me-know-what-you-think#c22154

Die aktuelle Vorlage ist hier: https://github.com/publiclab/plots2/tree/master/app/views/tag/index.html.erb

Schritte, die ausgebrochen werden könnten, umfassen:

  • [x] macht es 2-spaltig #5100
  • [x] HinzufĂŒgen von https://publiclab.org/stats/graph in einem iframe zur 2. Spalte
  • [x] HinzufĂŒgen der Links unter der Grafik
  • [x] Neugestaltung der Suchleiste (https://github.com/publiclab/plots2/pull/5121)
  • [ ] Neugestaltung der Tabelle, um stattdessen eine Card-UI zu verwenden (beginnend mit Panels, hier: https://getbootstrap.com/docs/3.3/components/#panels) in einer partiellen Vorlage

Ich wĂŒrde gerne Hilfe bei diesem!

HTML Ruby break-me-up design help wanted

Hilfreichster Kommentar

Kann ich hier helfen? Es wÀre toll, wenn ich dazu beitragen könnte! Vielen Dank

Alle 46 Kommentare

@jywarren Ich wĂŒrde gerne an diesem Thema arbeiten. Kann ich mit dem ersten Teil beginnen, es zweispaltig zu machen.

Das wĂ€re super! Vielen Dank – und beachten Sie, dass wir uns derzeit bei Bootstrap 3 befinden.
docs unter https://getbootstrap.com/docs/3.3/ -- einen Blick wert!

Am Fr, 15. MĂ€rz 2019 um 13:31 Uhr Lekhika Dugtal [email protected]
schrieb:

@warren https://github.com/warren Ich wĂŒrde gerne an diesem Thema arbeiten.
Kann ich mit dem ersten Teil beginnen, es zweispaltig zu machen.

—
Sie erhalten dies, weil Sie den Thread verfasst haben.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/5090#issuecomment-473376431 ,
oder den Thread stumm schalten
https://github.com/notifications/unsubscribe-auth/AABfJ8N48ACVTtYlHFbsChYZIFpjb3dpks5vW9jfgaJpZM4b24tN
.

@jywarren Soll ich an der Neugestaltung der Suchleiste arbeiten?

Sicher! Aber du kannst wahrscheinlich mit #5832 (glaube ich?) fortfahren, wo du hast
schon begonnen! Dankeschön!

Am Freitag, 15. MĂ€rz 2019 um 13:38 Uhr schrieb Divya Baid [email protected] :

@jywarren https://github.com/jywarren Soll ich an der Neugestaltung des arbeiten ?
Suchleiste?

—
Sie erhalten dies, weil Sie erwÀhnt wurden.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/5090#issuecomment-473378955 ,
oder den Thread stumm schalten
https://github.com/notifications/unsubscribe-auth/AABfJxYkgDzS1hVu9VHErqzbSdRS-RzPks5vW9qKgaJpZM4b24tN
.

@jywarren Ist das, was der erste Teil sagt? Ich habe eine PR geöffnet, die sie in zwei Teile aufteilt.
Screenshot from 2019-03-16 10-11-38

@jywarren Ist es möglich, dass dieselbe Person auch an anderen Teilen arbeitet?
Wenn ja, dann wĂŒrde ich gerne auch den zweiten Teil machen. :)

@CleverFool77 Ich möchte hier helfen! Lass es mich wissen, wenn du irgendwo feststeckst, wir können es gemeinsam lösen!

Ich werde dich sicher @IshaGupta18 fragen :)

Hallo @IshaGupta18 ,
Ich arbeitete daran, Iframe zu zeigen. Soll ich den Code also noch einmal zusammen mit Ajax schreiben oder soll ich den in graph.html generierten Graphen hier anzeigen. Ich habe Ruby on Rails noch nicht in einem vollwertigen Projekt verwendet. Also noch dazu lernen.

Ja, ich denke, mehrere Personen können parallel arbeiten und/oder zusammenarbeiten!

FĂŒr den iframe zeigen wir den Graphen bereits wie gewĂŒnscht unter /stats/graph an, also
Ich denke, wir können es einfach mit einem Iframe anstelle von Ajax einbinden. Macht das
Sinn ergeben?

Am So, 17.03.2019 um 7:30 Uhr Lekhika Dugtal [email protected]
schrieb:

Hallo @IshaGupta18 https://github.com/IshaGupta18 ,
Ich arbeitete daran, Iframe zu zeigen. Also soll ich den Code schreiben
wieder zusammen mit ajax oder soll ich den in graph.html generierten Graphen anzeigen?
Hier.

—
Sie erhalten dies, weil Sie erwÀhnt wurden.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/5090#issuecomment-473656623 ,
oder den Thread stumm schalten
https://github.com/notifications/unsubscribe-auth/AABfJ3l_BuDF5LweofawSM1wzy2lk9S1ks5vXicsgaJpZM4b24tN
.

@jywarren So etwas -
Screenshot from 2019-03-18 23-57-57

Marge habe ich noch nicht gegeben. Und das Tag-Diagramm wird nicht geschnitten, sondern ich habe es gezoomt, da es nicht sichtbar war, weil ich 3-4 verschiedene kleine Diagramme hatte. Also habe ich es jetzt gezoomt

Kann ich hier helfen? Es wÀre toll, wenn ich dazu beitragen könnte! Vielen Dank

@jywarren @gauravano Können wir zuerst die PR zusammenfĂŒhren, weil ich denke, alle anderen Änderungen werden zuerst davon abhĂ€ngig sein. PR-Link
@gautamig54 Sie können mit jedem Teil dieser Ausgabe beginnen. Aber jetzt werden die weiteren Teile wohl voneinander abhÀngig sein. :Denken:

@gautamig54 Im Moment arbeite ich an der Statistikgrafik PR, die in der zweiten Spalte angezeigt werden soll. Wenn Sie interessiert sind, können Sie sich fĂŒr den dritten Teil entscheiden :wink:

@CleverFool77 Okay! Ich fange mit dem 3. an! Vielen Dank!

Soll ich die letzte Aufgabe erledigen? Kann ich mit diesem Teil anfangen?
Vielen Dank!

Hoppla wollte das Ganze nicht schließen! Jetzt wieder geöffnet - und danke fĂŒr die Zusammenarbeit!

Oh, oops, ok - also, ich sehe, es gibt eine Sequenz. Es sieht so aus, als hĂ€tte @CleverFool77 bereits #5100 begonnen und könnten wir das iframe-Zeug dort fertigstellen, bevor wir es zusammenfĂŒhren (normalerweise wĂ€ren 2 PRs großartig, aber diese sind so aneinander gebunden - es machte fĂŒr mich keinen Sinn, zusammenzufĂŒgen die erste und veröffentlichen dann eine halbwegs leere Seite, weißt du?), damit wir die Basisseite sofort zusammenfĂŒhren können, dann können wir mit den nĂ€chsten paar fortfahren? Danke an alle, das ist cool!

Wenn wir ein neues Element hinzufĂŒgen möchten, das das "Zoomen des Tag-Graphen" ist, denke ich, dass es möglicherweise mit dieser Dokumentation möglich ist: http://js.cytoscape.org/

und dieser code? -- https://github.com/publiclab/plots2/blob/master/app/views/tag/graph.html.erb

@jywarren Ich fĂŒge den iframe im selben pr mit einem anderen Commit hinzu.
In Bezug auf die Zoomfunktion war dies erforderlich, da mein lokales Diagramm nicht viele Daten enthÀlt.
Vielen Dank

@divyabaid16 Ich denke, wir mĂŒssen uns im letzten Teil

Ja, du hast absolut recht @CleverFool77.
Ich werde also die Tabelle in die Karten-UI umwandeln (da ich die Änderungen bereits vorgenommen habe) und eine PR dafĂŒr senden. Und dann können wir gemeinsam mit weiteren ErgĂ€nzungen in der BenutzeroberflĂ€che arbeiten. Hört sich gut an?
Vielen Dank!

Das ist großartig @divyabaid16 :

Okay, es tut mir sehr leid, dass ich dir @CleverFool77 nicht antworten

Super alle, danke!

Am Mo, 18.03.2019, 23:50 Uhr Lekhika Dugtal [email protected]
schrieb:

Das ist großartig @divyabaid16 https://github.com/divyabaid16

—
Sie erhalten dies, weil Sie erwÀhnt wurden.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/5090#issuecomment-474187999 ,
oder den Thread stumm schalten
https://github.com/notifications/unsubscribe-auth/AABfJ3dnNcKajiyf-RCUqtYVfAzl6Soyks5vYF5JgaJpZM4b24tN
.

Wir wĂŒrden in Zukunft sicherlich Ihre Hilfe brauchen @IshaGupta18 :
Ruhe dich aus und gute Besserung @jywarren

@divyabaid16 Pingen Sie mich an, nachdem Sie mit der Grundstruktur der Karten fertig sind :wink: . Wir werden zusammenarbeiten und es gemeinsam erarbeiten :smile:

Ja sicher @CleverFool77
Ich wĂŒrde gerne mit dir arbeiten :smile:

Hallo @jywarren Ich habe festgestellt, dass der Fix im vorgestellten Teil dieses Problems durchgefĂŒhrt wird. Aber Sie haben in Ausgabe #5004 erwĂ€hnt, dass dies in der Datenbank gespeichert ist. Kann ich dann einen Einblick dazu haben?

ZusammengefĂŒhrt #5100! Nun zu den nĂ€chsten Schritten; vielleicht ist https://github.com/publiclab/plots2/pull/5172 als nĂ€chstes

Ich sehe Fortschritte bei https://github.com/publiclab/plots2/pull/5169 sowie auf einem Kartenlayout!

Stellen wir sicher, dass das Kartenlayout von https://github.com/publiclab/plots2/pull/5176 von @divyabaid16 kompatibel ist ! Was fĂŒr eine erstaunliche Zusammenarbeit mit mehreren Personen hier !!! 🙌

Hallo @jywarren Das grundlegende Kartenlayout ist fĂŒr den Personenbereich. In Bezug auf das grundlegende Kartenlayout im Tag-Bereich denke ich, dass @divyabaid16 es funktioniert, wie sie bereits erwĂ€hnt hat. Sie hat im Sortierbereich großartige Arbeit geleistet. Wenn also dieselbe Person beide Arbeiten erledigt, kann die KompatibilitĂ€t leicht ĂŒberprĂŒft werden.

SpÀter wird die BenutzeroberflÀche von der Grundstruktur durch die Zusammenarbeit verschiedener Mitwirkender verbessert.
Vielen Dank !!!

Hallo @gautamig54 , brauchst du Hilfe?

@CleverFool77 Danke! Ich arbeite an der 3. Aufgabe. Ich werde sicher um Hilfe bitten, wenn es sein muss 👍

Ich bin tatsĂ€chlich verwirrt, ob ich die PR fĂŒr die Karten-UI senden soll oder nicht. Weil es ein Carrier Forward meiner vorherigen 2 PRs ist.

@jywarren @CleverFool77 Ich habe die 3. Aufgabe erledigt und eine PR dafĂŒr eingereicht. Bitte ĂŒberprĂŒfen Sie es.

Auf der Tags-Seite ist die Anzahl der Tags, die links angezeigt werden, höher, was dazu fĂŒhrt, dass rechts unter der Grafik leerer Raum entsteht. Ich denke, diesbezĂŒglich sollten einige Änderungen im Design vorgenommen werden. Was sagt @jywarren @CleverFool77?

Screen Shot 2019-03-21 at 7 58 16 PM

HI @divyabaid16 Ich wollte fragen, welche beiden vorherigen PR. Ist es nicht eine PR, die fĂŒr eine Art ist?

Oh, ich sehe es jetzt in Bezug auf das, was Sie mit 2 PRs meinen, @divyabaid16. Ich denke, wir mĂŒssen warten. :Denken:
@gautamig54 Mal sehen, was in der neuen BenutzeroberflĂ€che dafĂŒr vorhanden ist.

@CleverFool77 Vorgestellte Themen wurden dem neuen UI-Design nicht hinzugefĂŒgt?

@gautmig54 Ja, es wurde noch nicht hinzugefĂŒgt. Ich habe @jywarren in diesen Kommentaren oben nach dieser Frage gefragt.
Vielen Dank :)

Ah, sorry, antworte jetzt:

Hallo @jywarren Ich habe festgestellt, dass der Fix im vorgestellten Teil dieses Problems durchgefĂŒhrt wird. Aber Sie haben in Ausgabe #5004 erwĂ€hnt, dass dies in der Datenbank gespeichert ist. Kann ich dann einen Einblick dazu haben?

Ja, ich denke, wir sind jetzt damit einverstanden :

Obwohl wir die Seite einfach in "Tags" umbenennen sollten und vielleicht können wir den Text entfernen (ich kann ihn in die Datenbank kopieren, damit wir ihn leichter Àndern können, anstatt ihn in der Codebasis zu verfolgen!)

Eine Sache ist, dass die TabellenĂŒberschriften etwas ĂŒberfĂŒllt sind - ich denke, wir werden sie jedoch entfernen, wenn wir Karten erstellen?

image

Schauen Sie jetzt auf https://github.com/publiclab/plots2/pull/5176/ zum Sortieren, dann können Sie zur Kartenanzeige ĂŒbergehen! ( @CleverFool77 Ich habe deine Notiz zum Kartendesign der Personenseite gesehen, du hast völlig Recht, danke!)

https://github.com/publiclab/plots2/pull/5239 komplett - jetzt nur noch die Karten-UI!

Ich werde die PR fĂŒr diesen Teil bis heute senden.
Vielen Dank!

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

milaaraujo picture milaaraujo  Â·  3Kommentare

keshavsethi picture keshavsethi  Â·  3Kommentare

first-timers[bot] picture first-timers[bot]  Â·  3Kommentare

first-timers[bot] picture first-timers[bot]  Â·  3Kommentare

grvsachdeva picture grvsachdeva  Â·  3Kommentare