-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add bold to the focused word in context for readability #47
Conversation
Warning : This change is breaking the former data model
@Jean-Baptiste-Camps I'd be happy to get your review on this one :) |
I also rewrote the CLI :
|
Created corpus-from-dir to easily reimport previous dump Not tested the last one
Bon, ce qui est bien avec les longs voyages, c'est qu'on a du temps. @Jean-Baptiste-Camps : quelques gros changements d'interface entre autres, de CLI et un de modèle qui induit d'exporter / réimporter.
L'aide du client se trouve en faisant |
Petite remarque, mais je pense que c'est intentionnel: le 'last edit tokens' me renvoie, non pas au dernier token modifié, mais à la dernière page vue de l'écran 'edit tokens', ce qui est un peu différent. |
Yup. Comme je t'ai dit au tel, on s'est limité à la last page pour le moment ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parfait, je merge !
Merde, il fallait squasher pour faire propre :( |
C'est ce que t'as fait. PARFAIT ! |
Changed required by @ArianePinche in #44
This pull requests breaks the former data model, you'll need to export / import your old corpora.