Issue 61775

Summary: Too Many Fonts in Context Menu; also context menu can take many seconds to appear with too many fonts installed
Product: Writer Reporter: pesala <pesala>
Component: uiAssignee: eric.savary
Status: CLOSED FIXED QA Contact: issues@sw <issues>
Severity: Trivial    
Priority: P3 CC: don.troodon, issues, kpalagin, masaya.k, niemayer, www.openoffice.org, yoshimit
Version: OOo 2.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: ENHANCEMENT Latest Confirmation in: ---
Developer Difficulty: ---

Description pesala 2006-02-08 10:11:38 UTC
I only have 210 fonts installed, but that still makes it impractical to select fonts 
from the context menu. Users with thousands of fonts are not unusual. This item 
shouldn't be on the context menu at all. One user with 3,500 fonts asked if this was 
the reason that the context menu took five seconds to appear. 

Possible solutions:

• Allow editing of the context menu or make this item optional
• Show a list of ten of the most recently used fonts with a ...more submenu
Comment 1 michael.ruess 2006-02-08 16:01:37 UTC
Reassigned to ES.
Comment 2 eric.savary 2006-02-08 16:52:12 UTC
Reassigned
Comment 3 termyte97 2006-05-02 01:59:41 UTC
Or context menu's Fonts either optional, editable or show only up to 5 recent
used fonts or most used fonts.

I have observed that having 1000+ fonts can cause a 60 secs freeze in the Open
office writer when right-click in the context menu.
Comment 4 taladan 2006-06-05 15:12:15 UTC
I think that this issue is quite possibly the same as my issue:

http://qa.openoffice.org/issues/show_bug.cgi?id=65566

with ~ 6,000 fonts on the system writer locks down when I right click.  Not 60
seconds and then pop context...just completely locks down.  Is there any way to
edit what appears in the context menu?

Tal
Comment 5 bytesmythe 2006-06-06 20:40:58 UTC
*** Issue 61775 has been confirmed by votes. ***
Comment 6 hwoarang 2006-06-30 22:56:19 UTC
*** Issue 65566 has been marked as a duplicate of this issue. ***
Comment 7 hajo4 2006-07-14 00:27:03 UTC
I would suggest a new tab "Fonts" in the "Tools/Customize"-dialog,
e.g. with checkboxes for selecting a set of favorite fonts.
Comment 8 andreschnabel 2007-01-10 18:15:12 UTC
*** Issue 58491 has been marked as a duplicate of this issue. ***
Comment 9 kpalagin 2007-04-05 10:13:18 UTC
*** Issue 18620 has been marked as a duplicate of this issue. ***
Comment 10 kpalagin 2007-04-05 10:19:06 UTC
*** Issue 56174 has been marked as a duplicate of this issue. ***
Comment 11 kpalagin 2007-04-05 10:34:38 UTC
Dear developers,
please consider this issue.
The biggest problem is that context menu is slow (many seconds) to appear 
every time it is called if large number of fonts is installed (which is not 
uncommon). In some cases (56174) we even get hung because of that.

I am tempted to change this to defect because 5 seconds delay for right-click 
to cause some effect is defect.

(And this is in addition to usability problem when selecting from hundreds of 
items in single menu).

Thanks a lot for your attention.
WBR,
K. Palagin.
Comment 12 pniemayer 2007-04-05 11:22:17 UTC
As the author of ticket 18620 I can nothing but say that the handling of this
issue is an embarrassment for the whole Open Source community.

_4 YEARS_ ago I provided not only a defect report but also a patch to enable
users to use lots of fonts with OpenOffice.

This patch was disregarded with sniffy reasoning such as "... subject to
undergo changes in the near future anyway. The scenario 'install a
thousand' keeps me from regarding this as a 'performance defect' yet.".

Well done, OpenOffice team, you were successful in keeping me from ever
contributing to your project again.

So please don't complain again that so few developers out there are actually
working on OpenOffice - you don't want our input, so be it!

(BTW: I still have to use OpenOffice executables compiled on my own, since I
haven't un-installed the thousands of fonts. I need them, and so do others.)
Comment 13 kpalagin 2007-04-06 08:39:06 UTC
pniemayer,
thanks a lot for your response.
Any chance you could provide short specification of what your patch does (so 
that I could propose it for Russian derivative of OpenOffice)?
Thank you very much.

P.S. Please do not be upset with developers - they have to deal with dozens of 
issues per day, so it is inevitable that something falls through cracks. If 
that happens to just be persistent.
Comment 14 pniemayer 2007-04-11 11:32:35 UTC
My patch - as uploaded to issue 18620 - is extremely simple: Assuming that
nobody would want to choose one font out of a thousand by using a right-click
selection list, I just put in a hard limit on how many fonts would be included
in the context menu. The user can then still use the font selection text entry
field at the top to type names or browse through fonts - a much saner method
when lots of fonts are installed.

This simple change made StarOffice usable again, at last...
Comment 15 kpalagin 2007-04-11 11:47:59 UTC
pniemayer,
thank you very much for your reply.
Does your patch reduce the time it takes for context menu to appear?
Is it still valid (that is, can it be used with current code)?
Comment 16 pniemayer 2007-04-11 20:21:42 UTC
Yes, of course it reduces the time to render the context menu dramatically. It
was meant to make the difference between "unusable" (as OOO is with many fonts
installed) and "usable".

I don't know whether the 4-years-old-patch would still apply unchanged to the
current source - haven't updated for quite a while.

And BTW: Me being fed up with this project is not because I blame developers for
not having the time to fix a problem. I'm through with this project because even
if a ready-to-use solution is offered, it's refused for no reason other than
snootiness.
Comment 17 kpalagin 2007-04-12 07:34:07 UTC
pniemayer,
developers of Russian derivative of OpenOffice are ready to integrate the 
patch if it is ok code-wise. Could you, please, make sure that patch is valid 
for current master?

Thank you very much!
WBR,
K. Palagin.
Comment 18 dsilas 2007-04-24 02:12:01 UTC
Is the four-year-old patch which fixes this issue going to be applied anytime
soon? This bug makes Writer unusable for me because I have quite a few
fonts--like many other people--and have difficulty remembering not to
right-click and bring up the context menu. This should have been fixed when the
patch was submitted in *2003*. 
Comment 19 pesala 2008-05-16 04:46:50 UTC
This still affects release version 2.4 and Development version Beta 3.0

• Show a list of ten or twenty favourite fonts that the user can define. 
Comment 20 hwoarang 2008-05-16 14:05:50 UTC
@pesala:
do not change the Version field. This field is to inform which version the bug
was initially found. 
Thank you.
Comment 21 pniemayer 2008-05-16 16:07:01 UTC
@hwoarang: Well, then the Version should be 1.1 RC2, for which the bug was
initially reported in issue 18620.

But does it really matter? Nobody applied the existing simple fix during the
last 5 years, so I doubt the next years will see any change to that.
Comment 22 turacin 2008-07-24 19:30:26 UTC
In my experience, performance seems to be getting worse on Windows XP.  With ~ 
96 fonts:  In 2.4.0, a right-click immediately shows the context menu, while in 
3.0 Beta, a right-click takes a few ( > 5 ) seconds to show the context menu on 
the same machine.  I've never noticed this problem with any other previous 
betas.
Comment 23 Mathias_Bauer 2008-09-01 13:51:08 UTC
I accidently came across this issue and I agree that the patch in issue 18620
was handled badly. My apology for this, we should change that.

While thinking about it - I wonder whether it makes sense at all to have the
fonts in the context menu. But I promise - first we will apply the patch and
then I will discuss that. ;-)

Peter, please have a look on the patch of issue 18620, IMHO it makes sense to
apply it.
Comment 24 pb 2008-11-24 06:03:55 UTC
pb: patch integrated in cws fwk98.
Files changed:
/svtools/source/control/stdmenu.cxx 264194.
Comment 25 pb 2008-12-14 18:25:19 UTC
pb -> es: please verify. Now only 100 (maximum) fonts will displayed.
Comment 26 joerg.skottke 2008-12-17 12:28:05 UTC
Verified by code review. 
PB used a slightly different approach ( he just takes the min value - either
number of fonts or 100 instead of rewriting the entire loop)
Comment 27 eric.savary 2009-03-17 13:44:47 UTC
Ok in OOO310m5