| Subject: | Tickit::Widget::Frame->style($style) should be ->set_style($style) in docs |
Looks like there's a minor typo in the Tickit::Widget::Frame
documentation - the ->set_style method is documented as ->style, which
leads to some minor confusion when trying to change it dynamically after
construction =)
I added a die when using ->style as a mutator so I could track any stray
cases in my code, not sure if that part's worth keeping though.
cheers,
Tom
| Subject: | tickit-frame-docpatch.diff |
diff -uNPr Tickit-0.19-original/lib/Tickit/Widget/Frame.pm Tickit-0.19-S2yM7S/lib/Tickit/Widget/Frame.pm
--- Tickit-0.19-original/lib/Tickit/Widget/Frame.pm 2012-07-19 00:16:27.000000000 +0100
+++ Tickit-0.19-S2yM7S/lib/Tickit/Widget/Frame.pm 2012-08-12 11:11:06.191968520 +0100
@@ -145,10 +145,11 @@
sub style
{
my $self = shift;
+ die 'This is an accessor only, perhaps you meant ->set_style?' if @_;
return $self->{style};
}
-=head2 $frame->style( $style )
+=head2 $frame->set_style( $style )
Accessor for the C<style> property, which controls the way the actual frame is
drawn around the inner widget. Must be one of the following names: