From: csarven Date: Fri, 21 Nov 2008 02:38:27 +0000 (-0500) Subject: Tag markup and styling for tag_self, tag_user, form#tag_user X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=ca8f45fc6ea609450de75afc3438d9c43c42a7bb;p=quix0rs-gnu-social.git Tag markup and styling for tag_self, tag_user, form#tag_user darcs-hash:20081121023827-eefa4-ed91f63cac35c078dc385546a69209cbef2a0d5e.gz --- diff --git a/actions/showstream.php b/actions/showstream.php index 419c4421ad..b9ab699d3e 100644 --- a/actions/showstream.php +++ b/actions/showstream.php @@ -355,12 +355,17 @@ class ShowstreamAction extends StreamAction { common_element('dt', 'tags', _('Tags')); common_element_start('dd', 'tags'); $tags = Profile_tag::getTags($profile->id, $profile->id); + + common_element_start('ul', 'tags xoxo'); foreach ($tags as $tag) { - common_element('a', array('rel' => 'tag', + common_element_start('li'); + common_element('a', array('rel' => 'bookmark tag', 'href' => common_local_url('peopletag', array('tag' => $tag))), $tag); + common_element_end('li'); } + common_element_end('ul'); common_element_end('dd'); common_element_end('dl'); diff --git a/actions/tagother.php b/actions/tagother.php index ed9d7c80d1..f957241526 100644 --- a/actions/tagother.php +++ b/actions/tagother.php @@ -88,7 +88,7 @@ class TagotherAction extends Action { } common_element_start('form', array('method' => 'post', - 'id' => 'tagother', + 'id' => 'tag_user', 'name' => 'tagother', 'action' => $this->self_url())); common_hidden('token', common_session_token()); diff --git a/lib/profilelist.php b/lib/profilelist.php index ae0acd9ced..de9c6395a4 100644 --- a/lib/profilelist.php +++ b/lib/profilelist.php @@ -26,9 +26,8 @@ class ProfileList { var $profile = NULL; - function __construct($profile, $owner=NULL) { + function __construct($profile) { $this->profile = $profile; - $this->owner = $owner; } function show_list() { @@ -58,7 +57,6 @@ class ProfileList { 'id' => 'profile-' . $this->profile->id)); $user = common_current_user(); - if ($user && $user->id != $this->profile->id) { # XXX: special-case for user looking at own # subscriptions page @@ -111,51 +109,68 @@ class ProfileList { } $tags = Profile_tag::getTags($this->profile->id, $this->profile->id); - + + if ($tags) { - common_element_start('p', 'tags'); + common_element_start('div', 'tags_self'); + common_element_start('dl'); + common_element('dt', null, _("User's tags:")); + common_element_start('dd'); + common_element_start('ul', 'tags xoxo'); foreach ($tags as $tag) { - common_element('a', array('rel' => 'tag', + common_element_start('li'); + common_element('a', array('rel' => 'bookmark tag', 'href' => common_local_url('peopletag', array('tag' => $tag))), $tag); + common_element_end('li'); } - common_element_end('p'); + common_element_end('ul'); + common_element_end('dd'); + common_element_end('dl'); + common_element_end('div'); } - if ($this->owner) { + if ($user) { $action = NULL; - if ($this->owner->isSubscribed($this->profile)) { + if ($user->isSubscribed($this->profile)) { $action = 'subscriptions'; } else if (Subscription::pkeyGet(array('subscriber' => $this->profile->id, - 'subscribed' => $this->owner->id))) { + 'subscribed' => $user->id))) { $action = 'subscribers'; } if ($action) { - $tags = Profile_tag::getTags($this->owner->id, $this->profile->id); + $tags = Profile_tag::getTags($user->id, $this->profile->id); if ($tags) { - common_element_start('p', 'subtags'); - + common_element_start('div', 'tags_user'); + common_element_start('dl'); + common_element('dt', null, _("Your tags:")); + common_element_start('dd'); + common_element_start('ul', 'tags xoxo'); foreach ($tags as $tag) { - common_element('a', array('href' => common_local_url($action, - array('nickname' => $this->owner->nickname, + common_element_start('li'); + common_element('a', array('rel' => "bookmark tag", + 'href' => common_local_url($action, + array('nickname' => $user->nickname, 'tag' => $tag))), $tag); - } - - common_element_end('p'); + common_element_end('li'); + } + common_element_end('ul'); + common_element_end('dd'); + common_element_end('dl'); + common_element_end('div'); } - if ($this->owner->id == $user->id) { - common_element('a', array('href' => common_local_url('tagother', - array('id' => $this->profile->id)), - 'class' => 'tagother'), - _('Tag')); - } + common_element_start('p', 'tag_user'); + common_element('a', array('href' => common_local_url('tagother', + array('id' => $this->profile->id))), + _('Tag user')); + common_element_end('p'); } } diff --git a/theme/default/display.css b/theme/default/display.css index eb2c262355..8b22d2b09a 100644 --- a/theme/default/display.css +++ b/theme/default/display.css @@ -368,7 +368,7 @@ input#submit_yes:hover, input#submit_no:hover { } .avatar.stream { float: left; - margin: 0 10px 0 0; + margin: 0 10px 0.5em 0; } p.time { display: block; @@ -857,6 +857,77 @@ font-size: 60%; } +.tags_self, +.tags_user { +margin-left:4.5em; +} +.tags_self, +.tags_user { +/*float:right;*/ +} +.tags_self dl, +.tags_user dl { +margin-left:0; +} + +.tags_self dt, +.tags_user dt { +/*display:none;*/ +display:inline; +} + +.tags_self dd, +.tags_user dd { +margin-left:0; +display:inline; +} + + +ul.tags { +padding-left:0; +margin-left:0; +list-style-type:none; +display:inline; +} +ul.tags li { +display:inline; +margin-right:0.75em; +} +ul.tags li a { +padding-left:17px; +background:url(icon_tag-01.gif) no-repeat; +line-height:1.5; +} + +.tag_user { + +text-align:right; +margin-top:0; +margin-bottom:0; +} + +form#tag_user { +margin-left:8.75em; +clear:both; +} +form#tag_user p { +margin:0; +} + +form#tag_user label { +display:inline; +margin-right:1em; +} +form#tag_user .submit { +margin-left:4em; +} + +form#tag_user .input_instructions { +margin-left:4.5em; +} + + + /* ----- Mailbox ----- */ #messages { clear: both; diff --git a/theme/default/icon_tag-01.gif b/theme/default/icon_tag-01.gif new file mode 100644 index 0000000000..92e5742cee Binary files /dev/null and b/theme/default/icon_tag-01.gif differ