Adium

Opened 14 years ago

Closed 12 years ago

Last modified 11 years ago

#2997 closed defect (fixed)

Smooth operator visual error

Reported by: tommcquillen@gmail.com Owned by: nobody
Milestone: Adium 1.2 Component: Message View
Version: Severity: minor
Keywords: large message window smooth operator background Cc:
Patch Status: Accepted

Description

I'm using AdiumX version 0.88 running OSX version 10.4.4, using the smooth operator visual style. There appears to be some issues tiling the background image. See screenshot for explanation.

Attachments (2)

Picture-1.jpg (95.5 KB) - added by tommcquillen@… 14 years ago.
Screenshot showing bug
Smooth Operator Edit.AdiumMessageStyle.zip (129.3 KB) - added by Jordan 12 years ago.
Fixed for resolutions up to 2560x1600

Download all attachments as: .zip

Change History (16)

Changed 14 years ago by tommcquillen@…

Attachment: Picture-1.jpg added

Screenshot showing bug

comment:1 Changed 14 years ago by Derrick Brashear

yea i'm having this problem also. Even the ProtoScroll gives me this problem I'm wondering what's wrong with just this message style. All along it's fine until the recent update.

comment:2 Changed 14 years ago by Peter Hosey

I don't think it's tiling at all. I think those center images are a certain width, and normally cropped to the desired width.

comment:3 Changed 14 years ago by Chris Forsythe

Resolution: wontfix
Status: newclosed

Smooth Operator has been removed from 1.0

comment:4 Changed 14 years ago by Evan Schoenberg

But really should be put back.

comment:5 Changed 14 years ago by anonymous

throw in smooth op 2

comment:6 Changed 14 years ago by Derrick Brashear

aww... don't throw away smooth operator... it's such a coll thing.... especially the protoscroll that is based on the smooth operator

comment:7 Changed 14 years ago by Derrick Brashear

Resolution: wontfix
Status: closedreopened

I have found out why. When we drag our message box too big, then the white thing will appear. It is actually fine once u reduce the size of your message box. That white thing will be removed. Please don't throw away smooth operator. Hope it helps.

comment:8 Changed 13 years ago by Jordan

Component: NoneAdium UI
Keywords: large message window redraw fail added
Milestone: Needs dev review
Patch Status: None
Version: 0.881.0.2

Similar ticket #6767

comment:9 Changed 13 years ago by Jordan

Component: Adium UIMessage View
Milestone: Needs dev reviewGood idea for "later"
priority: normallowest
Severity: normalminor

This is actually just a problem with the smooth operator background images. Rather than having a left side and right side with the curved corners plus a tiling center, it's got large chunks for each section. When the actual width of the png file is exhausted, you no longer have message background, but instead chat window background.

This can be solved via adjusting the way the css handles the background drawing, or by making the PNG files for the chat background wider (the cheap way).

If you really love smooth operator, it should be an easy fix - otherwise you might try one of the alternatives on adiumxtras.com or wait for smooth operator 2 (supposedly in the works ;))

comment:10 Changed 13 years ago by Jordan

Keywords: smooth operator background added; redraw fail removed

Changed 12 years ago by Jordan

Fixed for resolutions up to 2560x1600

comment:11 Changed 12 years ago by Jordan

Milestone: Good idea for "later"Adium X 1.2
pending: 0

I just fixed this along with #6141. It can be included in 1.2 or later, but it would be good if someone else could verify I didn't create any major problems.

comment:12 Changed 12 years ago by Jordan

Patch Status: NoneInitially Included

Not actually a diff patch, but shouldn't be hard to apply... just write over the old one.

comment:13 Changed 12 years ago by Evan Schoenberg

Resolution: fixed
Status: reopenedclosed

(In [21768]) Smooth Operator fix from jas8522 for the background bubble breakage which occurred when the window width was large. I pngcrushed the image files. Fixes #2997

comment:14 Changed 11 years ago by Robert

Patch Status: Initially IncludedAccepted
Note: See TracTickets for help on using tickets.