On 22 Sep 2008, at 6:57 PM, Adam R. Maxwell wrote:

>
> 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.
>


That's why I now set the max width of the column (the min width was  
already set to 16).

Christiaan


-------------------------------------------------------------------------
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

Reply via email to