Adium

Ticket #6349 (new enhancement)

Opened 1 year ago

Last modified 2 months ago

Redesign Fonts and Colors handling to be not confusing

Reported by: boredzo Assigned to: duboisj
Priority: low Milestone: Adium X 1.4
Component: Adium UI Version:
Severity: normal Keywords:
Cc: Patch: None
Pending: 0

Description

1.0's FontsAndColors UI (as described in version 3 of that document) is confusing. In particular, users are unable to figure out how to set their font for outgoing plain-text messages.

Devs and contributors: Attach mock-up screenshots here. Describe if necessary.

Attachments

AdiumTextInspector.png (28.6 kB) - added by boredzo on 02/08/2007 10:41:59 PM.
Peter's mock-up of a Text Inspector panel.
duboisj.ColorPrefsWindow.png (114.3 kB) - added by duboisj on 05/13/2008 10:33:55 AM.
Preferences pane allowing user to set custom colors

Change History

02/08/2007 10:41:59 PM changed by boredzo

  • attachment AdiumTextInspector.png added.

Peter's mock-up of a Text Inspector panel.

02/08/2007 10:49:53 PM changed by boredzo

How the text inspector works:

Setting the plain-text fonts

The plain-text font is used for the display of plain-text.

  1. Go to Preferences→Messages.
  2. Click “Configure Incoming Plain-Text Font” or “Configure Outgoing Plain-Text Font”.
  3. Operate the Text Inspector.

The Reset buttons are never enabled in this case.

Setting the current font of the selection, overidding the plain-text font

  1. Be in an inputline or status field.
  2. Choose “Text Inspector” from the Format menu.
  3. Operate the Text Inspector.

The Reset buttons are enabled when the relevant attribute is present in the selection. Thus, for example, the fgcolor Reset button is not enabled when the selection does not possess a fgcolor.

Advantages

  • It is possible to set the font and/or fgcolor and/or bgcolor to be the same as the plain-text font, but explicit rather than display-only.
  • Totally unambiguous.

Disadvantages

  • No UI precedent.
    • OTOH, there's also no UI precedent for having plain and rich text in the same field. IMO, we can do whatever we want here—the only test is what makes sense to the users.

Other points

  • Wording of “Foreground [color]”—could be “Text” instead.
  • Wording of “Configure (Incoming|Outgoing) Plain-Text Font” buttons is subject to change.
  • When no message or status field is in the responder change, the contents of the Text Inspector (which is non-resizable) change to “Nothing to Inspect”.

02/08/2007 10:51:03 PM changed by boredzo

Also, see CFM's mock-up on #1882.

03/21/2007 06:16:54 AM changed by kiel

  • patch_status set to None.

Part of me feels that #3234 is relevant/related to this ticket.

04/05/2007 10:21:59 PM changed by tick

  • milestone changed from Adium X 1.1 to Good idea for "later".

05/13/2008 02:51:46 AM changed by praseodym

The inspector mockup currently misuses radio buttons to set a context for the rest of the window; in my opinion it should be using tabs instead.

05/13/2008 10:33:55 AM changed by duboisj

  • attachment duboisj.ColorPrefsWindow.png added.

Preferences pane allowing user to set custom colors

05/13/2008 10:40:22 AM changed by duboisj

I added a new file - duboisj.ColorPrefsWindow.png. This is what I put together for allowing the user to customize colors. The labels on the UI could be improved to make it more clear what the colors are set for. The advantage to having the color picker here is that when the user attempts to set prefs from the prefs pane, they see right away what the changes will be in the preview.

This way, if the user sets a custom background (a custom image or an unusual color), it is easy to pick fonts for messages that will show up well.

Some functionality should be added, IMO, such as the ability to determine which color 'wins' if there is a message coming over the wire in something other than the default color. Maybe a discussion on the developer list will help determine the optimal behavior?

05/15/2008 08:38:08 AM changed by djmori

06/02/2008 12:40:16 PM changed by Robby

  • pending changed.
  • milestone changed from Good idea for "later" to Needs dev review.

06/09/2008 08:56:52 PM changed by jas8522

  • priority changed from normal to low.
  • version deleted.
  • type changed from defect to enhancement.
  • milestone changed from Needs dev review to Adium X 1.4.

There was something recently on adium-devl discussing this... did it bear any fruit?

06/10/2008 02:31:56 AM changed by duboisj

There wasn't a lot in the way of conversation, and I set it aside for a couple of weeks. I had it working for a while, then let the code rot for a week or two. Yesterday afternoon, after I saw discussion of the next beta release going out, I checked out the latest and started merging some changes into that. I won't try to check it into the next beta, but I will submit the patch after the beta, or the release, is out the door.

06/10/2008 07:46:23 AM changed by jas8522

  • owner changed from nobody to duboisj.

Excellent, we'll leave it at 1.4 then for now, if someone wishes to bring it forward then we can always change that later.