feature(emojis): add an emojis picker #25
Conversation
Guts
force-pushed
the
feature/emoji-picker
branch
from
6cc4333 to
2c2a9d7
Compare
Guts
force-pushed
the
feature/emoji-picker
branch
from
8532057 to
cf1362c
Compare
for more information, see https://pre-commit.ci
…ince it's too complicated
gounux
left a comment
gounux
left a comment
There was a problem hiding this comment.
Merci beaucoup @Guts, c'est vraiment cool d'avoir des emojis 🔥 🎉 !
J'ai pu checkout en local et tester, 1e passe de review avec quelques questions :
-
si j'ai tout bien tout compris, ce choix de font compatible emojis serait adressé uniquement aux utilisateurs Linux ? Si c'est le cas, j'aurais tendance à ne pas mettre par défaut la font Noto color emoji,
-
est-ce que si la font est changée dans les settings, on irait pas redéclencher l'affichage du widget, de la même manière que si le pseudo ou l'instance est changée, ici ?
Lines 379 to 384 in 39799be
| def get_font_from_settings( | ||
| self, plugin_settings: Optional[PlgSettingsStructure] = None | ||
| ) -> QFont: |
There was a problem hiding this comment.
Est-ce que cette fonction pourrait trouver sa place directement dans les settings ?
| color_admin: str = "#ffa500" | ||
| messages_font_family: str = "Noto Color Emoji,10,-1,5,50,0,0,0,0,0" | ||
| messages_font_size_pts: int = 10 | ||
| default_font_emoji_family: str = "Noto Color Emoji" |
There was a problem hiding this comment.
Est-ce qu'on mettrait pas par défaut la font globale des paramètres généraux de QGIS ?
| if isinstance(item, QChatTextTreeWidgetItem): | ||
| item.setFont(MESSAGE_COLUMN, self.messages_font) |
There was a problem hiding this comment.
Est-ce qu'on pourrait pas mettre ça dans le init d'un QChatTextTreeWidgetItem ?
qchat/qchat/gui/qchat_tree_widget_items.py
Lines 160 to 163 in 39799be
|
Uh oh!
There was an error while loading. Please reload this page.