Bug 120207 - Change the default font choice order for LibreOffice Calc in Hebrew
Summary: Change the default font choice order for LibreOffice Calc in Hebrew
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
6.2.0.0.alpha0+
Hardware: All All
: medium minor
Assignee: Eyal Rozenberg
URL:
Whiteboard: target:7.2.0
Keywords:
Depends on:
Blocks: RTL-Hebrew
  Show dependency treegraph
 
Reported: 2018-09-30 09:08 UTC by Eyal Rozenberg
Modified: 2021-02-18 15:40 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Arial, Alef, Liberation Sans and Nachlieli at 10pt (151.86 KB, image/png)
2018-10-09 10:52 UTC, Eyal Rozenberg
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eyal Rozenberg 2018-09-30 09:08:05 UTC
Description:
Calc use is a lot more on-screen, and at lower size in points (10 instead of 12 on my Linux machine).

Currently, the font order is:

Nachlieli CLM;Alef;Noto Sans Hebrew;Liberation Sans;Arial;Arial Hebrew;Lucida Sans Unicode;Lucida Grande;Arial Unicode MS

according to: 
https://cgit.freedesktop.org/libreoffice/core/commit/?id=a2158c6ca2ba831665019827889f01e98dc93394

But Nachlieli CLM has relatively low readability, IMO, in small font sizes, and Alef is not much tter. I therefore suggest that the order become:

Liberation Sans;Noto Sans Hebrew;Nachlieli CLM;Alef;Arial;Arial Hebrew;Lucida Sans Unicode;Lucida Grande;Arial Unicode MS

Or alterntvely:

Arial;Liberation Sans;Noto Sans Hebrew;Nachlieli CLM;Alef;Arial Hebrew;Lucida Sans Unicode;Lucida Grande;Arial Unicode MS

(Note that I havent' sctually looked at the last three on the list recently.

Steps to Reproduce:
Type in some Hebrew text along a column.

Actual Results:
Text comes out in Nachlieli CLM, and is not very readoable

Expected Results:
Text comes out in a more readable font


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 Eyal Rozenberg 2018-10-09 10:52:51 UTC
Created attachment 145519 [details]
Arial, Alef, Liberation Sans and Nachlieli at 10pt

Some have raised the question of whether Alef should not be used before Liberation Sans. Well, no...

Attached is a screenshot with the three fonts used for the same content - a part of an LO Calc chart. Note that Liberation Sans' readability advantage is a bit of a "cheat" since its glyph width and height at 10pt are larger than Arial's and Alef's.

Nachlieli comes in third IMHO, but it's not as readable as Liberation Sans. (It's about as readable even as Liberation Sans at 9pt, maybe less; but that would definitely be an unfair comparison).
Comment 2 Buovjaga 2018-10-27 14:17:11 UTC
Ok. Feel free to propose a patch https://wiki.documentfoundation.org/Development/gerrit
Comment 3 QA Administrators 2019-10-28 03:29:30 UTC Comment hidden (obsolete)
Comment 4 Eyal Rozenberg 2019-10-28 05:59:38 UTC
(In reply to Buovjaga from comment #2)
> Ok. Feel free to propose a patch
> https://wiki.documentfoundation.org/Development/gerrit

I actually need to get around to proposing that patch. It's just that I've never done anything like that for LO before; I don't have a checked out copy of the sources.

Anyway, this bug should not be auto-closed...
Comment 5 Buovjaga 2019-10-28 06:20:55 UTC
(In reply to Eyal Rozenberg from comment #4)
> (In reply to Buovjaga from comment #2)
> > Ok. Feel free to propose a patch
> > https://wiki.documentfoundation.org/Development/gerrit
> 
> I actually need to get around to proposing that patch. It's just that I've
> never done anything like that for LO before; I don't have a checked out copy
> of the sources.
> 
> Anyway, this bug should not be auto-closed...

To avoid cloning the source, you can also edit directly in gerrit: https://wiki.documentfoundation.org/Documentation/GerritEditing
Comment 6 Eyal Rozenberg 2021-02-13 09:35:59 UTC
(In reply to Buovjaga from comment #5)

So, I've finally gotten around to doing this. Now I need reviewers for the patch...

https://gerrit.libreoffice.org/c/core/+/110790

Plus, I need more opinions on the order of fonts I've chosen.
Comment 7 Commit Notification 2021-02-16 14:16:18 UTC
Eyal Rozenberg committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/21da461f374357198bafaea02e28d24de02c6772

tdf#120207 Change the order of default Hebrew font choice for spreadsheets

It will be available in 7.2.0.

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

Affected users are encouraged to test the fix and report feedback.