Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

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
samth wants to merge 1 commit into racket:master
base: master
Choose a base branch
Loading
from samth:browser-support

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

LiberalArtist reacted with thumbs up emoji

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

@jackfirth jackfirth Jun 3, 2020

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)

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.

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.
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

Reviewers

1 more reviewer

@jackfirth jackfirth jackfirth left review comments

Reviewers whose approvals may not affect merge requirements

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /