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

Documentation pages are not displayed in Opera Mini #75458

Open
bodqhrohro opened this issue Aug 12, 2020 · 3 comments
Open

Documentation pages are not displayed in Opera Mini #75458

bodqhrohro opened this issue Aug 12, 2020 · 3 comments
Labels
C-bug Category: This is a bug. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@bodqhrohro
Copy link

Migrated from the www.rust-lang.org repository.

Page(s) Affected

For example, http://doc.rust-lang.org/std/vec/struct.Vec.html (lots of them actually, probably all).

What needs to be fixed?

It's just a typical case when Opera Mini transcoders try to execute JS, but do it partially and thus break the page at all :/
2020-08-12-152012_1366x768_scrot
2020-08-12-152041_1366x768_scrot
It's not even a layout problem, as the page is empty in Mobile View too.

Suggested Improvement

The documentation is almost plain HTML by its nature, unlike brand new web apps that need bleeding edge technologies, so I see no reasons to make it inaccessible in outdated browsers.

I see non-transpilled ES6 lambdas here that cause a syntax error, but that may be only a part of the problem.

As a last resort, if the root cause is hard to find, scripts may be disabled at all for Opera Mini by detecting the window.operamini object. That's an ugly quirk, but there already are IE detections in the code, so why not? :)

@bodqhrohro bodqhrohro added the C-bug Category: This is a bug. label Aug 12, 2020
@Nilstrieb Nilstrieb added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Jul 1, 2023
@Nilstrieb
Copy link
Member

There's https://rust-lang.github.io/rfcs/1985-tiered-browser-support.html, which does not cover opera mini. I'm not sure where the currernt policy is documented. cc @GuillaumeGomez

@GuillaumeGomez
Copy link
Member

This RFC is the latest browser support policy.

@bodqhrohro: You can disable JS in your web browser directly and it should work (hopefully). Rustdoc is working with JS disabled, but you won't have access to the search feature.

We could the window.operamini check, but since it's failing when parsing JS, it would very likely complexify the JS loading quite a lot for a web browser we don't support. And the IE detection you mentioned, it's only to display a message to the users, no JS involved in there.

@bodqhrohro
Copy link
Author

You can disable JS in your web browser directly

Opera Mini clients do not support this, JS is always executed on transcoders in a restricted manner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants