Skip to content
This repository was archived by the owner on Jun 24, 2025. It is now read-only.

Conversation

@SiriusXT
Copy link
Member

@SiriusXT SiriusXT commented Apr 14, 2025

Related to TriliumNext/Trilium#5594

disable scrolling when there are few tabs Scrolling allowed with many tabs, add_new_tab does not shift Dragging works correctly when scrolling is allowed The Electron window dragging works correctly
Vertical Layout
Horizontal Layout
Mobile

preview

@github-actions
Copy link

github-actions bot commented Apr 14, 2025

Folder/File Previous size New size Difference
/upload/TriliumNextNotes-Server-1700-merge-linux-x64.tar.xz 82.74MB +82.74MB (+100.00%)
TOTAL +82.74MB

@github-actions
Copy link

github-actions bot commented Apr 14, 2025

Folder/File Previous size New size Difference
/upload/TriliumNextNotes-1700-merge-linux-x64.deb 135.34MB +135.34MB (+100.00%)
/upload/TriliumNextNotes-1700-merge-linux-x64.flatpak 128.12MB +128.12MB (+100.00%)
/upload/TriliumNextNotes-1700-merge-linux-x64.rpm 143.22MB +143.22MB (+100.00%)
/upload/TriliumNextNotes-1700-merge-linux-x64.zip 173.85MB +173.85MB (+100.00%)
TOTAL +580.53MB

@SiriusXT SiriusXT marked this pull request as ready for review April 14, 2025 11:27
@SiriusXT SiriusXT changed the title Allow the tab row scroll Allow the Tab row scroll Apr 14, 2025
Copy link
Contributor

@eliandoran eliandoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty good work. :)

However, it seems that on the horizontal layout, the tabs are missing one pixel in height at the top which makes the tab appear slightly cut off (can be seen with the naked eye, albeit a little bit more difficult). At the bottom there is also a "half-pixel" border on the active tab, which there shouldn't be.

This happens on Firefox 137.

Before:

image

After:

image

@eliandoran eliandoran marked this pull request as draft April 14, 2025 17:02
@eliandoran eliandoran added this to the v0.93.0 milestone Apr 14, 2025
@SiriusXT SiriusXT marked this pull request as ready for review April 15, 2025 11:48
Copy link
Contributor

@eliandoran eliandoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using a laptop's touch pad on Windows on Electron seems that tapping on a tab to select it no longer works, you have to double-tab.

@eliandoran eliandoran marked this pull request as draft April 15, 2025 15:03
@SiriusXT SiriusXT marked this pull request as ready for review April 15, 2025 15:38
@eliandoran eliandoran merged commit afac983 into develop Apr 15, 2025
10 checks passed
@eliandoran eliandoran deleted the tab-scroll branch April 15, 2025 18:36
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants