Fonts are very small with adium theme

small fonts

using same theme in gwibber, fonts are fine

Ken can you please look into this. the chat fonts look very small and changing the font size in theme's config file had no effect

I suspect this is an upstream empathy bug, perhaps empathy just doesn't handle over-riding the font sizes in webkit based on desktop settings. Gwibber does do that, but it isn't really theme related as far as I know.

I have open an upstream empathy task for this.

https://bugs.launchpad.net/ubuntu/+source/empathy/+bug/504771/comments/9 have a workaround

Patch from Jamie Strandboge

omer, regarding patches on Launchpad.
a. do we apply them ourselves?
b. they are available on the back port repository (e.g. empathy 2.30.x patched)?

Thanks for the patch, however I think it would be better to fix empathy to handle setting the font based on the system settings. Font choice and size is pretty subjective, so honoring the system setting for document font would be best.

To expand on that, I think the adium theme should be modified removing references to the font from main.css and empathy/webkit should use the system document font by default and allow the theme to override that in the css.

Ken, Can we get this into Lucid with SRU. the settings with this patch are much better than the font size we have already.

The problem is I think that is rather subjective, for me the current fonts look much better. Choosing the proper font size is going to vary based on screen size/resolution and individual opinion. Which is why I think this is best fixed in empathy to honor system preferences.

I've made a patch to fix the font size bug.

What it does it to remove the DefaultFontFamily and DefaultFontSize keys from Info.plist file, also removes the font-size:11px from the Contents/Resources/main.css at line 111.

So now the font displayed is the system font.

Not an empathy issue

I have build the adium theme with your patch (took 46hours to build though) and the chat text now depends on the system fonts. https://edge.launchpad.net/~om26er/+archive/test Now the other problem is that the text of the sender/receiver is still very small and does not change at all with font changes. Marking triaged for now we will address this issue for Maverick.

Based on Omer's last comment, I feel this patch is not quite ready yet, adding patch-needswork tag.

I think Jordi's patch is the only one that makes sense. Hard-coding *any* font family/size would only move the problem somewhere else (I can already picture people using low-resolution netbooks complaining about the font being too big).

here the the patched theme with Jordi's patch https://launchpad.net/~om26er/+archive/test/+build/1712973/+files/adium-theme-ubuntu_0.1-0ubuntu2_all.deb The chat fonts are perfect but the text names of the send/receiver and also the text of the timestamps is very small. But the thing that really matters is the chat fonts so Yeah the patch pretty much does the trick :)

Sorry for all the spamming. Here is a patch which also changes the size of timestamps and also the sender/receivers names to the system fonts.

Also made a .deb with the changes implemented. (launchpad builders take a lot of time these days).

Yes, this new adium theme package solved the problem here. (:

This bug was fixed in the package adium-theme-ubuntu - 0.2-0ubuntu1

---------------
adium-theme-ubuntu (0.2-0ubuntu1) maverick; urgency=low

  * New upstream release.
    - Don't hard code font sizes (Omer Akram) (LP: #531835)
 -- Ken VanDine <email address hidden> Wed, 09 Jun 2010 14:36:38 -0400

Can we get this backported to Lucid please?

AfC

Note that empathy uses the application font as fallback for adium themes, not the document font. There is an upstream bug for this with a patch: https://bugzilla.gnome.org/show_bug.cgi?id=622266

uploaded to -proposed:

empathy (2.30.2-0ubuntu1) lucid-proposed; urgency=low

  * New upstream release:
    - empathy-2.30.1.1: tests fail (Guillaume Desmottes)
    - crash when closing chat with still-open notification bubble
      (Guillaume Desmottes)
    - Can't connect chatroom if space after adress (Guillaume Desmottes)
    - Updated translations
  * debian/patches/32_append_notifications.patch:
    - refresh to new version
  * debian/patches/91_git_fix_small_font.patch:
    - take document font for adium theme to fix small fonts (LP: #531835)

test case:
- install the new version from -proposed
- test that you still can chat with your contacts on different networks
- check that font size is more readable than previous version

Accepted empathy into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

I can verify that with empathy 2.30.2 I see no regression and if the adium theme is patched to not use hard coded fonts then it uses document fonts.

This was taken from git, so it should be fixed in maverick.

This bug was fixed in the package empathy - 2.30.2-0ubuntu1

---------------
empathy (2.30.2-0ubuntu1) lucid-proposed; urgency=low

  * New upstream release: (LP: #599294)
    - empathy-2.30.1.1: tests fail (Guillaume Desmottes)
    - crash when closing chat with still-open notification bubble
      (Guillaume Desmottes)
    - Can't connect chatroom if space after adress (Guillaume Desmottes)
    - Updated translations
  * debian/patches/32_append_notifications.patch:
    - refresh to new version
  * debian/patches/91_git_fix_small_font.patch:
    - take document font for adium theme to fix small fonts (LP: #531835)
 -- Didier Roche <email address hidden> Mon, 28 Jun 2010 11:36:50 +0200

i am unsubscribing the MIR team ... please change title of bug to say what action is here for MIR team and drop a comment. Thanks!

This bug isn't worthy of any more SRU's and is fixed in current release. Invalidating lucid tasks not marked as fix released.