Merge pull request #1062 from hypothesis/make-sidebar-open-on-group-link

Open the sidebar when direct-linked group

src/annotator/sidebar.coffee

@@ -34,7 +34,7 @@ module.exports = class Sidebar extends Host
 
     this.hide()
 
-    if config.openSidebar || config.annotations || config.query
+    if config.openSidebar || config.annotations || config.query || config.group
       this.on 'panelReady', => this.show()
 
     if @plugins.BucketBar?

src/annotator/test/sidebar-test.coffee

@@ -233,6 +233,58 @@ describe 'Sidebar', ->
         sidebar.onPan({type: 'panright', deltaX: 100})
         assert.equal(sidebar.gestureState.final, 0)
 
+  describe 'panelReady event', ->
+    it 'opens the sidebar when a direct-linked annotation is present.', ->
+      sidebar = createSidebar(
+        config={
+          annotations: 'ann-id',
+        }
+      )
+      show = sandbox.stub(sidebar, 'show')
+      sidebar.publish('panelReady')
+      assert.calledOnce(show)        
+
+    it 'opens the sidebar when a direct-linked group is present.', ->
+      sidebar = createSidebar(
+        config={
+          group: 'group-id',
+        }
+      )
+      show = sandbox.stub(sidebar, 'show')
+      sidebar.publish('panelReady')
+      assert.calledOnce(show)        
+
+    it 'opens the sidebar when a direct-linked query is present.', ->
+      sidebar = createSidebar(
+        config={
+          query: 'tag:foo',
+        }
+      )
+      show = sandbox.stub(sidebar, 'show')
+      sidebar.publish('panelReady')
+      assert.calledOnce(show)        
+
+    it 'opens the sidebar when openSidebar is set to true.', ->
+      sidebar = createSidebar(
+        config={
+          openSidebar: true,
+        }
+      )
+      show = sandbox.stub(sidebar, 'show')
+      sidebar.publish('panelReady')
+      assert.calledOnce(show)        
+
+    it 'does not show the sidebar if not configured to.', ->
+      sidebar = createSidebar(
+        config={
+        }
+      )
+      show = sandbox.stub(sidebar, 'show')
+      sidebar.publish('panelReady')
+      assert.notCalled(show)        
+
+
+
   describe 'swipe gestures', ->
     sidebar = null