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

Fix inline styles 2 #5657

Merged
merged 11 commits into from
Jun 4, 2021
Merged

Fix inline styles 2 #5657

merged 11 commits into from
Jun 4, 2021

Conversation

hamishwillee
Copy link
Collaborator

Part of fixing #5438

<figcaption><strong>The coordinate system for the right hand's grip space.</strong></figcaption>
<strong>Figure: The coordinate system for the left hand's grip space.</strong>

<img width="600px" alt="A diagram showing how the grip space indicates the local coordinate system for the player's hand relative to the world." src="dark_left.svg">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These images are SVG so they expand full screen,, which looks ridiculous. I have capped size here explicitly but we would be wise to have some infrastructure in yari to pin their size.

Thoughts?

Copy link
Member

Choose a reason for hiding this comment

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

These images are SVG so they expand full screen,, which looks ridiculous. I have capped size here explicitly but we would be wise to have some infrastructure in yari to pin their size.

Thoughts?

I personally have no insights to offer on how to solve that — but I agree about the problem

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks very much. Me either. Let's see what others think mdn/yari#3970

@github-actions
Copy link
Contributor

github-actions bot commented Jun 4, 2021

Preview URLs

Flaws

Note! 4 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/WebVR_API/Using_the_WebVR_API
Title: Using the WebVR API
on GitHub
Flaw count: 1

  • macros:
    • wrong xref macro used (consider changing which macro you use)

URL: /en-US/docs/Web/API/WebXR_Device_API
Title: WebXR Device API
on GitHub
Flaw count: 6

  • macros:
    • /en-US/docs/Web/API/WebGLRenderingContextBase/makeXRCompatibile does not exist
    • /en-US/docs/Web/API/WebGLContextAttributes/xrCompatible does not exist
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/WebXR_Device_API/Input_profiles
    • Can't resolve /en-US/docs/Web/WebXR_Device_API/Gamepads
    • Can't resolve /en-US/docs/Web/Media/3D_audio
    • Can't resolve /en-US/docs/Web/Media/3D_video

URL: /en-US/docs/Web/API/WebXR_Device_API/Movement_and_motion
Title: Movement, orientation, and motion: A WebXR example
on GitHub
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/XRWebGLLayer/viewport does not exist

URL: /en-US/docs/Web/API/WebXR_Device_API/Inputs
Title: Inputs and input sources
on GitHub
Flaw count: 16

  • macros:
    • /en-US/docs/Web/API/XRInputSource/gamepad does not exist
    • /en-US/docs/Web/API/XRInputSource/gamepad does not exist
    • /en-US/docs/Web/API/XRInputSourceChangeEvent does not exist
    • /en-US/docs/Web/API/XRInputSourceChangeEvent/session does not exist
    • /en-US/docs/Web/API/XRInputSourceChangeEvent/added does not exist
    • and 6 more flaws omitted
  • broken_links:
    • Can't resolve /en-US/docs/Web/WebXR_Device_API/Gamepads
    • Can't resolve /en-US/docs/Web/API/WebXR_Device_API/Input_profiles
    • Can't resolve /en-US/docs/Web/WebXR_Device_API/Gamepads
    • Can't resolve /en-US/docs/Web/API/WebXR_Device_API/Input_profiles
    • Can't resolve /en-US/docs/Web/WebXR_Device_API/Gamepads

URL: /en-US/docs/Web/API/WebXR_Device_API/Fundamentals
Title: Fundamentals of WebXR
on GitHub
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/API/WebXR_Device_API/Augmented_reality

URL: /en-US/docs/Web/API/WebVTT_API
Title: Web Video Text Tracks Format (WebVTT)
on GitHub
Flaw count: 23

  • macros:
    • /en-US/docs/Web/API/VTTCue/align does not exist
  • bad_bcd_links:
    • no explanation!
    • no explanation!
    • no explanation!
    • no explanation!
    • no explanation!
    • and 17 more flaws omitted

URL: /en-US/docs/Web/API/WebSockets_API/Writing_WebSocket_servers
Title: Writing WebSocket servers
on GitHub
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/JavaScript/Server-Side_JavaScript

External URLs

URL: /en-US/docs/Web/API/WebVR_API
Title: WebVR API
on GitHub


URL: /en-US/docs/Web/API/WebVR_API/Using_the_WebVR_API
Title: Using the WebVR API
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebXR_Device_API
Title: WebXR Device API
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebXR_Device_API/Movement_and_motion
Title: Movement, orientation, and motion: A WebXR example
on GitHub


URL: /en-US/docs/Web/API/WebXR_Device_API/Cameras
Title: Viewpoints and viewers: Simulating cameras in WebXR
on GitHub


URL: /en-US/docs/Web/API/WebXR_Device_API/Lighting
Title: Lighting a WebXR setting
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebXR_Device_API/Inputs
Title: Inputs and input sources
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebXR_Device_API/Rendering
Title: Rendering and the WebXR frame animation callback
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebXR_Device_API/Fundamentals
Title: Fundamentals of WebXR
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebVTT_API
Title: Web Video Text Tracks Format (WebVTT)
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebSockets_API/Writing_WebSocket_servers
Title: Writing WebSocket servers
on GitHub

(this comment was updated 2021-06-04 04:38:44.622972)

@hamishwillee hamishwillee marked this pull request as ready for review June 4, 2021 04:37
@hamishwillee hamishwillee requested a review from a team as a code owner June 4, 2021 04:37
@hamishwillee hamishwillee requested review from sideshowbarker and wbamberg and removed request for a team June 4, 2021 04:37
@sideshowbarker sideshowbarker merged commit 3e8265d into mdn:main Jun 4, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants