#418 Updated recipient selection color

[ioanmo226]

This PR updated recipient selection color

close #418 // if this PR closes an issue
close #2304 // if this PR closes an issue

---

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

[ioanmo226]

Let me know if you have better suggestion.

[sosnovsky]

It's still not very clear that recipient is selected - looks like just some another state, I would think that recipient wasn't selected and will try to select it by tapping.

What about just reducing opacity of selected recipient to 0.6:

Then it'll still be possible to see current state of recipient (key found, no key etc), but reduced opacity will show that it is currently selected. Plus I think we should remove typing cursor when recipient is selected, as it's not common to have something selected and active cursor simultaneously (you can check how it works in iOS Messages or Mail app)

[ioanmo226]

Cool thought! But here's the thing, to me, it feels like 100% opacity gives off a "selected" vibe, while 50% opacity seems more "unselected", What do you think?

[sosnovsky]

Cool thought! But here's the thing, to me, it feels like 100% opacity gives off a "selected" vibe, while 50% opacity seems more "unselected", What do you think?

Yeah, I agree on this but haven't come up with something better yet :) Probably it'll still be more clear than just switching recipient color to blue.
Among other ideas there were making font bold or changing border color and width, but not sure if it work well too.

[ioanmo226]

I agree. Let me proceed that way

[ioanmo226]

Setting opacity to 50% for keyNotFoundStateContext doesn't seem to work well. (As it already has alpha value of 0.1, doesn't have much visual difference even we set it to 0.05)
Is this fine?

[sosnovsky]

What about increasing alpha for keyNotFoundStateContext? For example to 0.3 or 0.4

[ioanmo226]

Ok

[ioanmo226]

@sosnovsky Ready for review

[sosnovsky]

Works well now 👍