Bug 95079 - add GHz to autocorrection exceptions
Summary: add GHz to autocorrection exceptions
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: Other All
: medium trivial
Assignee: Not Assigned
URL:
Whiteboard: ToBeReviewed target:5.2.0
Keywords: difficultyBeginner, easyHack, skillScript
Depends on:
Blocks:
 
Reported: 2015-10-15 09:35 UTC by Dennis Roczek
Modified: 2017-02-14 08:58 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Roczek 2015-10-15 09:35:18 UTC
there are many physics units with kilo, giga, terra, ... which gets autocorrected to lowercase. These should be added. Just had a user in #libreoffice which do ask that question.
Comment 1 Julien Nabet 2015-10-15 10:18:18 UTC
It seems this exception is already taken into account for some locales, see http://opengrok.libreoffice.org/search?q=ghz&project=core&defs=&refs=&path=&hist=

Which locale do you use?
Comment 2 Germano Massullo 2015-10-15 10:52:44 UTC
(In reply to Julien Nabet from comment #1)
> It seems this exception is already taken into account for some locales, see
> http://opengrok.libreoffice.org/
> search?q=ghz&project=core&defs=&refs=&path=&hist=
> 
> Which locale do you use?

Italian on Libreoffice 4.4.5.2
Comment 3 Dennis Roczek 2015-10-15 11:08:20 UTC
same here on de-DE
Comment 5 Björn Michaelsen 2015-11-21 14:29:12 UTC
sounds more like something to add to the extras/source/wordbook/technical.dic than to to add the each and every locale, see also: tdf#87605
Comment 6 Robinson Tryon (qubit) 2015-12-14 06:47:23 UTC Comment hidden (obsolete)
Comment 7 Dennis Roczek 2016-02-05 19:47:10 UTC
Normally I would, but I'm highly busy at the moment with other tasks (e.g. mapping for Numbers import filter)
Comment 8 Robinson Tryon (qubit) 2016-02-18 14:52:23 UTC Comment hidden (obsolete)
Comment 9 Commit Notification 2016-04-15 07:06:11 UTC
danielat998 committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fbfcde95eb0bde55857b1102716b4514eddd5417

tdf#95079 added a few things to techincal.dic

It will be available in 5.2.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 10 Björn Michaelsen 2016-04-22 08:13:43 UTC
(In reply to Commit Notification from comment #9)
> danielat998 committed a patch related to this issue.
> It has been pushed to "master":

Good job, I assume this can be marked as resolved then! (I wonder if it should be "GHz=" though -- for LibreOffice to get the case right?)