Fix #3258: Add share-to-classroom Button (#3743)

* Add share-to-classroom Button Solves #3258 * Add ng-click directive for twitter,classroom shares
oppia · Aug 14, 2017 · 254eb42 · 254eb42
1 parent 5b667e2
commit 254eb42
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/assets/images/general/classroom.png b/assets/images/general/classroom.png
diff --git a/core/templates/dev/head/components/share/SharingLinksDirective.js b/core/templates/dev/head/components/share/SharingLinksDirective.js
@@ -78,6 +78,9 @@ oppia.directive('sharingLinks', [
 
           $scope.twitterUrl = UrlInterpolationService.getStaticImageUrl(
             '/general/twitter.png');
+
+          $scope.classroomUrl = UrlInterpolationService.getStaticImageUrl(
+            '/general/classroom.png');
         }
       ]
     };

diff --git a/core/templates/dev/head/components/share/sharing_links_directive.html b/core/templates/dev/head/components/share/sharing_links_directive.html
@@ -12,11 +12,17 @@
   </li>
 
   <li>
-    <a ng-href="https://twitter.com/share?text=<[escapedTwitterText]>&url=<[serverName]>/<[activityType]>/<[activityId]>" onclick="return !window.open(this.href, '', 'height=460, width=640')" target="_window">
+    <a ng-href="https://twitter.com/share?text=<[escapedTwitterText]>&url=<[serverName]>/<[activityType]>/<[activityId]>" onclick="return !window.open(this.href, '', 'height=460, width=640')" ng-click="registerShareEvent('twitter')" target="_window">
       <img class="share-option-img" ng-src="<[twitterUrl]>" alt="Twitter">
     </a>
   </li>
 
+  <li>
+    <a ng-href="https://classroom.google.com/share?url=<[serverName]>/<[activityType]>/<[activityId]>" onclick="return !window.open(this.href, '', 'height=460, width=640')" ng-click="registerShareEvent('classroom')" target="_window">
+      <img class="share-option-img" ng-src="<[classroomUrl]>" alt="Classroom">
+    </a>
+  </li>
+
   <li ng-if="shareType === 'exploration'">
     <a ng-click="showEmbedExplorationModal(explorationId)" tooltip="Embed" tooltip-placement="top" tabindex="0">
       <i class="material-icons md-48 embed-link">&#xE86F;</i>