--- trunk/lib/Frey/Web.pm 2009/06/29 16:46:03 1108 +++ trunk/lib/Frey/Web.pm 2009/06/29 16:54:02 1109 @@ -2,6 +2,7 @@ use Moose::Role; with 'Frey::Session'; +with 'Frey::Class::Icon'; use Data::Dump qw/dump/; use Carp qw/confess cluck carp/; @@ -262,8 +263,6 @@ our @status; sub status { @status }; -our $icon_html; - sub html_page { my $self = shift; my $a = {@_}; @@ -314,7 +313,7 @@ my $revision = $svk->info->{Revision} || ''; $revision = $1 if $info->{'Mirrored From'} =~ m{Rev\.\s+(\d+)}; - $self->add_icon unless $icon_html; + $self->add_icon; my $title = undef || $a->{title} @@ -335,7 +334,7 @@ $self->_head_html, qq|$title|, '', - ( $icon_html || '' ), + ( $self->icon_html ), ( $a->{head} || '' ), qq| @@ -486,13 +485,13 @@ sub clean_status { my ($self) = shift; + warn "## clean_status"; @head = ( 'static/frey.css' ); @status = ( { 'ClassBrowser' => Frey::Class::Browser->new( usage_sort => 1, usage_on_top => 0 )->as_markup }, { 'Bookmarklets' => Frey::Bookmarklet->new->as_markup }, - { 'INC' => Frey::INC->new->as_markup }, + { 'INC' => Frey::INC->new->as_markup }, ); - $icon_html = ''; } =head2 status_parts @@ -516,68 +515,6 @@ =cut -=head2 add_icon - - Frey::Foo->add_icon; # /static/icons/Frey/Foo.png - Frey::Foo->add_icon('warning'); # /static/icons/Frey/Foo/warning.png - -=cut - -sub icon_path { - my ($self,$class,$variant) = @_; - - sub icon_exists { - my $class = shift; - $class =~ s{::}{/}g; - $class .= "/$variant" if $variant; - my $icon_path = 'static/icons/' . $class . '.png'; - return $icon_path if -e $icon_path; - return; - } - - my $path = icon_exists( $class ); - if ( ! $path ) { - my $super_class = $class; - while ( $super_class =~ s{::[^:]+$}{} && ! $path ) { - $path = icon_exists( $super_class ) unless $super_class eq 'Frey'; # don't default on Frey icon - } - } - - if ( ! $path ) { - $self->TODO( "add icon for $class" . ( $variant ? " variant $variant" : '' ) ); - return undef; - } - - warn "# $class from $self icon_path $path" if $self->debug; - return $path; -} - -sub add_icon { - my ($self,$variant) = @_; - - my $class = $self->class if $self->can('class'); - #$class ||= $self->title; - $class ||= ref($self); - my $icon_path = $self->icon_path( $class, $variant ) || return; - - $icon_html .= qq||; - warn "# using icon $icon_path"; - -=for later - - # FIXME http://en.wikipedia.org/wiki/Favicon suggest just rel="icon" but that doesn't seem to work! - my $ico_path = $icon_path; - $ico_path =~ s{png$}{ico}; - if ( ! -e $ico_path ) { - system "convert $icon_path $ico_path"; - warn "# convert $icon_path $ico_path : $@"; - } - $icon_html .= qq|| if -e $ico_path; - -=cut - -} - my $warn_colors = { '#' => '#444', '##' => '#888',