Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=701604
--- Comment #25 from David Juran <djuran(a)redhat.com> 2011-08-26 05:35:13 EDT ---
Package: evolution-3.0.2-3.fc15
Architecture: x86_64
OS Release: Fedora release 15 (Lovelock)
Comment
-----
Just tried to start evolution
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
Do not reply to this email. You can add comments to this bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=3512
Karl Tomlinson (:karlt) <karlt(a)mozbugz.karlt.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #555845|review?(karlt(a)mozbugz.karlt |review+
Flags|.net) |
--- Comment #71 from Karl Tomlinson (:karlt) <karlt(a)mozbugz.karlt.net> 2011-08-25 17:20:55 PDT ---
Comment on attachment 555845
--> https://bugzilla.mozilla.org/attachment.cgi?id=555845
patch to pass font-stretch to the linux/pango/fontconfig backend
>+ if (width <= (FC_WIDTH_ULTRACONDENSED + FC_WIDTH_EXTRACONDENSED) / 2) {
>+ return -4;
>+ case -4:
>+ return FC_WIDTH_ULTRACONDENSED;
Would it make sense to use NS_FONT_STRETCH_* for the constants here?
>@@ -444,16 +444,24 @@ gfxUserFcFontEntry::AdjustPatternToCSS(F
>+ int fontWidth = FC_WIDTH_NORMAL;
>+ FcPatternGetInteger(aPattern, FC_WIDTH, 0, &fontWidth);
>+ int cssWidth = gfxFontconfigUtils::FcWidthForThebesStretch(mStretch);
>+ if (cssWidth != fontWidth) {
>+ FcPatternDel(aPattern, FC_WIDTH);
>+ FcPatternAddInteger(aPattern, FC_WIDTH, cssWidth);
>+ }
>+
A missing FC_WIDTH is treated differently from a FC_WIDTH_NORMAL, so I think
its best to initialize fontWidth to something invalid like -1 (as for
FC_WEIGHT) above to ensure that FC_WIDTH is set.
--
Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
Do not reply to this email. You can add comments to this bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=3512
Jonathan Kew <jfkthame(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #555845| |review?(karlt(a)mozbugz.karlt
Flags| |.net)
--- Comment #70 from Jonathan Kew <jfkthame(a)gmail.com> 2011-08-25 14:20:01 PDT ---
Created attachment 555845
--> https://bugzilla.mozilla.org/attachment.cgi?id=555845&action=edit
patch to pass font-stretch to the linux/pango/fontconfig backend
This seems to work in initial testing, for both "installed" fonts and fonts
loaded via @font-face.
We should add a test for this, at least covering the @font-face case.
--
Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
Do not reply to this email. You can add comments to this bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=3512
--- Comment #69 from Jonathan Kew <jfkthame(a)gmail.com> 2011-08-25 09:15:18 PDT ---
This seems to work on Mac OS X (in my limited testing, at least), but the
Windows and Linux (pango/fontconfig) backends need some additional love; they
appear to ignore the stretch setting during font selection.
--
Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
Do not reply to this email. You can add comments to this bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=3512
Andreas Gal :gal <gal(a)uci.edu> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |gal(a)uci.edu
--- Comment #68 from Andreas Gal :gal <gal(a)uci.edu> 2011-08-25 08:33:07 PDT ---
Whats the status of this bug? pdf.js wants this.
--
Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.