Skip to content

feat(compose): move recipients between fields - #2000

Open
isaacsamual994-lang wants to merge 5 commits into
runbox:masterfrom
isaacsamual994-lang:feat/132-drag-recipients
Open

feat(compose): move recipients between fields#2000
isaacsamual994-lang wants to merge 5 commits into
runbox:masterfrom
isaacsamual994-lang:feat/132-drag-recipients

Conversation

@isaacsamual994-lang

Copy link
Copy Markdown

Summary

  • make existing To/CC/BCC recipient chips draggable
  • preserve the source field during drag so dropping moves the recipient instead of copying it
  • avoid duplicate recipients when dropping onto the same or target field

Closes #132.

Testing

  • Added regression tests before the implementation; both failed before the fix and pass afterward.
  • Targeted recipient drag/drop tests: 2 passed.
  • Full unit suite: 259 passed (2 skipped).
  • ng lint runbox7: 0 errors (existing repository warnings remain).
  • Production build: passed.

I also ran the repository CI wrapper locally. Its Cypress phase could not start Cypress 12.17.4 on the local darwin-arm64 environment (Cypress.app: bad option: --no-sandbox); the unit phase completed successfully before that environment-specific launcher failure.

Copy link
Copy Markdown
Author

Implementation is ready for review. I re-verified the current PR branch locally: the targeted recipient drag/drop regression tests pass (2/2), ng lint runbox7 completes with 0 errors (existing repository warnings only), and the production build passes. GitHub currently shows both workflows as awaiting maintainer approval and the PR as requiring one approving review. Could a maintainer please approve the workflows and review the PR when convenient? Also, could you confirm whether #132 is eligible for the Runbox 7 Feature Bounty Program and which reward tier applies?

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.

#Mail :Support drag-and-drop of recipients between fields in Compose

1 participant