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

Fenêtre contenus Geotribu #157

Merged
merged 34 commits into from
Apr 29, 2024
Merged

Fenêtre contenus Geotribu #157

merged 34 commits into from
Apr 29, 2024

Conversation

gounux
Copy link
Contributor

@gounux gounux commented Mar 13, 2024

Ajout d'un fenêtre "Contenus Geotribu", lancée à partir d'une action présente dans le menu Web et la toolbar, qui utilise le flux JSONFeed pour récupérer les 50 derniers articles / RDP

Idées d'amélioration :

  • ajouter des petites icônes sympas pour les boutons
  • actions à définir au clic droit sur une ligne
  • créer un widget pour suggestion d'article basé sur le widget suggestion de news
  • passer les paramètres GET à la requête GitHub
  • changer l'icone de la toolbar QGIS

@github-actions github-actions bot added enhancement New feature or request UI User interface: forms, widgets... labels Mar 13, 2024
@Guts
Copy link
Member

Guts commented Mar 13, 2024

C'est quoi l'idée derrière ?
Chercher dans les contenus ? Pouvoir en proposer un nouveau ?

Un panneau dockable c'est un peu too much pour Geotribu non ? ou c'est plus pour se faire plaisir ?

@gounux
Copy link
Contributor Author

gounux commented Mar 13, 2024

Un panneau dockable c'est un peu too much pour Geotribu non ? ou c'est plus pour se faire plaisir ?

C'est inutile, donc il faut le faire ! Non plus sérieusement, ça peut être sympa d'avoir la liste des articles et les auteurs/autrices, si ça peut encourager la rédaction d'articles ...

C'est quoi l'idée derrière ? Chercher dans les contenus ? Pouvoir en proposer un nouveau ?

Oui c'est un peu ça, même si bon on va pas refaire un moteur de recherche comme sur le site web. Rassembler les actions pour proposer des articles / news dans des boutons

Je propose ça aussi pour me dépoussiérer le clavier en pyqgis. C'est vrai que c'est quand même un brin costaud, ceci dit certaines choses comme une refonte du client RSS peuvent être intéressantes pour d'autres trucs à venir

@Guts
Copy link
Member

Guts commented Mar 13, 2024

Pour la recherche y'a #52

@Guts
Copy link
Member

Guts commented Mar 13, 2024

OK vas-y, fais-toi plaisir on verra bien.

Je suis pas convaincu par le dock mais bon ça n'est que mon avis. Au pire, on en fait une option.

@Guts
Copy link
Member

Guts commented Mar 13, 2024

au fait @gounux c'est pas vraiment documenté mais en plus du RSS, il y a aussi le JSON Feed sur Geotribu : https://geotribu.fr/feed_json_created.json

Si jamais tu préfères le JSON au XML et vu que le parser de QgsNewsFeed utilise du JSON...

@gounux
Copy link
Contributor Author

gounux commented Mar 13, 2024

Ah super ! Oui je préfère le JSON au XML :D

Je suis pas convaincu par le dock mais bon ça n'est que mon avis. Au pire, on en fait une option.

C'est vrai, un simple QDialog qui s'ouvre suffit amplement

@gounux gounux changed the title Geotribu toolbox Fenêtre contenus Geotribu Mar 14, 2024
@gounux gounux requested a review from Guts March 18, 2024 09:00
@gounux gounux marked this pull request as ready for review March 18, 2024 09:00
@gounux
Copy link
Contributor Author

gounux commented Mar 18, 2024

cc @Guts c'est prêt pour review cette histoire de fenêtre des contenus. Manquerait peut-être des petites icônes sympas sur les boutons et actions clic-droit mais ça peut être fait plus tard j'ai l'impression

Copy link
Member

@Guts Guts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bon c'est cool mais y'a encore pas mal de choses qui traînent.

En plus des remarques déjà mises en review :

  • trouver une autre icône pour mettre dans la barre des tâches.
  • trouver d'autres icônes pour typer les contenus dans la liste, les logos sont trop redondants. Dans l'idéal :
    • utiliser l'icône de l'en-tête YAML
    • fallback sur une icône générique (une pour les articles, une pour les RDP)
  • utiliser si possible des icônes intégrées à QGIS (cf https://pyqgis-icons-cheatsheet.geotribu.fr/)
  • ajouter la description de l'article en tooltip ?
  • je trouve la date pas lisible

Sinon, ergonomiquement parlant (déso je m'enflamme) :

  • mettre les résultats dans un tableau triable par type, date, titre, description (non sortable). Un peu comme la sortie du CLI mais en Qt (donc + jouli et fonctionnel)
  • ajouter un filtre Qt pour avoir un semblant de recherche
  • ajouter une combobox pour sélectionner un tag

qtribu/gui/dlg_contents.py Outdated Show resolved Hide resolved
qtribu/gui/dlg_contents.py Outdated Show resolved Hide resolved
qtribu/gui/dlg_contents.py Outdated Show resolved Hide resolved
qtribu/gui/dlg_contents.py Outdated Show resolved Hide resolved
qtribu/gui/dlg_contents.py Outdated Show resolved Hide resolved
qtribu/gui/geotribu_toolbox.py Outdated Show resolved Hide resolved
qtribu/logic/json_feed.py Outdated Show resolved Hide resolved
@gounux gounux marked this pull request as draft April 15, 2024 13:52
@Guts
Copy link
Member

Guts commented Apr 26, 2024

notes mémo :

  • ajouter liste déroulante pour filtrer sur les auteur/ices
  • ajouter une valeur "marqueur" genre - en haut des listes déroualntes pour permettre de déselectionner un filtre

@gounux
Copy link
Contributor Author

gounux commented Apr 28, 2024

cc @Guts , c'est prêt pour review je pense

Un nouveau formulaire pour suggérer article est implémenté mais ne déclenche pas d'action, ce qui peut être fait dans une autre PR je pense (ouverture d'une issue GitHub avec les paramètres du template dans l'URL)

@gounux gounux marked this pull request as ready for review April 28, 2024 10:15
@gounux gounux requested a review from Guts April 28, 2024 10:15
@Guts
Copy link
Member

Guts commented Apr 28, 2024

cc @Guts , c'est prêt pour review je pense

Un nouveau formulaire pour suggérer article est implémenté mais ne déclenche pas d'action, ce qui peut être fait dans une autre PR je pense (ouverture d'une issue GitHub avec les paramètres du template dans l'URL)

Ah mince, je l'ai aussi fait de mon côté haha ! Bon je vois et fusionne nos deux créations ok ?

Copy link

sonarcloud bot commented Apr 28, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
13.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Member

@Guts Guts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai modifié les icônes des boutons pour que ce soit plus distinct.

Il faudra refaire une passe sur le formulaire d'article car certains champs ne correspondent pas : icône, catégorie, licence...

Mais ça, c'est une autre PR !

@gounux
Copy link
Contributor Author

gounux commented Apr 29, 2024

Ok alors c'est bon pour merger celle-ci ?

Modulo les duplications de code qui viennent sûrement du nouveau formulaire copié-collé depuis l'autre

@Guts
Copy link
Member

Guts commented Apr 29, 2024

Ok alors c'est bon pour merger celle-ci ?

Modulo les duplications de code qui viennent sûrement du nouveau formulaire copié-collé depuis l'autre

Oui, c'est ce que signifie le "Approved" 😉

@Guts Guts merged commit d9bd377 into main Apr 29, 2024
6 of 7 checks passed
@Guts Guts deleted the feature/geotribu-toolbox branch April 29, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI User interface: forms, widgets...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants