Skip to content

fix(mail): keep sender column tied to row folder#1831

Open
KaannKara wants to merge 2 commits into
runbox:masterfrom
KaannKara:fix-folder-from-to-columns
Open

fix(mail): keep sender column tied to row folder#1831
KaannKara wants to merge 2 commits into
runbox:masterfrom
KaannKara:fix-folder-from-to-columns

Conversation

@KaannKara
Copy link
Copy Markdown

Summary

  • Fixes Incorrect folder content when switching folders #1825 by choosing From/To values from each row's actual folder instead of the currently selected folder.
  • Covers both normal message lists and Xapian search results so stale rows do not briefly redraw with the wrong sender/recipient field while switching into or out of Sent folders.
  • Adds focused regression tests in separate test commit.

Validation

  • npx tsc -p src/tsconfig.spec.json --noEmit
  • npx eslint src/app/common/messagelist.spec.ts src/app/xapian/searchmessagedisplay.spec.ts
  • npm run lint (passes with existing warnings)
  • npm run build (passes with existing warnings)
  • npm run policy (exits 0; reports existing historical commit-message warnings)

Karma browser specs could not be executed in this environment because Firefox is not installed and ChromeHeadless is not registered in this repo's Karma config.

AI disclosure

This contribution was prepared with AI assistance and manually reviewed before submission.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

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.

Incorrect folder content when switching folders

1 participant