microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
160.19k stars 28.06k forks source link

Show sticky inline items for list items when horizontal scrolling is enabled #212802

Open DFelten opened 1 month ago

DFelten commented 1 month ago

Type: Bug

The toolbar within the test explorer is hidden if the sidebar is not wide enough. In a previous version, the toolbar was always visible on hover.

On hover:

CleanShot 2024-05-15 at 13 17 06@2x

CleanShot 2024-05-15 at 13 17 40@2x

VS Code version: Code - Insiders 1.90.0-insider (bbc4ba1eff648ee542972589216f211b7a6a08f2, 2024-05-14T05:49:49.204Z) OS version: Darwin arm64 23.4.0 Modes:

System Info |Item|Value| |---|---| |CPUs|Apple M1 Max (10 x 2400)| |GPU Status|2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
webgpu: enabled| |Load (avg)|3, 6, 7| |Memory (System)|32.00GB (1.32GB free)| |Process Argv|--crash-reporter-id 511166e9-0c5e-4225-a6af-efd2d90c3f8d| |Screen Reader|no| |VM|0%|
Extensions (77) Extension|Author (truncated)|Version ---|---|--- better-comments|aar|3.0.2 codesnap|adp|1.3.4 vscode-m3u|af4|1.0.0 TabOut|alb|0.2.2 Bookmarks|ale|13.5.0 project-manager|ale|12.8.0 flutter-snippets|ale|3.0.0 toggle-excluded-files|amo|2.0.0 clipboard-history|Anj|1.0.7 melos-code|bla|1.1.1 vscode-intelephense-client|bme|1.10.4 ccs-flutter-color|cir|1.2.4 gitignore|cod|0.9.0 dbclient-jdbc|cwe|1.3.5 vscode-mysql-client2|cwe|7.3.7 vscode-mac-color-picker|dae|1.1.0 markdown-table-prettify|dar|3.6.0 dart-code|Dar|3.89.20240513 flutter|Dar|3.89.20240501 vscode-markdownlint|Dav|0.55.0 githistory|don|0.6.20 gitlens|eam|15.0.3 bloc|Fel|6.6.6 mason|Fel|0.1.12 flutter-coverage|flu|0.0.4 shell-format|fox|7.2.5 copilot|Git|1.192.0 copilot-chat|Git|0.16.2024051501 vscode-github-actions|git|0.26.2 vscode-pull-request-github|Git|0.88.1 todo-tree|Gru|0.0.226 terraform|has|2.30.1 json-to-dart|hir|3.5.8 rest-client|hum|0.25.1 pubspec-assist|jer|2.3.2 shortcut-menu-bar|jer|3.0.4 intellij-idea-keybindings|k--|1.7.2 indent-to-bracket|LAK|1.2.2 flutter-intl|loc|1.19.3 syntax-project-pbxproj|mar|0.1.3 flutter-snippets|Mar|1.0.5 Kotlin|mat|1.7.1 rainbow-csv|mec|3.11.0 git-graph|mhu|1.30.0 dart-barrel-file-generator|miq|6.2.2 diff-merge|mos|0.7.0 vscode-docker|ms-|1.29.1 vscode-language-pack-de|MS-|1.89.2024050109 remote-containers|ms-|0.365.0 live-server|ms-|0.4.13 vsliveshare|ms-|1.0.5918 vsliveshare-pack|ms-|0.4.0 vscode-flutter-freezed-helper|mth|0.1.0 sqltools|mtx|0.28.3 sqltools-driver-mysql|mtx|0.6.2 awesome-flutter-snippets|Nas|4.0.1 vscode-streamdeck|nic|4.1.6 vscode-yaml-sort|Pas|6.5.17 vscode-versionlens|pfl|1.9.2 material-icon-theme|PKi|5.3.0 remove-comments|pli|1.2.2 vscode-yaml|red|1.14.0 vscode-sort-json|ric|1.20.0 command-variable|rio|1.63.0 flutter-riverpod-snippets|rob|1.2.2 vscode-coverage-gutters|rya|2.11.1 lcov-lang|Ser|1.0.1 swift-lang|ssw|1.9.0 freezed-model-gen|Sus|1.0.1 errorlens|use|3.17.0 vscode-lldb|vad|1.10.0 vscode-wakatime|Wak|24.5.0 vscode-todo-highlight|way|1.0.5 php-debug|xde|1.34.0 vscode-surround|yat|1.5.0 markdown-all-in-one|yzh|3.6.2 material-theme|zhu|3.17.2 (3 theme extensions excluded)
A/B Experiments ``` vsliv368cf:30146710 vspor879:30202332 vspor708:30202333 vspor363:30204092 tftest:31042121 vswsl492cf:30256198 vsc_aa:30263845 vscod805cf:30301675 vsaa593cf:30376535 py29gd2263:31024238 c4g48928:30535728 962ge761:30841072 pythongtdpath:30726887 welcomedialog:30812478 pythonidxpt:30768918 pythonnoceb:30776497 asynctok:30898717 dsvsc013:30777762 dsvsc014:30777825 dsvsc015:30821418 pythontestfixt:30866404 pythonregdiag2:30926734 pythonmypyd1:30859725 pythoncet0:30859736 h48ei257:31000450 pythontbext0:30879054 accentitlementst:30870582 dsvsc016:30879898 dsvsc017:30880771 dsvsc018:30880772 cppperfnew:30980852 pythonait:30973460 showvideot:31016890 chatpanelt:31014475 0ee40948:31013168 a69g1124:31018687 dvdeprecation:31040973 pythonprt:31036556 dwnewjupyter:31046869 ```
connor4312 commented 1 month ago

You can configure this in the workbench.list.horizontalScrolling setting. The test explorer works the same as other views when this is enabled.

DFelten commented 1 month ago

@connor4312 So this is changed/worsened by purpose?

In the previous version of VSCode (current stable 1.89.1) the behaviour was different with a better user experience. Here I didn't have to scroll to start tests. The mentioned setting is already active since I use this all the time for the folder view, but does not help with the problem here.

Stable VSCode

CleanShot 2024-05-15 at 18 05 58@2x

Is it possible to just disable scrolling for the test explorer? Unlike the other views, here there is an action at the end...

connor4312 commented 1 month ago

Previously, this view did not respect the workbench.list.horizontalScrolling like other views did (https://github.com/microsoft/vscode/issues/210542) and now it does. Views with toolbar actions like debug-related views behave the same way under the settting.

DFelten commented 1 month ago

Thanks for the info. Unfortunately, the usability suffers from the customization. However, if this is planned, I will create a feature request to be able to set this separately.

connor4312 commented 1 month ago

sure, I'll have this open as an overall feature request. Might be a bit of a pain to fix though due to DOM constraints