Adium

Ticket #8966 (reopened defect)

Opened 4 months ago

Last modified 4 days ago

divider lines in mockie message style

Reported by: dmainzman Assigned to: jas8522
Priority: normal Milestone: Adium X 1.3
Component: Adium Core Version: 1.2.5
Severity: normal Keywords: divider lines mockie
Cc: Patch: Initially Included
Pending: 0

Description

In previous versions of Adium using the Mockie message appearance style, a faint grey divider line appeared between messages from the same person. Now in the 1.2.1 betas, I have noticed that the line is gone. Each message seems to blend into the next, as the background color for each user blends uniformly without the divider line. Under the Mockie style, since all messages in a row from each person are grouped under one instance of the screen name, the only way to tell what is a new message and what is just a line break is by looking at the time stamps, which not all users have enabled.

Attachments

mockieDividerFix.diff (4.1 kB) - added by jas8522 on 01/21/2008 08:21:20 PM.
This fixes the divider lines and should also fix a potential issue with text overlap and certain time formats, though some might complain about too much spacing on the right side :P
mockie_margin_fix.diff (0.6 kB) - added by jas8522 on 05/05/2008 06:52:10 PM.

Change History

01/19/2008 04:08:55 AM changed by smn987

Yes, I just upgraded to 1.2.1 as well, and this regression creates serious usability problems with the Mockie style. A fix would be MUCH appreciated!

01/21/2008 07:26:46 PM changed by jas8522

  • keywords changed from divider, lines, mockie to divider lines mockie.
  • owner changed from nobody to jas8522.
  • milestone set to Adium X 1.2.2.

I caused this when I removed paragraph tags from it and had to move things around to get it to display properly... I completely overlooked those dividers!

01/21/2008 07:26:52 PM changed by jas8522

  • status changed from new to assigned.

01/21/2008 08:21:20 PM changed by jas8522

  • attachment mockieDividerFix.diff added.

This fixes the divider lines and should also fix a potential issue with text overlap and certain time formats, though some might complain about too much spacing on the right side :P

01/22/2008 10:31:48 AM changed by jas8522

  • patch_status changed from None to Needs Dev Review.

01/22/2008 02:46:30 PM changed by evands

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

(In [22458]) Correct between-message lines and spacing in Mockie. Patch from jas8522. Fixes #8966

01/22/2008 02:46:56 PM changed by evands

(In [22459]) Merged [22458]: Correct between-message lines and spacing in Mockie. Patch from jas8522. Fixes #8966

01/24/2008 03:23:16 AM changed by smn987

Almost, but not quite, fixed. Notice that with this fix the divider line still doesn't go all the way across — it stops 40 pixels away from the right end. Seems the problem is that in Mockie's layout.css, the .text class's margin has been changed from zeros all around to "margin: 0 40px 0 0". Not sure why that right margin was added, but changing it back to zero fixes the problem. (I also see that for the .sender class, padding-right has been changed from zero to 30px; is there a reason for this? I'd think it should be changed back to zero as well, but I haven't really figured out what it does.)

Apologies for not just providing a convenient diff — I'm not really a developer and not even quite sure how to make a diff (especially because I'm not working with SVN, just changed a CSS file on my local machine); I just know how to occasionally muck around or tinker with Adium's internals on my computer sometimes when there's a feature that's important to me :-)

05/01/2008 02:59:14 AM changed by proton

  • status changed from closed to reopened.
  • resolution deleted.

05/01/2008 03:09:26 AM changed by smn987

Thanks for reopening this. I see that there's been another change to the css since my earlier comment — it looks like now the .text class uses the padding property rather than the margin property. In any case, the same basic issue remains. In pre-1.2.2 versions of Adium, messages sent and received using the Mockie style allowed the message text to flow around the timestamp. That is, the timestamp is in the upper-right corner of the message, but only one line high, so in a multi-line message the message text on lines below the timestamp would be able to make use of the whole width of the text area. But since the 1.2.2 changes, the style now includes a huge margin/padding on the right side, so when you have a very long multi-line message, there's a empty space running down the right side of the text area, beneath the timestamp.

Not only does this waste ugly and waste a huge amount of space, but it causes long messages to require many more lines than they did before, which detracts from usability. The message text and the timestamp are in different colors and easily distinguished, so I don't see any reason why we can't just have the text flow around the timestamp as it did in earlier versions, and jas8522 seems to indicate in an above comment that the change which causes this was meant to clean up the css rather than to address a usability issue, so my belief is that we can just reduce the size of this margin back to 2px or 4px like the rest, but someone more familiar with these changes than me might want to chime in n whether I'm overlooking something.

05/05/2008 06:52:10 PM changed by jas8522

  • attachment mockie_margin_fix.diff added.

05/05/2008 06:53:22 PM changed by jas8522

  • patch_status changed from Needs Dev Review to Initially Included.
  • version changed from 1.2.1b1 to 1.2.5.
  • milestone changed from Adium X 1.2.2 to Adium X 1.3.

Fix attached for that margin issue.