Fixes oppia#3819: Strange dragging behaviour of play later list in le…

…arner dashboard. (oppia#6544) * Added overflow : visible and top : 0 * Fix spaceing * Updated comments
mschmittfull · May 3, 2019 · 3f1eeb3 · 3f1eeb3
1 parent 3723a8f
commit 3f1eeb3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/core/templates/dev/head/pages/learner_dashboard/LearnerDashboard.ts b/core/templates/dev/head/pages/learner_dashboard/LearnerDashboard.ts
@@ -370,10 +370,9 @@ oppia.controller('LearnerDashboard', [
         },
         sort: function(e, ui) {
           /* eslint-disable quote-props */
-          // Reset the position of the window on scrolling. This keeps the mouse
-          // position and elements in sync.
+          // Making top : 0px to avoid irregular change in position.
           ui.helper.css(
-            {'top': ui.position.top + $(window).scrollTop() + 'px'});
+            {'top': '0 px'});
           /* eslint-enable quote-props */
         },
         update: function(e, ui) {

diff --git a/core/templates/dev/head/pages/learner_dashboard/learner_dashboard.html b/core/templates/dev/head/pages/learner_dashboard/learner_dashboard.html
@@ -785,6 +785,10 @@ <h1 class="stat-value"><[subscriptionsList.length]></h1>
   </div>
   <style>
 
+    body {
+      overflow-x: visible;
+    }
+
     .oppia-learner-dashboard-container {
       margin: 0 auto 30px auto;
       max-width: 1400px;