Skip to content

fix(ui): show favorites section based on the current folder#13221

Merged
ChristophWurst merged 1 commit into
mainfrom
fix/ui/favorites-section-visibility
Jul 6, 2026
Merged

fix(ui): show favorites section based on the current folder#13221
ChristophWurst merged 1 commit into
mainfrom
fix/ui/favorites-section-visibility

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

The visibility of the favorites section was driven by the unified inbox's starred envelopes for regular folders too, while the section itself renders the current folder's starred messages. Folders with starred mail hid the section until something populated the unified inbox list, and folders without starred mail showed an empty section once it was populated.

Assisted-by: Claude:claude-fable-5

Realization from #12953 (comment).

How to test

  1. Have a mailbox with favs
  2. Enable the option to pin favs
  3. Open the mailbox (NOT unified inbox!)
  4. Reload the page

main: no favs on top unless you navigate to the unified inbox and back (assuming you have favs there)
here: favs are shown on top

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

The visibility of the favorites section was driven by the unified
inbox's starred envelopes for regular folders too, while the section
itself renders the current folder's starred messages. Folders with
starred mail hid the section until something populated the unified
inbox list, and folders without starred mail showed an empty section
once it was populated.

Assisted-by: Claude:claude-fable-5
Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@hamza221

hamza221 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

/backport to stable5.10

@ChristophWurst ChristophWurst merged commit a240cdf into main Jul 6, 2026
42 checks passed
@ChristophWurst ChristophWurst deleted the fix/ui/favorites-section-visibility branch July 6, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants