Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TestDox HTML report not displayed correctly when browser has custom colour settings #4772

Merged
merged 1 commit into from Sep 6, 2021

Conversation

DanielSiepmann
Copy link
Contributor

Browsers support custom colour settings.
Some people might change default text and background colour.

The report defines custom colour for successful and failed tests.
It should therefore ensure that those colours work as expected.
Background colour and default text colour are set in addition.
This ensures an expected end result, no matter what colours might be set as defaults.

@DanielSiepmann
Copy link
Contributor Author

Example before:
image
after:
image

@sebastianbergmann
Copy link
Owner

Please send this against the 8.5 branch. Thanks!

@DanielSiepmann DanielSiepmann changed the base branch from master to 8.5 September 6, 2021 07:01
@DanielSiepmann
Copy link
Contributor Author

Done. Will try to use the oldest available branch in the future.

@DanielSiepmann DanielSiepmann marked this pull request as draft September 6, 2021 07:19
Browsers support custom colour settings.
Some people might change default text and background colour.

The report defines custom colour for successful and failed tests.
It should therefore ensure that those colours work as expected.
Background colour and default text colour are set in addition.
This ensures an expected end result, no matter what colours might be set as defaults.
@DanielSiepmann DanielSiepmann marked this pull request as ready for review September 6, 2021 07:21
@sebastianbergmann sebastianbergmann added feature/logging Issues related to logging test results type/bug Something is broken labels Sep 6, 2021
@sebastianbergmann sebastianbergmann changed the title Improve Testdox HTML styling on custom browser settings TestDox HTML report not displayed correctly when browser has custom colour settings Sep 6, 2021
@sebastianbergmann sebastianbergmann merged commit f24f66b into sebastianbergmann:8.5 Sep 6, 2021
@DanielSiepmann DanielSiepmann deleted the patch-1 branch September 6, 2021 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/logging Issues related to logging test results type/bug Something is broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants