Skip to content

Add folder expand option#10077

Merged
rafaeltonholo merged 5 commits into
thunderbird:mainfrom
shamim-emon:fix-issue-10036
Nov 18, 2025
Merged

Add folder expand option#10077
rafaeltonholo merged 5 commits into
thunderbird:mainfrom
shamim-emon:fix-issue-10036

Conversation

@shamim-emon

@shamim-emon shamim-emon commented Nov 8, 2025

Copy link
Copy Markdown
Collaborator

🧩 Summary

This PR introduces a new general setting called drawerExpandAllFolder, allowing users to control whether folders in the navigation drawer are expanded or collapsed by default.

🛠️ Changes

  • Added a new general setting drawerExpandAllFolder to the app’s settings system.
  • Integrated the setting into DrawerContract and the related UI components to determine the initial expanded/collapsed state of side-menu folders.
  • Exposed the option under General Settings > Display Setting > DisplayVisualSettings for user control.
  • Updated relevant test cases to ensure proper behavior for the new setting.

🎯 Impact

Users can now personalize how their folder drawer behaves on startup, improving the customization and usability of the app’s navigation experience.

@shamim-emon shamim-emon requested a review from a team as a code owner November 8, 2025 10:57
@shamim-emon shamim-emon changed the title Fix issue 10036 Add folder expand option Nov 8, 2025
@shamim-emon shamim-emon force-pushed the fix-issue-10036 branch 4 times, most recently from bce5764 to a73a98a Compare November 8, 2025 14:42
@platform34

Copy link
Copy Markdown

@shamim-emon Could I make a suggestion to change the description of setting "Auto-Expand Folder" found under settings > account > folders, to avoid confusion with settings your PR will introduce. The setting's name I mention is already ambiguous. It seems to define the default folder selected after switching account view, rather than expanding any folder.

@shamim-emon

shamim-emon commented Nov 9, 2025

Copy link
Copy Markdown
Collaborator Author

@shamim-emon Could I make a suggestion to change the description of setting "Auto-Expand Folder" found under settings > account > folders, to avoid confusion with settings your PR will introduce. The setting's name I mention is already ambiguous. It seems to define the default folder selected after switching account view, rather than expanding any folder.

@platform34 thanks for pointing it out. I updated the title for your mentioned setting, so that it is more appropriate with it's usage.

@rafaeltonholo rafaeltonholo added the merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. label Nov 10, 2025
@wmontwe wmontwe removed the merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. label Nov 14, 2025
@shamim-emon shamim-emon force-pushed the fix-issue-10036 branch 3 times, most recently from 9d7d34b to 15fbb4a Compare November 14, 2025 15:50
rafaeltonholo
rafaeltonholo previously approved these changes Nov 14, 2025

@rafaeltonholo rafaeltonholo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! I've just left a few comments for your consideration, but feel free to merge it without addressing them.

Comment thread legacy/core/src/main/java/com/fsck/k9/preferences/DefaultDrawerConfigManager.kt Outdated
Comment thread legacy/core/src/main/java/com/fsck/k9/preferences/DefaultDrawerConfigManager.kt Outdated
@@ -239,7 +239,7 @@
<com.fsck.k9.ui.settings.account.FolderListPreference
android:key="account_setup_auto_expand_folder"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should also rename the key to better fit with the new title. However, this can be done in a follow-up ticket.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thank you for your suggestion. I would prefer to do this in a follow-up ticket.

…ntract.kt and expose it to UI to control initial collapsed/expanded state of side menu folders
… to be toggled from General Settings → Display Settings section

@rafaeltonholo rafaeltonholo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 🚀

@rafaeltonholo rafaeltonholo merged commit 5a3de74 into thunderbird:main Nov 18, 2025
13 checks passed
@thunderbird-botmobile thunderbird-botmobile Bot added this to the Thunderbird 16 milestone Nov 18, 2025
@shamim-emon shamim-emon deleted the fix-issue-10036 branch November 18, 2025 11:56
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.

Add folder expand option

4 participants