On Monday, September 22, 2008, at 08:16AM, "Christiaan Hofman" <[EMAIL
PROTECTED]> wrote:
>
>On 22 Sep 2008, at 5:05 PM, Adam R. Maxwell wrote:
>
>>
>> On Sep 22, 2008, at 7:53 AM, Christiaan Hofman wrote:
>>
>>>
>>> On 22 Sep 2008, at 4:39 PM, Adam R. Maxwell wrote:
>>>
>>>>
>>>> On Sep 22, 2008, at 5:44 AM, Christiaan Hofman wrote:
>>>>
>>>>> Or just download the last nightly at <http://
>>>>> bibdesk.demokratia.org/
>>>>> beta> and use Autosize Column, because it now gives a more sensible
>>>>> default size.
>>>>
>>>> Out of curiosity, what did you do to the level indicator cell? That
>>>> looks like something that's begging for a comment in the code,
>>>> especially taken with the commit comment (100 pts doesn't seem like
>>>> an insane size to me).
>>>
>>> Just override cellSize, to return something with a maximum width of
>>> 100. NSLevelIndicatorCell returns {40000, 12}, which is rather big.
>>
>> It seems like a bit of an implementation detail if cellSize is used
>> to determine column width.
>
>I disagree, it's supposed to give the size the cell requires if it
>isn't constraint ("Returns the minimum size needed to display the
>receiver.")
But there is no minimum size required to display it, so why not return
NSZeroSize, which is just as meaningful as any other value? Anything calling
cellSize should be prepared to deal with large sizes, since that same
documentation says that it returns 10000,10000 for non image or text cells.
Any autosizing of table columns should be constrained by the max/min width of
the column, and sizing heuristics are out of place in the cell. However, that's
just my opinion...apologies to the list for pushing this off-topic.
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Bibdesk-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bibdesk-users
