diff options
author | wreis <wreis@03d0b0b2-0e1a-0410-a411-fdb2f4bd65d7> | 2009-07-27 18:34:34 +0000 |
---|---|---|
committer | wreis <wreis@03d0b0b2-0e1a-0410-a411-fdb2f4bd65d7> | 2009-07-27 18:34:34 +0000 |
commit | d738d11fb21607bb66617ac95a455be8c9fa9e62 (patch) | |
tree | b0babb8ea470fe91ba181a2a8c414a50505b43fd /lib/Reaction | |
parent | 7ece76fce1b940dd3ba81166254d971d8ff1d991 (diff) | |
download | reaction-d738d11fb21607bb66617ac95a455be8c9fa9e62.tar.gz reaction-d738d11fb21607bb66617ac95a455be8c9fa9e62.zip |
Object consumes Action and Collection consumes Pager and Action, so Member::WithActions go away
Diffstat (limited to 'lib/Reaction')
-rw-r--r-- | lib/Reaction/Manual/ActionPrototypes.pod | 2 | ||||
-rw-r--r-- | lib/Reaction/UI/ViewPort/Collection.pm | 6 | ||||
-rw-r--r-- | lib/Reaction/UI/ViewPort/Collection/Grid.pm | 4 | ||||
-rw-r--r-- | lib/Reaction/UI/ViewPort/Collection/Grid/Member/WithActions.pm | 13 | ||||
-rw-r--r-- | lib/Reaction/UI/ViewPort/ListView.pm | 3 | ||||
-rw-r--r-- | lib/Reaction/UI/ViewPort/Object.pm | 2 | ||||
-rw-r--r-- | lib/Reaction/UI/Widget/Collection/Grid/Member.pm | 2 | ||||
-rw-r--r-- | lib/Reaction/UI/Widget/Collection/Grid/Member/WithActions.pm | 105 | ||||
-rw-r--r-- | lib/Reaction/UI/Widget/Object.pm | 18 |
9 files changed, 28 insertions, 127 deletions
diff --git a/lib/Reaction/Manual/ActionPrototypes.pod b/lib/Reaction/Manual/ActionPrototypes.pod index abf3a2d..388dc4c 100644 --- a/lib/Reaction/Manual/ActionPrototypes.pod +++ b/lib/Reaction/Manual/ActionPrototypes.pod @@ -42,7 +42,7 @@ By default, enable C<create>, C<update>, C<delete>, C<delete_all>, actions. know how many actions to expect to lay out the UI properly. =item Default to member-class -L<Grid::Member::WithActions|Reaction::UI::ViewPort::Collection::Grid::Member::WithActions> +L<Grid::Member|Reaction::UI::ViewPort::Collection::Grid::Member> =back diff --git a/lib/Reaction/UI/ViewPort/Collection.pm b/lib/Reaction/UI/ViewPort/Collection.pm index a3cb0af..d64ab42 100644 --- a/lib/Reaction/UI/ViewPort/Collection.pm +++ b/lib/Reaction/UI/ViewPort/Collection.pm @@ -10,6 +10,9 @@ use MooseX::Types::Moose qw/Str HashRef/; use namespace::clean -except => [ qw(meta) ]; extends 'Reaction::UI::ViewPort'; +with 'Reaction::UI::ViewPort::Collection::Role::Pager'; +with 'Reaction::UI::ViewPort::Role::Actions'; + has members => (is => 'rw', isa => 'ArrayRef', lazy_build => 1); has collection => (is => 'ro', isa => IM_Collection, required => 1); @@ -120,8 +123,7 @@ constructor as items are instantiated. The class to use when instantiating items to represent the member items. See: L<Object|Reaction::UI::ViewPort::Object>, -L<Member|Reaction::UI::ViewPort::Grid::Member>, -L<Member::WithActions|Reaction::UI::ViewPort::Grid::Member::WithActions>, +L<Member|Reaction::UI::ViewPort::Collection::Grid::Member>. =head1 INTERNAL METHODS diff --git a/lib/Reaction/UI/ViewPort/Collection/Grid.pm b/lib/Reaction/UI/ViewPort/Collection/Grid.pm index 1cf3fd9..eb10d3d 100644 --- a/lib/Reaction/UI/ViewPort/Collection/Grid.pm +++ b/lib/Reaction/UI/ViewPort/Collection/Grid.pm @@ -3,7 +3,7 @@ package Reaction::UI::ViewPort::Collection::Grid; use Reaction::Class; use aliased 'Reaction::InterfaceModel::Collection' => 'IM_Collection'; -use aliased 'Reaction::UI::ViewPort::Collection::Grid::Member::WithActions'; +use aliased 'Reaction::UI::ViewPort::Collection::Grid::Member'; use namespace::clean -except => [ qw(meta) ]; use MooseX::Types::Moose qw/ArrayRef HashRef Int/; @@ -44,7 +44,7 @@ has member_action_count => ( ); #################################### -sub _build_member_class { WithActions }; +sub _build_member_class { Member }; sub _build_field_labels { my $self = shift; diff --git a/lib/Reaction/UI/ViewPort/Collection/Grid/Member/WithActions.pm b/lib/Reaction/UI/ViewPort/Collection/Grid/Member/WithActions.pm deleted file mode 100644 index ea3d832..0000000 --- a/lib/Reaction/UI/ViewPort/Collection/Grid/Member/WithActions.pm +++ /dev/null @@ -1,13 +0,0 @@ -package Reaction::UI::ViewPort::Collection::Grid::Member::WithActions; - -use Reaction::Class; - -use namespace::clean -except => [ qw(meta) ]; -extends 'Reaction::UI::ViewPort::Collection::Grid::Member'; - -with 'Reaction::UI::ViewPort::Role::Actions'; - -__PACKAGE__->meta->make_immutable; - - -1; diff --git a/lib/Reaction/UI/ViewPort/ListView.pm b/lib/Reaction/UI/ViewPort/ListView.pm index d1778bd..d2574bb 100644 --- a/lib/Reaction/UI/ViewPort/ListView.pm +++ b/lib/Reaction/UI/ViewPort/ListView.pm @@ -5,10 +5,7 @@ use namespace::clean -except => [ qw(meta) ]; extends 'Reaction::UI::ViewPort::Collection::Grid'; with 'Reaction::UI::ViewPort::Collection::Role::Order'; -with 'Reaction::UI::ViewPort::Collection::Role::Pager'; -with 'Reaction::UI::ViewPort::Role::Actions'; __PACKAGE__->meta->make_immutable; - 1; diff --git a/lib/Reaction/UI/ViewPort/Object.pm b/lib/Reaction/UI/ViewPort/Object.pm index 9a8738e..37dfdc5 100644 --- a/lib/Reaction/UI/ViewPort/Object.pm +++ b/lib/Reaction/UI/ViewPort/Object.pm @@ -21,6 +21,8 @@ use MooseX::Types::Moose qw/ArrayRef HashRef/; use namespace::clean -except => [ qw(meta) ]; extends 'Reaction::UI::ViewPort'; +with 'Reaction::UI::ViewPort::Role::Actions'; + #everything is read only right now. Later I can make somethings read-write #but first I need to figure out what depends on what so we can have decent triggers has model => (is => 'ro', isa => IM_Object, required => 1); diff --git a/lib/Reaction/UI/Widget/Collection/Grid/Member.pm b/lib/Reaction/UI/Widget/Collection/Grid/Member.pm index 24168a6..3a47aa0 100644 --- a/lib/Reaction/UI/Widget/Collection/Grid/Member.pm +++ b/lib/Reaction/UI/Widget/Collection/Grid/Member.pm @@ -73,7 +73,7 @@ Renders the next C<field> fragment surrounded by a C<td> element. =item * L<Reaction::UI::Widget::Collection::Grid> -=item * L<Reaction::UI::Widget::Collection::Grid::Member::WithActions> +=item * L<Reaction::UI::Widget::Collection::Grid::Member> =back diff --git a/lib/Reaction/UI/Widget/Collection/Grid/Member/WithActions.pm b/lib/Reaction/UI/Widget/Collection/Grid/Member/WithActions.pm deleted file mode 100644 index f9ed586..0000000 --- a/lib/Reaction/UI/Widget/Collection/Grid/Member/WithActions.pm +++ /dev/null @@ -1,105 +0,0 @@ -package Reaction::UI::Widget::Collection::Grid::Member::WithActions; - -use Reaction::UI::WidgetClass; - -use namespace::clean -except => [ qw(meta) ]; -extends 'Reaction::UI::Widget::Collection::Grid::Member'; - -implements fragment actions { - render action => over $_{viewport}->actions; -}; - -implements fragment action { - render 'viewport'; -}; - -__PACKAGE__->meta->make_immutable; - -1; - -__END__; - -=head1 NAME - -Reaction::UI::Widget::Collection::Grid::Member::WithActions - Grid members with actions - -=head1 DESCRIPTION - -This is a subclass of L<Reaction::UI::Widget::Grid::Member> additionally -providing actions per member. - -=head1 FRAGMENTS - -=head2 actions - -Renders the C<action> fragment with every item in the viewports C<actions>. - -=head2 action - -Renders the C<viewport> fragment provided by L<Reaction::UI::Widget>, thus -rendering the current viewport stored in the C<_> topic argument provided -by the C<actions> fragment. - -=head1 LAYOUT SETS - -=head2 base - - share/skin/base/layout/collection/grid/member/with_actions.tt - -This layout set extends the C<collection/grid/member> layout set in the parent -skin. - -The following layouts are provided: - -=over 4 - -=item field_list - -First renders the original C<field_list> fragment, then the C<actions> fragment. - -=item action - -Simply renders the next C<action> fragment in line. - -=back - -=head2 default - - share/skin/default/layout/collection/grid/member/with_actions.tt - -This layout skin extends the C<collection/grid/member> layout set in the parent -skin. - -The following layouts are provided: - -=over 4 - -=item field_list - -The same as in the C<base> skin. - -=item action - -Renders the original C<action> fragment surrounded by a C<td> element. - -=back - -=head1 SEE ALSO - -=over 4 - -=item * L<Reaction::UI::Widget::Grid::Member> - -=item * L<Reaction::UI::Widget::Grid> - -=back - -=head1 AUTHORS - -See L<Reaction::Class> for authors. - -=head1 LICENSE - -See L<Reaction::Class> for the license. - -=cut diff --git a/lib/Reaction/UI/Widget/Object.pm b/lib/Reaction/UI/Widget/Object.pm index de83e25..8674e44 100644 --- a/lib/Reaction/UI/Widget/Object.pm +++ b/lib/Reaction/UI/Widget/Object.pm @@ -21,6 +21,14 @@ implements fragment field { render 'viewport'; }; +implements fragment actions { + render action => over $_{viewport}->actions; +}; + +implements fragment action { + render 'viewport'; +}; + __PACKAGE__->meta->make_immutable; 1; @@ -44,6 +52,16 @@ method return values. Renders the C<field> viewport passed by C<container_list>. +=head2 actions + +Renders the C<action> fragment with every item in the viewports C<actions>. + +=head2 action + +Renders the C<viewport> fragment provided by L<Reaction::UI::Widget>, thus +rendering the current viewport stored in the C<_> topic argument provided +by the C<actions> fragment. + =head1 DEPRECATED FRAGMENTS =head2 field_list |