Ticket #1134 (assigned defect)

Opened 9 months ago

Last modified 5 months ago

Use correct source list style on Leopard

Reported by: eschaton Assigned to: timothy (accepted)
Priority: highest Component: Colloquy (GUI)
Version: Latest Nightly Severity: normal
Keywords: Cc:

Description

Colloquy draws its own light-blue source list at the left of the window. On Leopard, it should just set the highlight style of the outline view to source list, which will change both the way the selection highlight is drawn and the way the background is drawn to match the operating system's standard source list style.

Attachments

Colloquy-1134.patch (5.2 kB) - added by eschaton on 01/29/08 22:54:52.
Colloquy-1134-take2.patch (3.1 kB) - added by eschaton on 02/05/08 00:20:38.
A better version of Colloquy-1134.patch

Change History

01/29/08 22:39:51 changed by eschaton

This is very straightforward: JVSideOutlineView should just invoke super's methods instead of its custom drawing code when running on an OS later than 10.4 and instances of super's class respond to the method's selector. JVChatWindowController will also need to use -[NSTableView setTableHighlightStyle:] to make the chats outline use source list style when running on an OS later than 10.4.

I've tested this out building against the 10.5 SDK, but I suspect Colloquy wouldn't want to just take that change and require everyone building it to use Xcode 3.0 or later at this point. (Though let me know if you would!)

01/29/08 22:54:52 changed by eschaton

  • attachment Colloquy-1134.patch added.

01/29/08 22:56:45 changed by eschaton

The attachment Colloquy-1134.patch will have Colloquy use Leopard source lists when run on Leopard, but not change which SDKs it builds against or its behavior on pre-Leopard operating systems.

02/01/08 21:28:48 changed by timothy

  • status changed from new to assigned.

This patch looks good. I am going to apply it and test it out before landing it.

02/05/08 00:20:09 changed by eschaton

I'll attached a better patch now. It minimizes the diffs, doesn't have a bogus -instancesRespondToSelector:, and doesn't use a magic constant (it just declares the constant that you would get if you were building with the 10.5 SDK). It'll be in Colloquy-1134-take2.patch.

02/05/08 00:20:38 changed by eschaton

  • attachment Colloquy-1134-take2.patch added.

A better version of Colloquy-1134.patch

02/05/08 02:06:27 changed by timothy

This is a much cleaner patch.

02/06/08 09:07:13 changed by timothy

I landed the second patch with some minor code style changes in [3777].

There are some spacing issues when using the new source list mode that I will look into fixing now.

  • The member icons are too far to the left, and the text does not line up with the room name like it use too.
  • The members are closer together vertically. They need a couple of pixels of space or something.