b4n commented on this pull request.

Otherwise LGTM (not that I speak Catalan though, but I'll trust you on that)

> @@ -1852,7 +1851,7 @@ msgstr "_Visualitza"
 
 #: ../data/geany.glade.h:391
 msgid "Change _Font..."
-msgstr "Canvia el _tipus de lletra..."
+msgstr "Canvia la _lletra..."

changing mnemonics isn't awesome, so 2 questions:
1. have you checked `l` isn't already used in that menu?
2. is the new mnemonic feeling more natural, or could/should you keep the `t` 
one? (as *lletra* has one, it would be doable without making it ugly and 
unexpected)

Not saying it's not a good change, just that it should possibly be done on 
purpose :)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1432#pullrequestreview-27724007

Reply via email to