mirror of
https://github.com/Foundry376/Mailspring.git
synced 2025-02-21 22:54:11 +08:00
c265cf0dfa
Summary: This diff implements a behavior change described in https://github.com/nylas/N1/issues/1722. Reply buttons should prefer to focus an existing draft in reply to the same message, if one is pristine, altering it as necessary to switch between reply / reply-all. If no pristine reply is already there, it creates one. Reply keyboard shortcuts should do the same, but more strictly - the shortcuts should switch between reply / reply-all for an existing draft regardless of whether it's pristine. This diff also cleans up the DraftStore and moves all the draft creation itself to a new DraftFactory object. This makes it much easier to see what's going on in the DraftStore, and I also refactored away the "newMessageWithContext" method, which was breaking the logic for Reply vs Forward between a bunch of different helper methods and was hard to follow. Test Plan: They're all wrecked. Will fix after concept is greenlighted Reviewers: evan, juan Reviewed By: juan Differential Revision: https://phab.nylas.com/D2776 |
||
|---|---|---|
| .. | ||
| components | ||
| extensions | ||
| fixtures | ||
| models | ||
| services | ||
| stores | ||
| tasks | ||
| action-bridge-spec.coffee | ||
| auto-update-manager-spec.coffee | ||
| buffered-process-spec.coffee | ||
| clipboard-spec.coffee | ||
| component-registry-spec.coffee | ||
| database-object-registry-spec.coffee | ||
| date-utils-spec.es6 | ||
| dom-utils-spec.coffee | ||
| extension-registry-spec.coffee | ||
| jasmine-helper.coffee | ||
| jasmine-jquery.js | ||
| jasmine.js | ||
| launch-services-spec.coffee | ||
| list-selection-spec.coffee | ||
| mail-rules-processor-spec.coffee | ||
| mailbox-perspective-spec.es6 | ||
| menu-manager-spec.coffee | ||
| module-cache-spec.coffee | ||
| n1-spec-reporter.coffee | ||
| nylas-api-spec.coffee | ||
| nylas-env-spec.coffee | ||
| nylas-protocol-handler-spec.coffee | ||
| nylas-test-utils.coffee | ||
| package-manager-spec.coffee | ||
| package-spec.coffee | ||
| quoted-html-transformer-spec.coffee | ||
| quoted-plain-text-transformer-spec.coffee | ||
| spec-bootstrap.coffee | ||
| spec-helper-platform.coffee | ||
| spec-helper.coffee | ||
| spec-suite.coffee | ||
| spellchecker-spec.coffee | ||
| style-manager-spec.coffee | ||
| styles-element-spec.coffee | ||
| task-factory-spec.es6 | ||
| theme-manager-spec.coffee | ||
| time-override.coffee | ||
| time-reporter.coffee | ||
| undo-manager-spec.coffee | ||
| utils-spec.coffee | ||