Plots2: Implémenter un nouveau design pour la page Tags (doit être rompu)

@jywarren J'aimerais travailler sur ce problème. Puis-je commencer par la première partie de la création de deux colonnes.

Ce serait super ! Merci -- et notez que nous sommes actuellement sur Bootstrap 3,
docs sur https://getbootstrap.com/docs/3.3/ -- ça vaut le détour !

Le ven 15 mars 2019 à 13:31 Lekhika Dugtal [email protected]
a écrit:

@warren https://github.com/warren J'aimerais travailler sur ce problème.
Puis-je commencer par la première partie de la création de deux colonnes.

-
Vous recevez ceci parce que vous avez créé le fil.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/5090#issuecomment-473376431 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ8N48ACVTtYlHFbsChYZIFpjb3dpks5vW9jfgaJpZM4b24tN
.

@jywarren Dois-je travailler sur la refonte de la barre de recherche ?

Sûr! Mais vous pouvez probablement aller de l'avant avec #5832 (je pense?) où vous avez
déjà commencé ! Merci!

Le vendredi 15 mars 2019 à 13h38, Divya Baid [email protected] a écrit :

@jywarren https://github.com/jywarren Dois-je travailler sur la refonte du
barre de recherche?

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/5090#issuecomment-473378955 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJxYkgDzS1hVu9VHErqzbSdRS-RzPks5vW9qKgaJpZM4b24tN
.

@jywarren C'est ce que dit la première partie ? J'ai ouvert un PR le divisant en deux parties.

@jywarren Est-il possible pour une même personne de travailler aussi sur d'autres pièces ?
Si oui, alors j'aimerais aller pour la deuxième partie aussi. :)

@CleverFool77 J'aimerais aider ici ! Faites-moi savoir si vous êtes coincé quelque part, nous pouvons le résoudre ensemble !

Je vais vous demander à coup sûr @IshaGupta18 :)

Salut @IshaGupta18 ,
Je travaillais sur la partie de montrer iframe. Dois-je donc réécrire le code avec ajax ou dois-je afficher le graphique généré dans graph.html ici. Je n'ai pas encore utilisé ruby ​​on rails sur un projet à part entière. Donc encore en train d'apprendre à ce sujet.

Oui, je pense que plusieurs personnes peuvent travailler en parallèle et/ou collaborer !

Pour l'iframe, nous montrons déjà le graphique comme nous le voulions dans /stats/graph, donc
Je pense que nous pouvons simplement l'inclure en utilisant un iframe au lieu d'ajax. Est-ce que
avoir du sens ?

Le dimanche 17 mars 2019 à 7h30 Lekhika Dugtal [email protected]
a écrit:

Salut @ IshaGupta18 https://github.com/IshaGupta18 ,
Je travaillais sur la partie de montrer iframe. Alors devrais-je écrire le code
à nouveau avec ajax ou devrais-je montrer le graphique généré dans graph.html
ici.

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/5090#issuecomment-473656623 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ3l_BuDF5LweofawSM1wzy2lk9S1ks5vXicsgaJpZM4b24tN
.

@jywarren Quelque chose comme ça -

Je n'ai pas encore donné de marge. Et le graphique des balises n'est pas coupé, je l'ai plutôt zoomé car il n'était pas visible car j'avais 3-4 petits graphiques différents. Donc pour l'instant j'ai zoomé

Puis-je être utile ici? Ce serait super si je pouvais contribuer ! Merci

@jywarren @gauravano Pouvons-nous d'abord fusionner les relations publiques car je suppose que tous les autres changements Lien RP
@ gautamig54 Vous pouvez commencer par n'importe quelle partie de ce problème. Mais maintenant, les autres parties dépendront les unes des autres, je suppose. :pensée:

@ gautamig54 En ce moment, je travaille sur le graphique de statistiques PR à afficher dans la deuxième colonne. Si vous êtes intéressé, vous pouvez opter pour la troisième partie :wink:

@CleverFool77 D'accord ! Je vais commencer par le 3ème ! Merci!

Dois-je terminer la dernière tâche ? Puis-je commencer par cette partie ?
Merci!

Oups, je ne voulais pas tout fermer ! Réouverture maintenant - et merci pour votre collaboration !

Oh, oups, ok - donc, je vois, il y a une séquence. Il semble que @CleverFool77 ait déjà commencé #5100 et pourrions-nous terminer le travail de l'iframe avant de le fusionner (généralement 2 PR seraient formidables, mais ils sont tellement liés les uns aux autres - cela n'avait pas de sens pour moi de fusionner le premier et ensuite publier une page qui est à moitié vide, vous savez?) afin que nous puissions fusionner la page de base à la fois, puis nous pouvons passer aux suivantes ? Merci à tous, c'est cool !

Si nous voulons ajouter un nouvel élément qui consiste à 'zoomer le graphique des balises', je pense que c'est possible en utilisant peut-être cette documentation: http://js.cytoscape.org/

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

@jywarren Je vais ajouter l'iframe dans le même pr avec un autre commit.
En ce qui concerne la fonction de zoom, elle était nécessaire car mon graphique local ne contient pas beaucoup de données.
Merci

@ divyabaid16 Je suppose que nous devons rompre dans la dernière partie. :pensant: Pas sûr cependant

Oui, vous avez tout à fait raison @CleverFool77.
Donc, ce que je vais faire, c'est convertir la table dans l'interface utilisateur de la carte (car j'ai déjà fait les modifications) et envoyer un PR pour cela. Et puis nous pouvons travailler en collaboration avec d'autres ajouts dans l'interface utilisateur. Ça a l'air bien?
Merci!

C'est super @divyabaid16 :wink:

D'accord, je suis vraiment désolé de ne pas avoir pu vous répondre @CleverFool77 . Je serai plus utile à partir de là. Un travail incroyable est fait ici! Faites-moi savoir si je peux aider. Merci tout le monde!

Génial à tous, merci !

Le lundi 18 mars 2019, 23h50 Lekhika Dugtal [email protected]
a écrit:

C'est super @divyabaid16 https://github.com/divyabaid16

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/5090#issuecomment-474187999 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ3dnNcKajiyf-RCUqtYVfAzl6Soyks5vYF5JgaJpZM4b24tN
.

Nous aurions sûrement besoin de votre aide à l'avenir @ IshaGupta18 :wink:
Reposez- vous et @jywarren

@ divyabaid16 Faites-moi un ping une fois que vous avez terminé avec la structure de base des cartes :wink: . Nous collaborerons et travaillerons ensemble :sourire:

Tu es sûr @CleverFool77
J'adorerais travailler avec toi :sourire:

Salut @jywarren, j'ai trouvé le correctif à faire dans la partie vedette de ce problème. Mais vous avez mentionné dans le numéro 5004 que cela est stocké dans la base de données. Puis-je avoir un aperçu à ce sujet?

Fusionné #5100 ! Passons maintenant aux étapes suivantes ; peut-être que https://github.com/publiclab/plots2/pull/5172 est le prochain ?

Je vois des progrès sur https://github.com/publiclab/plots2/pull/5169 aussi sur une mise en page de carte !

Assurons-nous que la disposition des cartes par @CleverFool77 est compatible avec le tri dans https://github.com/publiclab/plots2/pull/5176 par @divyabaid16 ! Quelle incroyable coopération multi-personnes ici !!! ??

Salut @jywarren La disposition de base de la carte est pour la section des gens. En ce qui concerne la disposition de base des cartes dans la section des balises, je suppose que @ divyabaid16 fonctionne comme elle l'a déjà mentionné. Elle a fait un excellent travail dans la section de tri, donc si la même personne fait les deux, la compatibilité sera vérifiée facilement.

Plus tard, l'interface utilisateur sera améliorée à partir de la structure de base avec une collaboration entre divers contributeurs.
Merci !!!

Salut @gautamig54 , Avez-vous besoin d'aide ?

@CleverFool77 Merci ! Je travaille sur la 3ème tâche. Je demanderai sûrement de l'aide si besoin

En fait, je ne sais pas s'il faut envoyer le PR pour l'interface utilisateur de la carte ou non. Parce que c'est un transporteur en avant de mes 2 PR précédents.

@jywarren @CleverFool77 J'ai effectué la 3ème tâche et j'ai soumis un PR pour la même chose. Veuillez l'examiner.

Sur la page des balises, le nombre de balises affichées à gauche est plus important, ce qui entraîne un espace vide à droite, sous le graphique. Je pense que certains amendements devraient être apportés à ce sujet dans la conception. Que dit @jywarren @CleverFool77 ?

HI @ divyabaid16 Je voulais demander quels sont les deux précédents PR. N'est-ce pas un PR qui est pour trier ?

Oh, je le vois maintenant concernant ce que vous voulez dire à propos de 2 PR, @ divyabaid16. Je suppose que nous devons attendre. :pensée:
@ gautamig54 Voyons ce qu'il y a dans la nouvelle interface utilisateur pour cela.

@CleverFool77 Les sujets en vedette n'ont pas été ajoutés à la nouvelle conception de l'interface utilisateur ?

@ gautmig54 Oui, il n'a pas encore été ajouté. J'ai posé cette question à @jywarren dans ces commentaires ci-dessus.
Merci :)

Ah, désolé, je réponds maintenant :

Salut @jywarren, j'ai trouvé le correctif à faire dans la partie vedette de ce problème. Mais vous avez mentionné dans le numéro 5004 que cela est stocké dans la base de données. Puis-je avoir un aperçu à ce sujet?

Oui, je pense que nous sommes d'accord maintenant : https://publiclab.org/tags

Bien que nous devions renommer la page simplement "Tags" et peut-être que nous pouvons supprimer le texte (je peux le copier dans la base de données afin que nous puissions le modifier plus facilement au lieu de le suivre dans la base de code !)

Une chose est que les en-têtes de table sont un peu encombrés - je suppose que nous les supprimerons quand nous ferons des cartes, cependant?

En regardant https://github.com/publiclab/plots2/pull/5176/ maintenant pour le tri, puis prêt à passer à l'affichage des cartes ! ( @ CleverFool77 j'ai vu votre note sur la conception de la carte de la page des personnes, vous avez tout à fait raison, merci !)

https://github.com/publiclab/plots2/pull/5239 terminé - maintenant juste l'interface utilisateur de la carte !

J'enverrai le PR pour cette partie d'ici aujourd'hui.
Merci!