Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added option for smooth scrolling to LXQt file dialog #740

Merged
merged 1 commit into from
Oct 21, 2021

Conversation

tsujan
Copy link
Member

@tsujan tsujan commented Oct 20, 2021

It appears only with list and compact modes. By default, smooth scrolling is enabled.

The reason behind this option is that Qt may slow down item sorting when the number of items is huge and scrolling is done per pixel in list or compact mode. As a workaround, the user can disable per-pixel scrolling by disabling smooth scrolling.

Closes #739

WARNING: All libfm-qt based apps should be recompiled.

It appears only with list and compact modes. By default, smooth scrolling is enabled.

The reason behind this option is that Qt may slow down item sorting when the number of items is huge and scrolling is done per pixel in list or compact mode. As a workaround, the user can disable per-pixel scrolling by disabling smooth scrolling.

Closes #739

WARNING: All libfm-qt based apps should be recompiled.
@tsujan
Copy link
Member Author

tsujan commented Oct 21, 2021

Tested by @Chiitoo and me. I couldn't find a suspicious code line after rereading it. Merging...

@stefonarch, I'll make a PR for pcmanfm-qt today and appreciate your testing it with this change before November.

@tsujan tsujan merged commit fc2bdc3 into master Oct 21, 2021
@tsujan tsujan deleted the smooth_scroll_option branch October 21, 2021 14:39
@stefonarch
Copy link
Member

stefonarch commented Oct 22, 2021

Only now translating I wonder if
Disable smooth scrolling in list and compact modes
should be
Disable smooth scrolling in list and compact views
woould be better.

@tsujan
Copy link
Member Author

tsujan commented Oct 22, 2021

Both are correct but, more precisely, we have a single view that can be in different modes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PCManFM-Qt: Smooth Scrolling Adds Noticeable Delays to Detailed List Mode
2 participants