Ian Ward wrote on 15/07/11 09:32 PM :
> Yu-Jie Lin wrote on 2011-07-15 16:27:
>> If uncomment the Filler line and comment the second Filler, it will result:
>>
>>   File "/usr/lib64/python2.6/site-packages/urwid/decoration.py", line 547, in
>> padding_values
>>     width, ignore = self._original_widget.pack(focus=focus)
>> TypeError: pack() takes at least 2 non-keyword arguments (1 given)
> 
> Yes, those look like bugs.  I guess so few people have been playing with
> fixed widgets no-one noticed or complained until now.
> 
>> I am new to Urwid, still trying to learn how it works, but I believe that 
>> there
>> is a missing definition of pack() in AttrMap, which is done by the commented 
>> the
>> if clause above. If uncomment that part, the code runs as I expect. I think 
>> it's
>> just a simple missing for pack() since other methods seem to be called
>> correctly, <http://excess.org/urwid/browser/urwid/decoration.py#L207>. And if
>> this is the case, then WidgetWrap may have the same issue,
>> <http://excess.org/urwid/browser/urwid/widget.py#L1305>.

I've just pushed up a fix, give it a try.

_______________________________________________
Urwid mailing list
[email protected]
http://lists.excess.org/mailman/listinfo/urwid

Reply via email to