Auto-adjust fits collapsed columns to the screen; dims manual offset (v1.6.0)

Asset/css/shrink-vertically.css

@@ -28,15 +28,17 @@
  * max-height alone does nothing. A stylesheet !important overrides the inline
  * (non-important) min-height so max-height can win.
  *
- * We override it to a small NON-ZERO value, not 0: Kanboard relies on that same inline
+ * We override it to a NON-ZERO value, not 0: Kanboard relies on that same inline
  * min-height to keep EMPTY columns tall enough to be a drop target, so zeroing it makes
- * you unable to drop a card into an empty collapsed column. 100px clears Kanboard's 70px
- * drag placeholder (.draggable-placeholder) with margin, and stays well below max-height
- * so tall columns still shrink; non-empty columns are unaffected (their content is taller).
+ * you unable to drop a card into an empty collapsed column. 250px gives a comfortable drop
+ * target (well above Kanboard's 70px .draggable-placeholder) while staying below the
+ * collapsed max-height, so the board never overflows the viewport (which would re-hide the
+ * horizontal scrollbar). Non-empty columns are unaffected -- their content is taller -- and
+ * short columns simply get this min-height, which is only cosmetic (no scrollbar).
  */
 #board td .board-task-list.board-task-list-compact {
     max-height: calc(100vh - var(--sv-shrink-offset, 240px)) !important;
-    min-height: 100px !important;
+    min-height: 250px !important;
 }
 
 /*

Asset/js/auto-adjust.js

@@ -0,0 +1,86 @@
+/*
+ * ShrinkVertically -- "Auto adjust" mode. Measure the collapsed board live and set the
+ * --sv-shrink-offset CSS variable so the columns exactly fill the remaining viewport and the
+ * native right-side vertical scrollbar is not needed. Loaded only when the setting is on
+ * (Settings -> Shrink Vertically); it overrides the static offset injected by layout/variable.php.
+ *
+ * The collapsed column height is calc(100vh - var(--sv-shrink-offset)). To fit the screen we need:
+ *     offset = (task-list top, i.e. the chrome above) + (horizontal scrollbar below) + a small margin
+ * so column_bottom + scrollbar stays within the viewport. Both terms are measured from the DOM, so
+ * this adapts to any screen size, theme, or header height.
+ *
+ * Re-measures on resize, on collapse/expand (a class toggle), and after a board refresh
+ * (BoardDragAndDrop replaces #board-container -- so, like the other plugins, the observer is bound
+ * to the STABLE parent, not to the container that gets swapped out).
+ */
+(function () {
+    "use strict";
+
+    var BOTTOM_MARGIN = 16; // breathing room below the columns
+    var scheduled = false;
+
+    function apply() {
+        scheduled = false;
+
+        var list = document.querySelector("#board .board-task-list.board-task-list-compact");
+
+        if (! list) {
+            return; // board not collapsed -> nothing to auto-size
+        }
+
+        var top = list.getBoundingClientRect().top;
+        var container = document.getElementById("board-container");
+        var bar = container ? (container.offsetHeight - container.clientHeight) : 0; // horizontal scrollbar
+
+        if (bar < 0) {
+            bar = 0;
+        }
+
+        var offset = Math.ceil(top + bar + BOTTOM_MARGIN);
+
+        if (offset < 0) {
+            offset = 0;
+        }
+
+        var next = offset + "px";
+        var root = document.documentElement;
+
+        // Only write when it actually changes, so mid-drag class churn does not thrash layout.
+        if (root.style.getPropertyValue("--sv-shrink-offset") !== next) {
+            root.style.setProperty("--sv-shrink-offset", next);
+        }
+    }
+
+    function schedule() {
+        if (! scheduled) {
+            scheduled = true;
+            window.requestAnimationFrame(apply);
+        }
+    }
+
+    function init() {
+        var container = document.getElementById("board-container");
+
+        if (! container) {
+            return; // not a board page
+        }
+
+        schedule();
+        window.addEventListener("resize", schedule);
+
+        if (window.MutationObserver && container.parentNode) {
+            new MutationObserver(schedule).observe(container.parentNode, {
+                childList: true,
+                subtree: true,
+                attributes: true,
+                attributeFilter: ["class"]
+            });
+        }
+    }
+
+    if (document.readyState === "loading") {
+        document.addEventListener("DOMContentLoaded", init);
+    } else {
+        init();
+    }
+})();

Asset/js/top-scrollbar.js

@@ -26,6 +26,10 @@
     // Keep the thumb wide enough to grab, even on a very wide board.
     var MIN_THUMB = 24;
 
+    // Ignore a few px of residual overflow (padding / sub-pixel rounding) so a board that visually
+    // fits is treated as "no horizontal scroll" instead of showing an immovable near-full thumb.
+    var FIT_TOLERANCE = 4;
+
     function getContainer() {
         return document.getElementById("board-container");
     }
@@ -74,17 +78,29 @@
             thumb.style.transform = "translateX(" + Math.round(left) + "px)";
         }
 
-        // Recompute thumb width/position; hide the whole bar when nothing to scroll.
+        // The board needs scrolling only if its content exceeds the viewport by more than the
+        // tolerance (absorbs a residual px or two so a fitting board reads as "no scroll").
+        function needsScroll(m) {
+            return m && m.content - m.viewport > FIT_TOLERANCE;
+        }
+
+        // Recompute the thumb. The (light) track stays visible for a STABLE height; when the board
+        // fits, just drop the knob (thumb width 0) so it reads as "nothing to scroll" without the
+        // layout jump that hiding the whole bar would cause.
         function layout() {
             var m = metrics();
 
-            if (!m || m.content <= m.viewport || m.trackW <= 0) {
-                track.style.display = "none";
+            if (!m || m.trackW <= 0) {
                 return;
             }
 
             track.style.display = "";
 
+            if (!needsScroll(m)) {
+                thumb.style.width = "0px";
+                return;
+            }
+
             var thumbW = Math.round(m.trackW * m.viewport / m.content);
             if (thumbW < MIN_THUMB) { thumbW = MIN_THUMB; }
             if (thumbW > m.trackW) { thumbW = m.trackW; }
@@ -99,6 +115,7 @@
             if (c && c.parentNode && track.nextSibling !== c) {
                 c.parentNode.insertBefore(track, c);
             }
+            observeSizes();
         }
 
         // Board scrolled by any other means -> keep the thumb in sync. Capture-phase on
@@ -107,7 +124,7 @@
             if (dragging) { return; }
             if (e.target !== getContainer()) { return; }
             var m = metrics();
-            if (!m || m.content <= m.viewport) { return; }
+            if (!needsScroll(m)) { return; }
             positionThumb(m, thumb.offsetWidth);
         }, true);
 
@@ -153,7 +170,7 @@
         track.addEventListener("pointerdown", function (e) {
             if (e.target === thumb || dragging) { return; }
             var m = metrics();
-            if (!m || m.content <= m.viewport) { return; }
+            if (!needsScroll(m)) { return; }
 
             var thumbW = thumb.offsetWidth;
             var maxThumb = m.trackW - thumbW;
@@ -169,18 +186,48 @@
             positionThumb(m, thumbW);
         });
 
-        window.addEventListener("resize", layout);
+        // Single source of truth for recomputing the thumb: a ResizeObserver on the container
+        // (viewport) and #board (content). It fires exactly when the width settles/changes -- the
+        // right moment -- which also fixes the stale thumb from the first layout() running before the
+        // board finished sizing. Both are replaced on a board refresh, so re-observe the current ones
+        // (from reattachTrack). Without ResizeObserver, fall back to a window resize listener.
+        var sizeObserver = window.ResizeObserver ? new ResizeObserver(function () { layout(); }) : null;
+        var observedContainer = null;
+        var observedBoard = null;
 
-        // Observe the STABLE parent so we keep working after #board-container is replaced by
-        // a card move / AJAX polling; re-seat the track and recompute sizes on any change.
-        if (window.MutationObserver) {
-            new MutationObserver(function () {
-                reattachTrack();
+        function observeSizes() {
+            if (!sizeObserver) {
                 layout();
-            }).observe(boardParent, { childList: true, subtree: true });
+                return;
+            }
+
+            var c = getContainer();
+            var b = document.getElementById("board");
+
+            if (c && c !== observedContainer) {
+                if (observedContainer) { sizeObserver.unobserve(observedContainer); }
+                sizeObserver.observe(c);
+                observedContainer = c;
+            }
+
+            if (b && b !== observedBoard) {
+                if (observedBoard) { sizeObserver.unobserve(observedBoard); }
+                sizeObserver.observe(b);
+                observedBoard = b;
+            }
         }
 
-        layout();
+        if (!sizeObserver) {
+            window.addEventListener("resize", layout);
+        }
+
+        // Re-seat the track and re-observe after #board-container is replaced by a card move / AJAX
+        // polling (we watch the STABLE parent, since the container itself is swapped out).
+        if (window.MutationObserver) {
+            new MutationObserver(reattachTrack).observe(boardParent, { childList: true, subtree: true });
+        }
+
+        observeSizes();
     }
 
     if (document.readyState === "loading") {

Controller/ConfigController.php

@@ -23,6 +23,7 @@ class ConfigController extends BaseController
         $offset = (int) $this->configModel->get('shrink_vertically_offset', self::DEFAULT_OFFSET);
         $barSize = (int) $this->configModel->get('shrink_vertically_bar_size', self::DEFAULT_BAR_SIZE);
         $topScrollbar = (int) $this->configModel->get('shrink_vertically_top_scrollbar', 1);
+        $auto = (int) $this->configModel->get('shrink_vertically_auto', 0);
 
         $this->response->html($this->helper->layout->config('shrinkVertically:config/show', array(
             'title' => t('Settings').' > '.t('Shrink Vertically'),
@@ -30,6 +31,7 @@ class ConfigController extends BaseController
                 'shrink_vertically_offset' => $offset,
                 'shrink_vertically_bar_size' => $barSize,
                 'shrink_vertically_top_scrollbar' => $topScrollbar,
+                'shrink_vertically_auto' => $auto,
             ),
             'errors' => array(),
         )));
@@ -39,18 +41,24 @@ class ConfigController extends BaseController
     {
         $values = $this->request->getValues();
 
-        $offset = isset($values['shrink_vertically_offset']) ? (int) $values['shrink_vertically_offset'] : self::DEFAULT_OFFSET;
+        // In auto mode the offset input is disabled client-side, so the browser does not submit it.
+        // Fall back to the stored value (not the default) so enabling auto never wipes the manual offset.
+        $offset = isset($values['shrink_vertically_offset'])
+            ? (int) $values['shrink_vertically_offset']
+            : (int) $this->configModel->get('shrink_vertically_offset', self::DEFAULT_OFFSET);
         $offset = max(self::MIN_OFFSET, min(self::MAX_OFFSET, $offset));
 
         $barSize = isset($values['shrink_vertically_bar_size']) ? (int) $values['shrink_vertically_bar_size'] : self::DEFAULT_BAR_SIZE;
         $barSize = max(self::MIN_BAR_SIZE, min(self::MAX_BAR_SIZE, $barSize));
 
         $topScrollbar = isset($values['shrink_vertically_top_scrollbar']) ? 1 : 0;
+        $auto = isset($values['shrink_vertically_auto']) ? 1 : 0;
 
         if ($this->configModel->save(array(
             'shrink_vertically_offset' => $offset,
             'shrink_vertically_bar_size' => $barSize,
             'shrink_vertically_top_scrollbar' => $topScrollbar,
+            'shrink_vertically_auto' => $auto,
         ))) {
             $this->flash->success(t('Settings saved successfully.'));
         } else {

Plugin.php

@@ -32,6 +32,14 @@ class Plugin extends Base
                 'template' => 'plugins/ShrinkVertically/Asset/js/top-scrollbar.js',
             ));
         }
+
+        // Optional: auto-adjust the collapsed column height to the screen (measured live on the
+        // board), overriding the static pixel offset so the vertical scrollbar is not needed.
+        if ((int) $this->configModel->get('shrink_vertically_auto', 0) === 1) {
+            $this->hook->on('template:layout:js', array(
+                'template' => 'plugins/ShrinkVertically/Asset/js/auto-adjust.js',
+            ));
+        }
     }
 
     public function getPluginName()
@@ -51,7 +59,7 @@ class Plugin extends Base
 
     public function getPluginVersion()
     {
-        return '1.4.2';
+        return '1.6.0';
     }
 
     public function getPluginHomepage()

README.md

@@ -86,6 +86,11 @@ These settings are global (per Kanboard instance) and admin-only. Go to
   in the page head as the `--sv-shrink-offset` CSS variable. Increase it if the scrollbar
   is still clipped (for example with a taller theme header); decrease it to make the
   collapsed columns taller. Default: 240.
+- **Auto adjust** -- instead of a fixed offset, measure the collapsed board live and size
+  the columns to fit the screen automatically, so the right-side vertical scrollbar is not
+  needed. It re-measures on window resize, on collapse/expand, and after a board refresh.
+  While enabled, the pixel-offset field above is disabled and dimmed (the measured value is
+  used instead). Loaded as `Asset/js/auto-adjust.js` only when the option is on. Default: off.
 - **Top Horizontal Scrollbar thickness in pixels** -- thickness of the custom top
   scrollbar (all browsers). Injected as the `--sv-bar-size` CSS variable. Default: 25.
 - **Add a top horizontal scrollbar to the board** -- toggles the mirrored top scrollbar
@@ -94,3 +99,19 @@ These settings are global (per Kanboard instance) and admin-only. Go to
 ## License
 
 AGPL-3.0. See LICENSE.
+
+## More Kanboard plugins by Ruben (drbeco)
+
+All free and AGPL-3.0, at [code.beco.cc](https://code.beco.cc/beco):
+
+- **[RecoReco](https://code.beco.cc/beco/RecoReco)** -- calendar-scheduled recurring cards (yearly,
+  monthly, weekly, daily), driven by the card due date.
+- **[FinanceBuddy](https://code.beco.cc/beco/FinanceBuddy)** -- attach a money value (debit/credit
+  and installments) to cards, shown on the card and totalled per column.
+- **[OrganonTweaks](https://code.beco.cc/beco/OrganonTweaks)** -- an umbrella of small
+  quality-of-life board tweaks: remove an empty column, always show the comment icon, emphasize due
+  dates, extra search keywords and shared board filters, and more.
+- **[BulkMoveTasks](https://code.beco.cc/beco/BulkMoveTasks)** -- move every task from one board
+  column to another in a single action.
+- **[TweakDrag](https://code.beco.cc/beco/TweakDrag)** -- board drag and touch niceties:
+  drag-to-scroll, a wider column gap, and smoother card dragging.

Template/config/show.php

@@ -11,6 +11,11 @@
         <p class="form-help">
             <?= t('Pixels reserved above and below the vertically collapsed board columns (page header, column header and the horizontal scrollbar). A larger value makes the columns shorter. Default: 240.') ?>
         </p>
+
+        <?= $this->form->checkbox('shrink_vertically_auto', t('Auto adjust (fit columns to the screen automatically)'), 1, isset($values['shrink_vertically_auto']) && $values['shrink_vertically_auto'] == 1, '', array('id' => 'form-shrink_vertically_auto')) ?>
+        <p class="form-help">
+            <?= t('When enabled, the collapsed column height is measured live on the board to fit the screen, so the right-side vertical scrollbar is not needed. The pixel offset above is then ignored (shown dimmed).') ?>
+        </p>
     </fieldset>
 
     <fieldset>
@@ -32,3 +37,23 @@
         <button type="submit" class="btn btn-blue"><?= t('Save') ?></button>
     </div>
 </form>
+
+<script>
+// Dim + disable the pixel-offset input while "Auto adjust" is checked (auto mode measures the height
+// live on the board, so the manual offset is inactive). Disabled inputs are not submitted; the
+// controller preserves the stored offset when it is absent, so the manual value is never lost.
+(function () {
+    "use strict";
+    var auto = document.getElementById("form-shrink_vertically_auto");
+    var offset = document.getElementById("form-shrink_vertically_offset");
+    if (! auto || ! offset) {
+        return;
+    }
+    function sync() {
+        offset.disabled = auto.checked;
+        offset.style.opacity = auto.checked ? "0.4" : "";
+    }
+    auto.addEventListener("change", sync);
+    sync();
+})();
+</script>

VERSION

@@ -1 +1 @@
-ShrinkVertically v1.4.2
+ShrinkVertically v1.6