facebookarchive / react-360

Create amazing 360 and VR content using React
https://facebook.github.io/react-360
Other
8.73k stars 1.23k forks source link

Doc bug - shadow props link goes to getting started #324

Closed jgwinner closed 7 years ago

jgwinner commented 7 years ago

Description

Documentation bug, missing page https://facebook.github.io/react-vr/docs/shadow-props.html

Expected behavior

On the page: https://facebook.github.io/react-vr/docs/vrbutton.html if I click on "Shadow Props" the status bar indicates it links to: https://facebook.github.io/react-vr/docs/shadow-props.html#props

I would expect a page detailing the "Shadow Props"

Actual behavior

However, clicking on "Shadow Props" goes to the getting started page with Chrome; (probably the 404 or something), IE gives a real 404 page. Frankly, this is better, as I thought I clicked on the wrong thing.

"Fries with that?" "Sure" "Here's your Birthday cake." "Huh? Did I order cake?" "No dear, the server is confused."

Reproduction

  1. Go to https://facebook.github.io/react-vr/docs/vrbutton.html
  2. Scroll down the page past the props style?: style

Layout Props... Shadow Props... Transforms...

  1. Click on "Shadow Props..."

Solution

Find or write the shadow-props.html page.

Additional Information

amberroy commented 7 years ago

thanks for catching this.

mikearmstrong001 commented 7 years ago

IJust to loop back. Shadow props are not actually supported in react-vr so the solution is to remove this link. Were you looking at using them?

jgwinner commented 7 years ago

I assume this would map to the LightShadow class in three.js? Not sure how that would map to the VR Button. I was going through the documentation while I'm writing a book on ReactVR for Packt publication. So that's how I stumbled on the broken link.

I'd say remove it. I'll remove it from my draft if you do.

mikearmstrong001 commented 7 years ago

It isn't the LightShadow, it's actually a style property for views

It has been removed