From 30efbba1929b0bf0b2e06a62ec91498595d2a936 Mon Sep 17 00:00:00 2001 From: groditi Date: Sat, 11 Jul 2009 23:05:02 +0000 Subject: add some comments in case someone has a hissy fit about MVC blah blah --- lib/Reaction/UI/View.pm | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'lib') diff --git a/lib/Reaction/UI/View.pm b/lib/Reaction/UI/View.pm index f81929d..44ab16a 100644 --- a/lib/Reaction/UI/View.pm +++ b/lib/Reaction/UI/View.pm @@ -61,6 +61,18 @@ sub render_window { my ($widget, $args) = $self->render_viewport_args($root_vp); $widget->render(widget => $rctx, $args); }; + +# From 2007-07-11 @ #reaction (so don't blame me if you think this violates some stupid school of thought) +#15:09 <@groditi> mst: one quick thing though. do you remember what layout_args is? +#15:10 <@mst> isn't it crap to be passed to the top level widget render? +#15:11 <@groditi> that's what i thought.. +#15:11 <@groditi> but it's kind of a no-op. there's an attr but nothing uses it +#15:11 <@groditi> i cant figure out if the data you give it ends up going anywhere +#15:12 <@groditi> although it'd be cool if anything you put there ended up being part of %_ in layout/widget land +#15:12 <@groditi> which would solve so fucking many of my gripes with widget/layout +#15:14 <@mst> I thought that was what it was supposed to do +#15:14 <@mst> wire it up :) + sub render_viewport_args { my ($self, $vp) = @_; my $layout_set = $self->layout_set_for($vp); -- cgit v1.2.3-54-g00ecf