diff options
author | Yorhel <git@yorhel.nl> | 2009-07-09 19:44:03 +0200 |
---|---|---|
committer | Yorhel <git@yorhel.nl> | 2009-07-09 19:44:03 +0200 |
commit | a130a4f79e04f8a6ccd4514b9a1d0285c63edfea (patch) | |
tree | cf726b7a5d505fc2a38b3819e05aad2fa3557e02 /lib/VNDB/Util/CommonHTML.pm | |
parent | 4d7d792d2979158fe76937967eef2b2a1fe47fdb (diff) |
More intuitive ordering of the copy and posts tabs
Related #vndb discussion:
<ImmLff> Yorhel: some minor interface suggestions - in release entry
swap the order of "copy" and "edit" buttons, so edit would be near
"rxxx" button as it was earlier; move "posts" tab near the
"discussions" tab(probably between discussions and history), so
discussion related tabs would be near each other
<ImmLff> at least it looks logical to me and definitely won't harm
<Yorhel> oh, I haven't really put any thought into the order of the tabs
<Yorhel> so your suggestions sounds good :)
<Yorhel> although, I'll place the posts tab after wishlist, to be
consistent with the other DB entries
Diffstat (limited to 'lib/VNDB/Util/CommonHTML.pm')
-rw-r--r-- | lib/VNDB/Util/CommonHTML.pm | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/lib/VNDB/Util/CommonHTML.pm b/lib/VNDB/Util/CommonHTML.pm index a6aa1dd3..981edfe8 100644 --- a/lib/VNDB/Util/CommonHTML.pm +++ b/lib/VNDB/Util/CommonHTML.pm @@ -40,6 +40,12 @@ sub htmlMainTabs { end; } + if($type eq 'u') { + li $sel eq 'posts' ? (class => 'tabselected') : (); + a href => "/$id/posts", 'posts'; + end; + } + if($type eq 'u' && $obj->{show_list}) { li $sel eq 'wish' ? (class => 'tabselected') : (); a href => "/$id/wish", 'wishlist'; @@ -54,10 +60,6 @@ sub htmlMainTabs { li $sel eq 'tags' ? (class => 'tabselected') : (); a href => "/$id/tags", 'tags'; end; - - li $sel eq 'posts' ? (class => 'tabselected') : (); - a href => "/$id/posts", 'posts'; - end; } if($type eq 'v' && $self->authCan('tag') && !$obj->{hidden}) { @@ -66,6 +68,12 @@ sub htmlMainTabs { end; } + if($type eq 'r' && $self->authCan('edit')) { + li $sel eq 'copy' ? (class => 'tabselected') : (); + a href => "/$id/copy", 'copy'; + end; + } + if( $type eq 'u' && ($self->authInfo->{id} && $obj->{id} == $self->authInfo->{id} || $self->authCan('usermod')) || $type =~ /[vrp]/ && $self->authCan('edit') && (!$obj->{locked} || $self->authCan('lock')) && (!$obj->{hidden} || $self->authCan('del')) || $type eq 'g' && $self->authCan('tagmod') @@ -75,12 +83,6 @@ sub htmlMainTabs { end; } - if($type eq 'r' && $self->authCan('edit')) { - li $sel eq 'copy' ? (class => 'tabselected') : (); - a href => "/$id/copy", 'copy'; - end; - } - if($type =~ /[vrp]/ && $self->authCan('del')) { li; a href => "/$id/hide", $obj->{hidden} ? 'unhide' : 'hide'; |