Skip to content

Commit 646f0cd

Browse files
increase navigator-dock-wrapper width so that it doesn't overflow in x
[1] reduced the navigator sidebar width from 330px to 250px, but it didn't take into account the overflow-x (which is hidden). when the elements were inserted dynamically (by javascript), they got scrolled off to the left side giving an impression that they are cut off. this patch increases the width to 266px as at that width, no overflow-x exists on navigator-dock-wrapper. -- [1] Reduce the navigator sidebar width and change bg color for navigator 85d604d Signed-off-by: Sahil Gautam <[email protected]> Change-Id: Iaf769bdce11dc27fa11af3f831f691bc2d6c9cad
1 parent 72e981a commit 646f0cd

File tree

1 file changed

+13
-1
lines changed

1 file changed

+13
-1
lines changed

browser/css/jssidebar.css

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -577,8 +577,20 @@ span.jsdialog.sidebar.ui-treeview-notexpandable {
577577
#navigation-options .tab.selected:after {
578578
border-bottom-color: rgba(var(--doc-type), 1);
579579
}
580+
581+
/*
582+
* NOTE: there are some hidden elements on the navigation sidebar
583+
* and the horizontal overflow on `navigator-dock-wrapper` is
584+
* hidden, so if the width is to be changed, then please do so
585+
* with the overflow-x visible so that it's not set too small.
586+
*
587+
* when the width is is less than the largest elements, which
588+
* might be hidden, the x overflow starts building up. and when
589+
* the elements are dynamically loaded, they might get scrolled
590+
* off to the left giving an impression that they are cut off.
591+
*/
580592
#navigation-sidebar {
581-
width: 250px;
593+
width: 266px;
582594
display: none;
583595
border-inline-end: 1px solid var(--color-border);
584596
/* bg set in #sidebar-dock-wrapper, #navigator-dock-wrapper */

0 commit comments

Comments
 (0)