Merge pull request #12 from NenoR96/fixingUI

fix: horizontal scroll on list
BuildFire · Feb 11, 2021 · 283370e · 283370e
2 parents 53cad6e + 84e3394
commit 283370e
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/src/widget/index.html b/src/widget/index.html
@@ -62,8 +62,11 @@
     </button>
   </form>-->
   <div role="separator" class="mdc-list-divider"></div>
-  <ul class="mdc-list mdc-list--two-line mdc-list--avatar-list" id="inboxMessages">
-  </ul>
+  <div class="scrollDiv">
+    <ul class="mdc-list mdc-list--two-line mdc-list--avatar-list" id="inboxMessages">
+    </ul>
+  </div>
+
   <div id="emptyState">
     <span class="primaryTextTheme">No results found</span>
     <icon class="material-icons icon icon-drawer2 empty-icon"></icon>

diff --git a/src/widget/style.css b/src/widget/style.css
@@ -197,10 +197,9 @@ span.mdc-chip__primary-action:focus {
 }
 
 .mdc-list {
-  position: absolute;
-  overflow-y: scroll;
-  height: 75vh;
-  width: 100%;
+  max-height: calc(100vh - 115px);
+  overflow-y: auto;
+  overflow-x: hidden;
 }
 
 .mdc-list-item__primary-text {

diff --git a/src/widget/widget.js b/src/widget/widget.js
@@ -137,7 +137,7 @@ function render(thread, otherUser, chipText, index,callback) {
           }
           actionItem = JSON.stringify(actionItem);
           navigate();
-        }
+        } else { actionItem = null; navigate(); }
       });
 
     } else {