From f65b272760048331e142bf9c97be952dc87166ed Mon Sep 17 00:00:00 2001 From: Florrie Date: Wed, 25 Jul 2018 13:27:03 -0300 Subject: Space for scrollbar in menu --- todo.txt | 3 +++ ui.js | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/todo.txt b/todo.txt index 0227763..76f0c2e 100644 --- a/todo.txt +++ b/todo.txt @@ -93,3 +93,6 @@ TODO: Press "M" to show a context menu. TODO: Figure out a keybinding for Mark (in edit mode) - probably Ctrl-M. (I prefer Ctrl to be used for functions that will open a dialog of some sort, though..) + +TODO: Only show scrollbars if the form is actually scrollable. (This would make + context menus look better.) diff --git a/ui.js b/ui.js index afb2731..f92d259 100644 --- a/ui.js +++ b/ui.js @@ -1477,8 +1477,8 @@ class ContextMenu extends FocusElement { let height = Math.min(10, this.form.inputs.length) - width += 2 // Space for the pane border - height += 2 + width += 3 // Space for the pane border and scrollbar + height += 2 // Space for the pane border this.w = width this.h = height -- cgit 1.3.0-6-gf8a5