Skip to content

fix: size sync folders panel to contents - #9343

Merged
Rello merged 1 commit into
feature/settingsV2from
codex/make-sync-folder-box-dynamic-size
Jan 20, 2026
Merged

fix: size sync folders panel to contents#9343
Rello merged 1 commit into
feature/settingsV2from
codex/make-sync-folder-box-dynamic-size

Conversation

@Rello

@Rello Rello commented Jan 20, 2026

Copy link
Copy Markdown
Collaborator

Motivation

  • The sync folders listing consumed a large fixed vertical space even when empty, leading to a big empty box in the settings dialog.
  • The folder listing should behave like the other account panels and only request the space it needs and be movable into its own panel for future layout changes.

Description

  • Move the sync folders UI out of the tab and into a dedicated syncFoldersPanel in src/gui/accountsettings.ui so it sits alongside the account status and file provider panels.
  • Make the tree view report a dynamic preferred height by adding FolderStatusView::sizeHint() in src/gui/folderstatusview.{h,cpp} which computes height from the number of rows, per-row height and scrollbar presence.
  • Enable size-to-contents for the view via _ui->_folderList->setSizeAdjustPolicy(QAbstractScrollArea::AdjustToContents) and refresh geometry when the model changes by connecting rowsInserted, rowsRemoved and modelReset to updateGeometry in src/gui/accountsettings.cpp.
  • Update stylesheets to target the new syncFoldersPanelContents widget and adjust the grid row stretch in src/gui/accountsettings.ui so the layout grows/shrinks correctly.

Testing

  • No automated tests were run as part of this change.

Codex Task

@Rello
Rello merged commit daba653 into feature/settingsV2 Jan 20, 2026
8 of 15 checks passed
@Rello
Rello deleted the codex/make-sync-folder-box-dynamic-size branch January 20, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant