Conversation link as default, permalink for /notice

diff --git a/lib/activityhandlerplugin.php b/lib/activityhandlerplugin.php
index 59e3d45f17d68a49118338f10103934ec5a93f26..6a148d9dd545f709fe0483140d9202245c25a6b0 100644 (file)
--- a/lib/activityhandlerplugin.php
+++ b/lib/activityhandlerplugin.php
@@ -594,7 +594,7 @@ abstract class ActivityHandlerPlugin extends Plugin
         $nli->showNoticeLink();
         $nli->showNoticeSource();
         $nli->showNoticeLocation();
-        $nli->showContext();
+        $nli->showPermalink();
         $nli->showRepeat();
 
         $nli->showNoticeOptions();

diff --git a/lib/noticelistitem.php b/lib/noticelistitem.php
index c1f4132c19d404d43fa04f0aac8a0c0631d7fd25..81fcb72caa77f25d7922b1fc31306a9609d78a08 100644 (file)
--- a/lib/noticelistitem.php
+++ b/lib/noticelistitem.php
@@ -139,9 +139,7 @@ class NoticeListItem extends Widget
             $this->showNoticeLink();
             $this->showNoticeSource();
             $this->showNoticeLocation();
-            if ($this->notice->hasConversation()) {
-                $this->showContext();
-            }
+            $this->showPermalink();
             $this->showRepeat();
             Event::handle('EndShowNoticeInfo', array($this));
         }
@@ -311,7 +309,7 @@ class NoticeListItem extends Widget
     {
         $this->out->elementStart('a', array('rel' => 'bookmark',
                                             'class' => 'u-url timestamp',
-                                            'href' => $this->notice->getLocalUrl()));
+                                            'href' => Conversation::getUrlFromNotice($this->notice)));
         $this->out->element('time', array('class' => 'dt-published',
                                           'datetime' => common_date_iso8601($this->notice->created),
                                           // TRANS: Timestamp title (tooltip text) for NoticeListItem
@@ -464,20 +462,19 @@ class NoticeListItem extends Widget
     }
 
     /**
-     * show link to notice this notice is a reply to
+     * show link to single-notice view for this notice item
      *
-     * If this notice is a reply, show a link to the notice it is replying to. The
-     * heavy lifting for figuring out replies happens at save time.
+     * A permalink that goes to this specific object and nothing else
      *
      * @return void
      */
-    function showContext()
+    function showPermalink()
     {
         $this->out->element('a',
-                            array('href' => $this->notice->getConversationUrl(),
-                                  'class' => 'conversation'),
-                            // TRANS: Addition in notice list item if notice is part of a conversation.
-                            _('in context'));
+                            array('href' => $this->notice->getLocalUrl(),
+                                  'class' => 'permalink'),
+                            // TRANS: Addition in notice list item for single-notice view.
+                            _('permalink'));
     }
 
     /**

diff --git a/lib/threadednoticelist.php b/lib/threadednoticelist.php
index 15d286dbf97529c621e376461615881e243924d0..18d13505c8ddcde23ae79d57aa02a558c06f02de 100644 (file)
--- a/lib/threadednoticelist.php
+++ b/lib/threadednoticelist.php
@@ -185,11 +185,6 @@ class ThreadedNoticeListItem extends NoticeListItem
         return 3;
     }
 
-    function showContext()
-    {
-        // Silence!
-    }
-
     /**
      * finish the notice
      *
@@ -299,11 +294,6 @@ class ThreadedNoticeListSubItem extends NoticeListItem
         //
     }
 
-    function showContext()
-    {
-        //
-    }
-
     function getReplyProfiles()
     {
         $all = parent::getReplyProfiles();

diff --git a/plugins/ConversationTree/lib/conversationtreeitem.php b/plugins/ConversationTree/lib/conversationtreeitem.php
index d7db04e1ddce05004231c741bc02a920efbe5e82..65911dfc48abff943a1a6454b6be0e38c30c0da7 100644 (file)
--- a/plugins/ConversationTree/lib/conversationtreeitem.php
+++ b/plugins/ConversationTree/lib/conversationtreeitem.php
@@ -69,18 +69,6 @@ class ConversationTreeItem extends NoticeListItem
         return;
     }
 
-    /**
-     * show link to notice conversation page
-     *
-     * Since we're only used on the conversation page, we skip this
-     *
-     * @return void
-     */
-    function showContext()
-    {
-        return;
-    }
-
     /**
      * show people this notice is in reply to
      *

diff --git a/plugins/GNUsocialProfileExtensions/lib/noticetree.php b/plugins/GNUsocialProfileExtensions/lib/noticetree.php
index 4bdffc9428e23b34d938919bcc7c706caf3e6629..95df52838846d218ce296052f35bab5444b552a8 100644 (file)
--- a/plugins/GNUsocialProfileExtensions/lib/noticetree.php
+++ b/plugins/GNUsocialProfileExtensions/lib/noticetree.php
@@ -146,11 +146,6 @@ class NoticeTreeItem extends NoticeListItem
         return;
     }
 
-    function showContext()
-    {
-        return;
-    }
-
     //Just changing the link...
     function showReplyLink()
     {

diff --git a/plugins/QnA/QnAPlugin.php b/plugins/QnA/QnAPlugin.php
index 1cbab3b60b30d19ce9b98e3b2b89185476b771f9..e0f5fed12bc5017bf397411f06e74b2b9ebf2d00 100644 (file)
--- a/plugins/QnA/QnAPlugin.php
+++ b/plugins/QnA/QnAPlugin.php
@@ -339,7 +339,7 @@ class QnAPlugin extends MicroAppPlugin
         $nli->showNoticeLink();
         $nli->showNoticeSource();
         $nli->showNoticeLocation();
-        $nli->showContext();
+        $nli->showPermalink();
         $nli->showRepeat();
 
         $nli->showNoticeOptions();

diff --git a/theme/base/css/display.css b/theme/base/css/display.css
index 06235c791cb38e57de7fa21a8fb76e6bfd1f0214..d78c2f3d48d2b1b588abf72f7406921b1c4ef09d 100644 (file)
--- a/theme/base/css/display.css
+++ b/theme/base/css/display.css
@@ -983,6 +983,14 @@ content: ":";
     display: none;
 }
 
+.notice .permalink:after {
+    content: ']';
+}
+
+.notice .permalink:before {
+    content: '[';
+}
+
 /* old school conversation style */
 
 #conversation .notices .notices {