Skip to content

Add meta charset to documentation layout to fix visual regressions tests#361

Merged
aduth merged 2 commits intomainfrom
aduth-visual-regression-failure
Jun 16, 2023
Merged

Add meta charset to documentation layout to fix visual regressions tests#361
aduth merged 2 commits intomainfrom
aduth-visual-regression-failure

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Jun 16, 2023

There have been some visual regression failures on recent pull requests related to content encoding (see #359 (comment)). Based on my understanding, it doesn't seem like this tag should be necessary, as utf-8 should already be the default for HTML5 documents. But it does appear to have a positive effect, where builds appear to be passing again (after 4 retries).

@aduth aduth requested a review from a team June 16, 2023 13:55
@jmdembe
Copy link
Contributor

jmdembe commented Jun 16, 2023

Looking around, it seems likeutf-8 is should always be declared, so that could explain the regression.

@aduth aduth merged commit 00dacfd into main Jun 16, 2023
@aduth aduth deleted the aduth-visual-regression-failure branch June 16, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants