Sticky scroll sets line-height for the sticky row even though I set setRowLineHeight: false in the ListOptions

[TylerLeonhardt]

Does this issue occur when all extensions are disabled?: Yes/No

• VS Code Version:
• OS Version:

Steps to Reproduce:

1. See how this is set:
   

   vscode/src/vs/platform/quickinput/browser/quickInputTree.ts

   Line 541 in ca19ca6

   setRowLineHeight: false,

2. See this:
   <div class="monaco-tree-sticky-row monaco-list-row focused" data-index="0" data-parity="even" id="list_id_3_0" role="option" aria-label="extensions.json, extensions.json" aria-selected="false" aria-expanded="true" style="top: 0px; height: 24px; line-height: 24px;">
   Notice the line height is set to 24px.

This causes my item to jiggle when I start scrolling:

Screen.Recording.2024-03-13.at.4.00.56.PM.mov

if I get rid of that line-height setting, the behavior works as expected. No jittering

vscode/src/vs/base/browser/ui/tree/abstractTree.ts

Line 1667 in ca19ca6

stickyElement.style.lineHeight = `${stickyNode.height}px`;

[benibenj]

Thanks for reporting, was unaware this existed. I now consider the setRowHeight and setRowLineHeight options