ActionBar: Tooltips für deaktivierte Aktionen in ir.pl

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

diff --git a/SL/Layout/ActionBar/Action.pm b/SL/Layout/ActionBar/Action.pm
index 8b83295c5467bb10a35a878c9a5261b4076889cb..5d03b7c0569b18b7378b5f47440e87bf51dd3cfe 100644 (file)
--- a/SL/Layout/ActionBar/Action.pm
+++ b/SL/Layout/ActionBar/Action.pm
@@ -4,6 +4,7 @@ use strict;
 use parent qw(Rose::Object);
 
 use SL::Presenter;
 use parent qw(Rose::Object);
 
 use SL::Presenter;

+    require SL::Layout::ActionBar::Submit;

 
 use Rose::Object::MakeMethods::Generic (
   'scalar --get_set_init' => [ qw(id params text) ],
 
 use Rose::Object::MakeMethods::Generic (
   'scalar --get_set_init' => [ qw(id params text) ],


@@ -29,29 +30,15 @@ sub from_descriptor {
   return {
      separator => SL::Layout::ActionBar::Separator->new,
      combobox  => SL::Layout::ActionBar::ComboBox->new,
   return {
      separator => SL::Layout::ActionBar::Separator->new,
      combobox  => SL::Layout::ActionBar::ComboBox->new,

-  }->{$descriptor} or die 'unknown descriptor';
+  }->{$descriptor} || do { die 'unknown descriptor' };

 }
 
 }
 

-# TODO: this necessary?
+# this is mostly so that outside consumer don't need to load subclasses themselves

 sub simple {
   my ($class, $data) = @_;
 
   my ($text, %params) = @$data;
 sub simple {
   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);
-  }
-
-  if ($params{actions}) {
-    require SL::Layout::ActionBar::ComboBox;
-    return SL::Layout::ActionBar::ComboBox->new(text => $text, %params);
-  }
+  return SL::Layout::ActionBar::Submit->new(text => $text, params => \%params);

 }
 
 # shortcut for presenter
 }
 
 # shortcut for presenter


@@ -60,8 +47,13 @@ sub p {
   SL::Presenter->get
 }
 
   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[]')
 }
 


@@ -86,7 +78,7 @@ 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:
 
 
 TODO: