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

examples: fix dark theme on empty location hash #21932

Merged
merged 1 commit into from
Jun 2, 2021

Conversation

felixmariotto
Copy link
Contributor

following #21929

Accessing https://threejs.org/examples with a browser with dark theme preference lead to the following output, whereas the body behind the empty iframe has a nice dark theme.

120200460-a521d080-c224-11eb-8f5c-718111a092a5

Commit demo ( depends on your browser settings ) : https://raw.githack.com/felixmariotto/three.js/dark_examples/examples/

@Mugen87 I have not been able to understand why the iframe was transparent when testing locally... most susprinsing.

@mrdoob mrdoob added this to the r130 milestone Jun 2, 2021
@mrdoob mrdoob merged commit e4db023 into mrdoob:dev Jun 2, 2021
@mrdoob
Copy link
Owner

mrdoob commented Jun 2, 2021

Thanks!

mrdoob pushed a commit that referenced this pull request Jun 2, 2021
@@ -383,6 +383,7 @@ iframe {
width: 100%;
height: 100%;
overflow: auto;
display: none;
Copy link
Owner

Choose a reason for hiding this comment

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

Heh... This made all the projects in https://threejs.org/ disappear 😅

Copy link
Owner

Choose a reason for hiding this comment

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

I've reverted this line for now: 4f4c845

Would display: unset; work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mrdoob I can't find the html file of the home page, it's not in the repo ?

We could replace this line with the following, since #viewer in the homepage is a div :

iframe#viewer {
	display: none;
}

Copy link
Owner

Choose a reason for hiding this comment

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

@mrdoob I can't find the html file of the home page, it's not in the repo ?

It's in the gh-pages branch.

We could replace this line with the following, since #viewer in the homepage is a div :

Sounds good!

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