[ViewVC] Diff of: cvs/deliantra/Deliantra-Client/DC/Item.pm

Comparing deliantra/Deliantra-Client/DC/Item.pm (file contents):
Revision 1.10 by root, Wed Dec 26 20:46:39 2007 UTC vs.
Revision 1.14 by root, Sun Jan 11 23:45:52 2009 UTC

-package dc::Item;
+package DC::Item;
 use strict;
 use utf8;
 use Encode;
 }
 sub do_n_dialog {
    my ($cb) = @_;
-   my $w = new dc::UI::Toplevel
+   my $w = new DC::UI::Toplevel
       on_delete => sub { $_[0]->destroy; 1 },
       has_close_button => 1,
    ;
-   $w->add (my $vb = new dc::UI::VBox x => "center", y => "center");
+   $w->add (my $vb = new DC::UI::VBox x => "center", y => "center");
-   $vb->add (new dc::UI::Label text => "Enter item count:");
+   $vb->add (new DC::UI::Label text => "Enter item count:");
-   $vb->add (my $entry = new dc::UI::Entry
+   $vb->add (my $entry = new DC::UI::Entry
       text => $last_enter_count,
       on_activate => sub {
          my ($entry) = @_;
          $last_enter_count = $entry->get_text;
          $cb->($last_enter_count);
    );
    $entry->grab_focus;
    $w->show;
 }
+my $bg_cursed = [1  , 0  , 0, 0.5];
+my $bg_magic  = [0.2, 0.2, 1, 0.5];
 sub update_widgets {
    my ($self) = @_;
    # necessary to avoid cyclic references
-   dc::weaken $self;
+   DC::weaken $self;
    my $button_cb = sub {
       my (undef, $ev, $x, $y) = @_;
       my $targ = $::CONN->{player}{tag};
       if ($self->{container} == $::CONN->{player}{tag}) {
          $targ = $::CONN->{open_container};
       }
-      if (($ev->{mod} & dc::KMOD_SHIFT) && $ev->{button} == 1) {
+      if (($ev->{mod} & DC::KMOD_SHIFT) && $ev->{button} == 1) {
          $::CONN->send ("move $targ $self->{tag} 0")
             if $targ || !($self->{flags} & F_LOCKED);
-      } elsif (($ev->{mod} & dc::KMOD_SHIFT) && $ev->{button} == 2) {
+      } elsif (($ev->{mod} & DC::KMOD_SHIFT) && $ev->{button} == 2) {
          $self->{flags} & F_LOCKED
             ? $::CONN->send ("lock " . pack "CN", 0, $self->{tag})
             : $::CONN->send ("lock " . pack "CN", 1, $self->{tag})
       } elsif ($ev->{button} == 1) {
          $::CONN->send ("examine $self->{tag}");
          my $move_prefix = $::CONN->{open_container} ? 'put' : 'drop';
          if ($self->{container} == $::CONN->{open_container}) {
             $move_prefix = "take";
          }
-         my $shortname = dc::shorten $self->{name}, 14;
+         my $shortname = DC::shorten $self->{name}, 14;
          my @menu_items = (
             ["examine", sub { $::CONN->send ("examine $self->{tag}") }],
             ["mark",    sub { $::CONN->send ("mark ". pack "N", $self->{tag}) }],
             ["ignite/thaw",  # first try of an easier use of flint&steel
                         do_n_dialog (sub { $::CONN->send ("move $targ $self->{tag} $_[0]") })
                      }
                   ]
                )
             ),
-            ["bind <i>apply $shortname</i> to a key"   => sub { dc::Macro::quick_macro ["apply $self->{name}"] }],
+            ["bind <i>apply $shortname</i> to a key"   => sub { DC::Macro::quick_macro ["apply $self->{name}"] }],
          );
-         dc::UI::Menu->new (items => \@menu_items)->popup ($ev);
+         DC::UI::Menu->new (items => \@menu_items)->popup ($ev);
       }
       1
    };
       . "Middle click - apply\n"
       . "Shift-Middle click - lock/unlock\n"
       . "Right click - further options"
       . "</small>\n";
-   my $bg = $self->{flags} & F_CURSED ? [1  , 0  , 0, 0.5]
+   my $bg = $self->{flags} & F_CURSED ? $bg_cursed
-          : $self->{flags} & F_MAGIC  ? [0.2, 0.2, 1, 0.5]
+          : $self->{flags} & F_MAGIC  ? $bg_magic
           : undef;
-   my $desc = dc::Item::desc_string $self;
+   my $desc = DC::Item::desc_string $self;
+   my $face_tooltip = "<b>$desc</b>\n\n$tooltip_std";
+   if (my $face = $self->{face_widget}) {
+      if ($face->{bg} != $bg) {
+         $face->{bg} = $bg;
+         $face->update;
+      }
+      $face->set_face      ($self->{face})      if $face->{face}      != $self->{face};
+      $face->set_anim      ($self->{anim})      if $face->{anim}      != $self->{anim};
+      $face->set_animspeed ($self->{animspeed}) if $face->{animspeed} != $self->{animspeed};
+      #$face->set_tooltip (
+      #   "<b>Face/Animation.</b>\n"
+      # . "Item uses face #$self->{face}. "
+      # . ($self->{animspeed} ? "Item uses animation #$self->{anim} at " . (1 / $self->{animspeed}) . "fps. " : "Item is not animated. ")
+      # . "\n\n$tooltip_std"
+      #);
+      $face->set_tooltip ($face_tooltip);
+   } else {
-   $self->{face_widget} ||= new dc::UI::Face
+      $self->{face_widget} = new DC::UI::Face
-      can_events => 1,
+         can_events => 1,
-      can_hover  => 1,
+         can_hover  => 1,
+         bg         => $bg,
+         face       => $self->{face},
-      anim       => $self->{anim},
+         anim       => $self->{anim},
-      animspeed  => $self->{animspeed}, # TODO# must be set at creation time
+         animspeed  => $self->{animspeed}, # TODO# must be set at creation time
+         tooltip    => $face_tooltip,
-      on_button_down => $button_cb,
+         on_button_down => $button_cb,
+      ;
-   ;
+   }
-   $self->{face_widget}{bg}        = $bg;
-   $self->{face_widget}{face}      = $self->{face};
-   $self->{face_widget}{anim}      = $self->{anim};
-   $self->{face_widget}{animspeed} = $self->{animspeed};
-#   $self->{face_widget}->set_tooltip (
-#      "<b>Face/Animation.</b>\n"
-#    . "Item uses face #$self->{face}. "
-#    . ($self->{animspeed} ? "Item uses animation #$self->{anim} at " . (1 / $self->{animspeed}) . "fps. " : "Item is not animated. ")
-#    . "\n\n$tooltip_std"
-#   );
-   $self->{face_widget}->set_tooltip ("<b>$desc</b>\n\n$tooltip_std");
-   $self->{desc_widget} ||= new dc::UI::Label
+   $self->{desc_widget} ||= new DC::UI::Label
       can_events => 1,
       can_hover  => 1,
       ellipsise  => 2,
-      align      => -1,
+      align      => 0,
       on_button_down  => $button_cb,
       on_tooltip_show => sub {
          my ($widget) = @_;
          $::CONN && $::CONN->ex ($self->{tag}, sub {
             my ($long_desc) = @_;
-            $long_desc =~ s/\s+$//;
+            $long_desc = DC::Protocol::sanitise_xml ($long_desc);
             $self->{long_desc} = $long_desc;
             $widget->set_tooltip ("<b>$long_desc</b>\n\n$tooltip_std");
          });
       },
    $self->{desc_widget}->set_text ($desc);
    my $long_desc = $self->{long_desc} || $desc;
    $self->{desc_widget}->set_tooltip ("<b>$long_desc</b>\n\n$tooltip_std");
-   $self->{weight_widget} ||= new dc::UI::Label
+   $self->{weight_widget} ||= new DC::UI::Label
       can_events => 1,
       can_hover  => 1,
       ellipsise  => 0,
-      align      => 0,
       on_button_down => $button_cb,
    ;
    $self->{weight_widget}{bg} = $bg;
-   $self->{weight_widget}->set_text (dc::Item::weight_string $self);
+   $self->{weight_widget}->set_text (DC::Item::weight_string $self);
    $self->{weight_widget}->set_tooltip (
       "<b>Weight</b>.\n"
     . ($self->{weight} >= 0 ? "One item weighs $self->{weight}g. " : "You have no idea how much this weighs. ")
     . ($self->{nrof} ? "You have $self->{nrof} of it. " : "Item cannot stack with others of it's kind. ")
     . "\n\n$tooltip_std"

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/Deliantra-Client/DC/Item.pm (file contents): Revision 1.10 by root, Wed Dec 26 20:46:39 2007 UTC vs. Revision 1.14 by root, Sun Jan 11 23:45:52 2009 UTC

Diff Legend

Comparing deliantra/Deliantra-Client/DC/Item.pm (file contents):
Revision 1.10 by root, Wed Dec 26 20:46:39 2007 UTC vs.
Revision 1.14 by root, Sun Jan 11 23:45:52 2009 UTC