Adium

Ticket #9524 (closed defect: fixed)

Opened 8 months ago

Last modified 8 months ago

Message font size changes with selection of "show received message fonts"

Reported by: mathuaerknedam Assigned to: nobody
Priority: lowest Milestone: Adium 1.2.5
Component: Adium UI Version: 1.2.4
Severity: trivial Keywords:
Cc: Patch: None
Pending: 0

Description (Last modified by jas8522)

While opening #9523, I decided that since the developers might disagree with my position I should separate this out so that the two issues (font and color) don't get confused.

Here's how to reproduce the problem I'm seeing:

  1. Go to the messages pane of the Adium preferences
  2. If the "show received message fonts" checkbox is selected, deselect it.
  3. Select the Stockholm messagestyle.
  4. Note the size of the message "Away for the sake of being away".
  5. Select "show received message fonts".
  6. Note the size of the message "Away for the sake of being away".

One could argue that "received message fonts" should include not only the font-family (typeface) but also the font-size, but surely "receives message colors" would not! For the record, I think selecting "received message fonts" should change only the font-family, and not the font size.

Change History

04/01/2008 10:30:11 AM changed by jas8522

  • severity changed from normal to trivial.
  • component changed from Adium Core to Adium UI.
  • summary changed from message size changes with selection of "show received message fonts" to Message font size changes with selection of "show received message fonts".
  • priority changed from normal to lowest.
  • version changed from 1.2.4b2 to 1.2.4.
  • milestone set to Adium X 1.2.5.
  • description changed.

I personally think that this should show the size as well. One of two things should happen here:

  1. Show received message fonts should change to "Show received message fonts and sizes", or
  2. It should not take on the received message size, and a third option should be added to "Show received message size"

I personally think it should be the first one... but it doesn't matter that much to me.

04/03/2008 11:21:13 AM changed by evands

  • status changed from new to closed.
  • resolution set to fixed.

(In [23034]) I see no reason to force default attributes for decoded text to Lucda Grande 12; if the incoming HTML doesn't specify a font or size, we should produce an NSAttributedString which similarly doesn't specify. Along with [23033], fixes #5561 and fixes #9524.

04/03/2008 11:22:06 AM changed by evands

(In [23035]) Merged [23033] and [23034]: I see no reason to force default attributes for decoded text to Lucda Grande 12; if the incoming HTML doesn't specify a font or size, we should produce an NSAttributedString which similarly doesn't specify. Along with [23033], fixes #5561 and fixes #9524.