fix(color-contrast): do not incomplete on textarea

[straker]

Closes: #4947

[Garbee]

The description here has no content. We should probably at least have what the patch is doing and how it addresses the problem at a minimum for content. That way as we merge the commit history starts to contain some useful information when going through the history and bisecting problems.

[davidg-dq]

The description here has no content. We should probably at least have what the patch is doing and how it addresses the problem at a minimum for content. That way as we merge the commit history starts to contain some useful information when going through the history and bisecting problems.

Agreed. @straker please add a PR description here. All PRs need a description that tells the work that was done and anything that will help the reviewers.

[Garbee]

Drafting since Steve is out for a while on vacation.

[straker]

The description isn't needed if the issue it's closing details all the work, in this case #4947 (comment).

[Garbee]

A description is still needed. The point is once this is merged we take the PR title and description to make the commit body. Which is useful context to search in the repository history without needing to go elsewhere. When triaging things (particularly bisecting) having commit data that is meaningful to the changes is valuable.

[Garbee]

Also, the last comment in the issue is basically half of what we want in the PR description. The only extra thing the description needs is a minimal definition of what the original problem was.