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

Fix "None on top" sort mode #3966

Merged
merged 2 commits into from
Oct 30, 2023
Merged

Conversation

TranceLove
Copy link
Collaborator

@TranceLove TranceLove commented Oct 24, 2023

Description

When the option is "None on top", I think there should be no file and/or folder header labels displayed.

Issue tracker

Fixes #942

Manual tests

  • Done

Device: Pixel 6 emulator running Android 13

Folder and Files on Top options should continue to work

While None on Top option will simply sort the files and folders together.

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

@TranceLove TranceLove added Issue-Bug Related unexpected behavior or something worth investigating. Area-UIUX Related to user interface/experience (e.g. dialogs, notifications). labels Oct 24, 2023
@TranceLove TranceLove self-assigned this Oct 24, 2023
@TranceLove TranceLove changed the title Fix FileListSorter "None on top" sort mode Fix "None on top" sort mode Oct 24, 2023
@TranceLove
Copy link
Collaborator Author

Necessary work should be adequate; but I shall let you decide if it should be at 4.0 release.

@TranceLove TranceLove force-pushed the bugfix/942 branch 2 times, most recently from 7a8302c to e517896 Compare October 24, 2023 22:51
When the option is "None on top", I think there should be no file and/or folder header labels displayed.

Fixes TeamAmaze#942.
@VishalNehra VishalNehra merged commit b2a5e1d into TeamAmaze:release/4.0 Oct 30, 2023
3 checks passed
@TranceLove TranceLove deleted the bugfix/942 branch November 5, 2023 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-UIUX Related to user interface/experience (e.g. dialogs, notifications). Issue-Bug Related unexpected behavior or something worth investigating.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong headers for files/folders when 'None on Top' sort mode is selected
2 participants