-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
file fails to render if extension is not lowercase #536
Comments
Apparently, asciidoctor-browser-extension/app/manifest.json Lines 18 to 49 in 2c1e96e
According to https://stackoverflow.com/a/18143014/3600979 we would need to add every variation: {
"matches": [
"*://*/*.asc",
"*://*/*.Asc",
"*://*/*.aSc",
"*://*/*.asC",
"*://*/*.ASc",
"*://*/*.ASC",
"*://*/*.aSC",
"*://*/*.AsC",
// ... and so on
]
} Please note that the recommended file extension is |
I recommend against this change. A file extension is case sensitive, and the only official file extension for AsciiDoc moving forward is .adoc. That's what's going to be standardized. And it's going to be lowercase. |
A file extension can be case-sensitive, but with certain file system formats (e.g., the default Mac OS Extended (Journaled), which is broadly agnostic toward case; or the classic DOS 8.3 format, which is uppercase only), it very well might not be. I used |
I was referring to the AsciiDoc file extension. |
So in cases where a workflow, like mine, requires that a document pass through:
...and bounce back and forth between those... Would it not then make the most sense to allow some degree of case insensitivity—at the very least to allow a typical FILENAME.ASC (which all three systems would agree is the same file) to be readable by this browser extension? |
After giving it some thought, I don't think we should support uppercase file extension. As mentioned by Dan, the only official file extension for AsciiDoc moving forward is In other words, I recommend using |
this should be case-insensitive, no? But, for example,
filename.ASC
will not render, while
filename.asc
will render.
The text was updated successfully, but these errors were encountered: