Adium

Ticket #5561 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Messages prefpane preview will only display outgoing messages in Lucida Grande

Reported by: adium Owned by:
Milestone: Adium 1.2.5 Component: Message View
Version: 1.2.4 Severity: regression
Keywords: message view font Cc:
Patch Status:

Description

The outgoing font in message views appears to be stuck in Arial (or at least, it is for me...). I wrote the message style shown, so I know damn well it doesn't specify a font! This seems to occur whether or not there's a default font specified in Info.plist, too.

Attachments

Picture 4.png (138.3 KB) - added by adium 3 years ago.
Demonstation of the issue
Picture 11.png (99.0 KB) - added by mathuaerknedam 3 years ago.
Stockholm with Arail & Lucida Grande

Change History

Changed 3 years ago by adium

Demonstation of the issue

  Changed 3 years ago by evands

  • severity changed from normal to regression
  • milestone set to Adium X 1.0

in reply to: ↑ description   Changed 3 years ago by adium

Ok, so a correction: in the preview display, it's actually stuck in Lucida Grande; though I think I'm still right that this is no matter what font is set as default.

In chat windows, I've realised that it's using Arial as that's what my outgoing font is set to. I'd assumed that changing the display settings in preferences would also work for my font, but it seems, strangely, I'm wrong on that count.

  Changed 3 years ago by catfish_man

  • status changed from new to closed
  • resolution set to worksforme
  • milestone Adium X 1.0 deleted

Correct. You can change your font via OSX's formatting palette (command-t or the format menu). Making this less confusing is being discussed in #1882. If I'm misunderstanding the issue, please reopen :)

  Changed 3 years ago by evands

  • status changed from closed to reopened
  • resolution worksforme deleted

The 'correction' comment was just irrelevant to the originally reported issue :)

  Changed 3 years ago by jas8522

  • status changed from reopened to closed
  • patch_status set to None
  • resolution set to fixed

I believe this is now fixed.

  Changed 3 years ago by mathuaerknedam

I'm not sure when this might have regressed, but as of 1.2svn, The problem is back. Except that the outgoing message will only display as Lucida Grande. I'll attach a screenshot that shows that "Gotta run; catch ya later ;)" is a different font.

Changed 3 years ago by mathuaerknedam

Stockholm with Arail & Lucida Grande

  Changed 3 years ago by mathuaerknedam

Oh, and just so it's clear, this problem is only in the preview on messages pane of the prefs, not in actual messages.

  Changed 3 years ago by tedger

  • status changed from closed to reopened
  • version changed from 1.0b13 to 1.2svn
  • resolution fixed deleted
  • pending set to 0

Reoping as it has been reported that this has reappeared.

  Changed 3 years ago by mathuaerknedam

FWIW, the summary/title above specifies Arial, but the actual font that's "stuck" is Lucida Grande.

  Changed 3 years ago by zacw

  • summary changed from Outgoing font stuck in Arial to Preview Font for Message Styles Stuck on Lucida Grande
  • milestone set to Adium X 1.2

  Changed 2 years ago by tick

  • milestone changed from Adium X 1.2 to Adium X 1.3

Can anyone reproduce this?

Due to the work for the SoC stuff being on 1.2, I'm moving this to 1.3.

  Changed 2 years ago by hawkman

Yes. Under 1.1.2, the outgoing messages in the preview pane will not display the font chosen under Text Display:. The size, however, will be respected. Choose a completely wacky font to demonstrate this.

  Changed 2 years ago by jas8522

refs with screen capture #7766

  Changed 2 years ago by mathuaerknedam

FWIW, I'd summarize this as "messages prefpane preview will only display outgoing messages in Lucida Grande". This problem is limited to the prefpane preview; real chats are fine.

  Changed 2 years ago by jas8522

  • summary changed from Preview Font for Message Styles Stuck on Lucida Grande to Messages prefpane preview will only display outgoing messages in Lucida Grande

  Changed 2 years ago by evands

The preview chat specifies a particular font for the outgoing message but has no font specified for the incoming one. This is supposed to allow one to demonstrate in real time the effects of the "show received message fonts" checkbox.

The intended behavior is:

  • With 'show received message fonts' enabled, the outgoing message is in Lucida Grande and the incoming one is on the font specified in the pref pane.
  • With 'show received message fonts' disabled, both incoming and outgoing messages display in the font specified in the pref pane.

follow-up: ↓ 18   Changed 2 years ago by mathuaerknedam

Weird.

The current behavior is:

  • With 'show received message fonts' enabled, the outgoing message is in Lucida Grande and the incoming one is Comic Sans.
  • With 'show received message fonts' disabled, the outgoing message is in Lucida Grande and the incoming one is in the font specified in the pref pane.

in reply to: ↑ 17   Changed 2 years ago by evands

Oops, I got 'incoming' and outgoing' backwards in my post.

Replying to mathuaerknedam:

The current behavior is: * With 'show received message fonts' enabled, the outgoing message is in Lucida Grande and the incoming one is Comic Sans. * With 'show received message fonts' disabled, the outgoing message is in Lucida Grande and the incoming one is in the font specified in the pref pane.

The incoming message is behaving exactly as it intended. The outgoing message is misbehaving.

  Changed 2 years ago by jas8522

  • milestone changed from Adium X 1.3 to Adium X 1.2.5

Matching this up with #9524 and #9523 as they are all problems with the messages prefpane styles.

  Changed 2 years ago by jas8522

  • version changed from 1.2svn to 1.2.4

  Changed 2 years ago by evands

  • status changed from reopened 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.

  Changed 2 years ago 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.

Note: See TracTickets for help on using tickets.