Adium

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#9967 closed enhancement (fixed)

Make "About Adium" window scrollable

Reported by: David Munch Owned by: nobody
Milestone: Adium 1.4 Component: Adium UI
Version: 1.2.5 Severity: minor
Keywords: scroll about window moving text Cc:
Patch Status: Accepted

Description

Currently when you scroll in the "About Adium" window, the text speeds up for just a split second.

I would like to be able to either:

  • Scroll the text so the text moves faster.
  • Hold down the mouse-button in the window so the text moves faster.

Attachments (3)

9967.diff (1.2 KB) - added by SimDan 11 years ago.
Patch for clicking on duck to toggle between normal scroll speed and double speed
aboutBoxChanges.diff (13.3 KB) - added by wixardy 11 years ago.
patch to enable regular scrolling of the textview
AboutBox.nib.zip (18.3 KB) - added by wixardy 11 years ago.
nib for my patch

Download all attachments as: .zip

Change History (11)

comment:1 Changed 11 years ago by David Munch

On a second notice, proton just told me that the 'split second speed up' is actually just a trick of the eye. But the rest of the ticket still applies.

comment:2 Changed 11 years ago by jas8522

  • Milestone set to Adium X 1.4
  • priority changed from normal to lowest
  • Severity changed from normal to trivial

comment:3 Changed 11 years ago by Robby

  • Milestone changed from Adium X 1.4 to Adium X 1.3.x

Changed 11 years ago by SimDan

Patch for clicking on duck to toggle between normal scroll speed and double speed

comment:4 Changed 11 years ago by Robby

  • Milestone changed from Adium 1.3.x to Good idea for "later"
  • Patch Status changed from None to Needs Dev Review

comment:5 Changed 11 years ago by zacw

  • Patch Status changed from Needs Dev Review to Rejected

I don't think this is an appropriate patch to include; it would be confusing since nobody would know to do it anyway.

Changed 11 years ago by wixardy

patch to enable regular scrolling of the textview

Changed 11 years ago by wixardy

nib for my patch

comment:6 Changed 11 years ago by wixardy

I broke the autoscrolling out into a subclass of NSTextView that waits for a mouse scroll event. When the user scrolls on the textView the vertical scrollbar is enabled and the autoscroll stops.

Since the option key was a hidden key anyways I reassigned it to toggle autoscroll.

comment:7 Changed 11 years ago by zacw

  • Resolution set to fixed
  • Status changed from new to closed

(In [25554]) Patch from wixardy to make the about box scrollable. Fixes #9967.

comment:8 Changed 11 years ago by Robby

  • Milestone changed from Good idea for "later" to Adium 1.4
  • Patch Status changed from Rejected to Accepted

Sweet, thanks, wixardy! :)

Note: See TracTickets for help on using tickets.