Skip to content

Change encoding to UTF-8 for doc/styles.css #43

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jwakely
Copy link
Contributor

@jwakely jwakely commented Feb 3, 2022

CSS files should be UTF-8: https://www.w3.org/International/questions/qa-css-charset

boostorg/filesystem@9df2bfa#diff-36873c6a5a238ba4df45a47d97d80ead5ed664b3cb7e5a8fbe918020cc6889a4 replaced another ISO-8859-1 copyright symbol with (c), which would also be an option here. I just want to get rid of the ISO-8859-1 character in a file that doesn't specify its @charset, because it causes warnings for distro packagers.

See also boostorg/boost#618 and boostorg/multi_index#55 and boostorg/detail#22

@jwakely
Copy link
Contributor Author

jwakely commented Jun 14, 2022

Ping

Would you prefer to do this consistently with boostorg/boost#618 and boostorg/multi_index#55 and boostorg/detail#22 i.e. "Copyright (c) YYYY, Copyright Holder" ?

@jwakely
Copy link
Contributor Author

jwakely commented Jun 14, 2022

I've updated the PR that way now.

@e-kwsm
Copy link

e-kwsm commented Jan 22, 2025

FWIW: HTML files, e.g. index.html, are not UTF8.

@jwakely
Copy link
Contributor Author

jwakely commented Jan 22, 2025

FWIW: HTML files, e.g. index.html, are not UTF8.

The patch is for a CSS file, so I'm not sure what you're saying here, or if it matters to this PR.

@e-kwsm
Copy link

e-kwsm commented Jan 22, 2025

#61 supersedes this.

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