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

Add browser support policy for Scribble-generated documentation. #240

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

Conversation

samth
Copy link
Member

@samth samth commented Jun 1, 2020

The short summary: all evergreen browsers from the past year,
plus anything <5 years old with at least 1% share on docs.racket-lang.org.

Rendered


Features that are not needed for reading and understanding the
documentation, but that are intended for developers of the
documentation or other specialized audiences, such as links to
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"such as links to implementation" - is this the motivating use case for this pull request? (Just curious)

@sorawee
Copy link
Contributor

sorawee commented Jun 3, 2020

I think the requirement to have a minimal support for IE6 doesn't make much sense. IE6 by default can't even view sites with https protocol. Even when TLS1.0 is manually enabled, racket-lang.org (and most websites in the world) seems to be serving a content that IE6 doesn't understand, so it will fail with "The page cannot be displayed". This means that users who are still using it can't download Racket in the first place, so there's really no point to support it.

@sorawee
Copy link
Contributor

sorawee commented Jun 4, 2020

FYI: if anyone wants to test IE (including IE6), you can use VMs provided by Microsoft. Unfortunately, Microsoft took these VMs down from their website for unknown reasons, but some users archived them. Use them at your own risk.

As noted above, for IE6, we need to enable TLS1.0 to access https websites, but racket-lang.org can't be accessed regardless.

@samth samth self-assigned this Jul 2, 2020
sorawee added a commit to sorawee/scribble that referenced this pull request Sep 29, 2020
HTML supports the placeholder attribute, so we should use it
instead of implementing it ourselves.

The placeholder attribute is not available prior IE10 (not including
IE10). However:

1. In non-supporting browsers (e.g., IE9), the search functionality
should still work correctly.
2. Non-supporting browsers don't qualify for the "full support" level
detailed in racket#240
IE9 for instance is released in 2011, and Microsoft announced its end of
support in 2016.
mflatt pushed a commit that referenced this pull request Nov 6, 2020
HTML supports the placeholder attribute, so we should use it
instead of implementing it ourselves.

The placeholder attribute is not available prior IE10 (not including
IE10). However:

1. In non-supporting browsers (e.g., IE9), the search functionality
should still work correctly.
2. Non-supporting browsers don't qualify for the "full support" level
detailed in #240
IE9 for instance is released in 2011, and Microsoft announced its end of
support in 2016.
@LiberalArtist
Copy link
Contributor

FWIW, while I don't care about IE6, I do use lynx from time to time, and the docs are quite readable in it.

sorawee added a commit that referenced this pull request Oct 14, 2023
Browsers released within the last 5 years (the partial support tier
according to #240) support
URLSearchParams, so we should use it.
sorawee added a commit that referenced this pull request Oct 16, 2023
Browsers released within the last 5 years (the partial support tier
according to #240) support
URLSearchParams, so we should use it.
sorawee added a commit that referenced this pull request Oct 17, 2023
Browsers released within the last 5 years (the partial support tier
according to #240) support
URLSearchParams, so we should use it.
sorawee added a commit that referenced this pull request Oct 29, 2023
Browsers released within the last 5 years (the partial support tier
according to #240) support
URLSearchParams, so we should use it.
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.

4 participants