Desktop: White background in recipient list

[tobiasKaminsky]

See white background in recipient list

[jancborchardt]

This is fixed in the pull request at https://github.com/nextcloud/desktop/pull/1429 :)

[tobiasKaminsky]

Not sure how you do this in other repos, but in Android we keep issues open until PR is merged (and in PR we use "fix #123123123" so that Github auto closes this).
This ensures that other reporters can more easily find the issue (as it is technically not yet solved).

[jancborchardt]

Yep, we normally do that too, but since https://github.com/nextcloud/desktop/pull/1429 does not have a reference to this here, I closed it so we don’t have to clean up afterwards. I should have mentioned that, sorry. :)

@DominiqueFuchs you probably know if you include "…, fix #1429" in a commit it closes the issue automatically on merge? :)

[tobiasKaminsky]

It does not necessarily have to be in a commit, you can just put in the PR description (it just has to be the first line(s))

[DominiqueFuchs]

@jancborchardt yeah I know about this, generally I list relating issues like that in the PR, then they are automatically resolved when merging - just didn‘t know about this one.
I‘ll fix the conflict blocking the PR merge within this week

[DominiqueFuchs]

FTR: Merged #1429 🎃