Skip to content

Focus on recipients when forwarding - #1340

Merged
sosnovsky merged 3 commits into
masterfrom
feature/issue-1085-focus
Jan 27, 2022
Merged

Focus on recipients when forwarding#1340
sosnovsky merged 3 commits into
masterfrom
feature/issue-1085-focus

Conversation

@Kharchevskyi

@KharchevskyiKharchevskyi commented Jan 26, 2022•
edited by sosnovsky
Loading

Copy link
Copy Markdown
Contributor

This PR apply focus when forwardinf

close#1085

Tests(delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

sosnovsky
sosnovsky requested changes Jan 26, 2022
View reviewed changes
Comment threadFlowCrypt/Controllers/Compose/ComposeViewControllerInput.swift Outdated
@Kharchevskyi
Kharchevskyi marked this pull request as draft January 26, 2022 20:50
@Kharchevskyi
Kharchevskyi marked this pull request as ready for review January 27, 2022 09:44
sosnovsky
sosnovsky requested changes Jan 27, 2022
View reviewed changes
Comment threadFlowCrypt/Controllers/Compose/ComposeViewControllerInput.swift Outdated
sosnovsky
sosnovsky approved these changes Jan 27, 2022
View reviewed changes

@sosnovskysosnovsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅

@sosnovsky
sosnovsky enabled auto-merge (squash) January 27, 2022 10:57
@sosnovsky
sosnovsky merged commit 8ae3d9b into masterJan 27, 2022
@sosnovsky
sosnovsky deleted the feature/issue-1085-focus branch January 27, 2022 16:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sosnovskysosnovskysosnovsky approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

focus on recipients when forwarding

2 participants

@Kharchevskyi@sosnovsky