ActionBar: Standard-Aktions-Button optisch hervorheben

[mfinanz.git] / SL / Layout / ActionBar / Action.pm

diff --git a/SL/Layout/ActionBar/Action.pm b/SL/Layout/ActionBar/Action.pm
index 8ac1d2d63fa1d4508aa394935ed313273d8a3750..4243b652ba7e79c8e5b789396258ca7b5cdb4b4c 100644 (file)
--- a/SL/Layout/ActionBar/Action.pm
+++ b/SL/Layout/ActionBar/Action.pm
@@ -19,46 +19,33 @@ sub script {
   sprintf q|$('#%s').data('action', %s);|, $_[0]->id, JSON->new->allow_blessed->convert_blessed->encode($_[0]->params);
 }
 
   sprintf q|$('#%s').data('action', %s);|, $_[0]->id, JSON->new->allow_blessed->convert_blessed->encode($_[0]->params);
 }
 

-# static constructors
-
-sub from_descriptor {
-  my ($class, $descriptor) = @_;
-  require SL::Layout::ActionBar::Separator;
-
-  {
-     separator => SL::Layout::ActionBar::Separator->new,
-  } or die 'unknown descriptor';
-}
-
-# TODO: this necessary?
-sub simple {
+# this is mostly so that outside consumer don't need to load subclasses themselves
+sub from_params {

   my ($class, $data) = @_;
 
   my ($class, $data) = @_;
 

-  my ($text, %params) = @$data;
-
-  if ($params{submit}) {
-    require SL::Layout::ActionBar::Submit;
-    return SL::Layout::ActionBar::Submit->new(text => $text, params => \%params);
-  }
-
-  if ($params{function}) {
-    require SL::Layout::ActionBar::ScriptButton;
-    return SL::Layout::ActionBar::ScriptButton->new(text => $text, params => \%params);
-  }
+  require SL::Layout::ActionBar::Submit;

 
 

-  if ($params{combobox}) {
-
-  }
+  my ($text, %params) = @$data;
+  return if exists($params{only_if}) && !$params{only_if};
+  return if exists($params{not_if})  &&  $params{not_if};
+  return SL::Layout::ActionBar::Submit->new(text => $text, params => \%params);

 }
 
 }
 

+sub callable { 0 }
+

 # shortcut for presenter
 
 sub p {
   SL::Presenter->get
 }
 
 # shortcut for presenter
 
 sub p {
   SL::Presenter->get
 }
 

+sub init_params {
+  +{}
+}
+

 # unique id to tie div and javascript together
 sub init_id {
 # unique id to tie div and javascript together
 sub init_id {

+  $_[0]->params->{id} //

   $_[0]->p->name_to_id('action[]')
 }
 
   $_[0]->p->name_to_id('action[]')
 }
 


@@ -83,9 +70,8 @@ on click submit the form specified by form-selector with the additional params
 
 on click call the specified function (is this a special case of checks?)
 
 
 on click call the specified function (is this a special case of checks?)
 

-- disabled => true/false (done)
+- disabled => true/false/tooltip explaning why disabled (done)

 
 TODO:
 
 - runtime disable/enable
 
 TODO:
 
 - runtime disable/enable

-