Skip to content
This repository has been archived by the owner on Dec 6, 2019. It is now read-only.

Autocomplete and beer characteristics compared against style #137

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

flagos
Copy link
Contributor

@flagos flagos commented Apr 7, 2017

Hi,

The idea would be here to display the main characteristics of the beer in a graphic and to display it against beer style.

The beer style can be chosen from BJCP but can also not, just an autocompletion feature to guide the user, not to force him to pick one.

Note: Actually, the styles and some javascript are loaded dynamically. It's done this way temporarily for licensing issues. If you are okay with the feature, let me know and will try to solve this. Just let me know your feeling about that.

Cheers

@flagos
Copy link
Contributor Author

flagos commented Sep 8, 2017

Hi,

It's about 5 months since this feature request so I take the opportunity to send a gentle reminder. I know you are quite busy but I would like to know more your feelings about this feature request: do you think it's a good idea ? Would you mind to merge if some technical conditions are met ? Or the idea could be good to you but you need more time to assess it ?

Feel free to discuss, I will not over-argue on it :-)

@314r
Copy link
Owner

314r commented Sep 13, 2017

Dejà, merci pour cette contribution importante.

Je vois plusieurs problèmes :

  • le guide bjcp est copyrighté. La version de la conversion xml que tu utilises ne mentionne aucune licence, ce qui est très moyen, d'autant que site officiel renvoie vers deux sources xml sur Github, avec des licences restrictives. Donc on ne pourra pas l'inclure dans JB qui est en GPL. A ce stade, c'est rédhibitoire.

  • l'absence de traduction complique l'intégration.

  • si on ne se réfère pas à un guide, le truc perd tout son sens et l'interface devient même un peu bizarre.

  • le graphique ne se met pas à jour au changement de style.

  • sur une stout à 109 ebc le curseur est en dehors de la barre. Idem si on augmente les ibus, le rapport bugu, etc

  • le bloc de barres prend beaucoup de place en hauteur. Il faudrait au moins limiter son affichage au mode édition, et au mieux trouver un autre mode d'affichage moins invasif. Par exemple, afficher un graphe lors du clic sur un des élément de la barre actuelle.

A+

@flagos
Copy link
Contributor Author

flagos commented Sep 18, 2017

Salut,

Tout d'abord, merci du retour.

Par rapport à la licence, qui me semble être le principal point bloquant, je me suis effectivement pas trop pris la tête car le bjcp semble vouloir seulement limiter les usages commerciaux. Maintenant, à strictement parler, on est bien sur un problème d'incompatibilité avec la GPL. Le même souci semble se poser avec le guide BA.

Tout cela est finalement assez regrettable car ces guides ont justement été édités dans un esprit de partage de connaissance (et il me semble dans un esprit proche du logiciel libre) mais les termes de la licence empêchent son integration dans un logiciel sous GPL alors qu'on va retrouver cette feature ailleurs sur d'autres softwares.

A partir de là, je ne ne vois finalement que 2 solutions:

  • soit s'orienter vers un guide compatible GPL. Je n'en connais pas, mais je serai heureux d'en connaitre.
  • laisser à l'utilisateur la charge de télécharger le xml et d'accepter la licence afférante.

Cette deuxième solution aurait l'avantage de pouvoir supporter différents guides.

@314r
Copy link
Owner

314r commented Sep 23, 2017

Honnêtement , je ne suis pas sûr que le but du BJCP soit le partage de connaissances. Sur la page principale du site, il est aussi beaucoup question de certification, de sous... et des détournements de l'ancien président-trésorier...
J'imagine que oui, laisser le téléchargement à la discrétion de l'utilisateur serait le plus malin.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants