Skip to content

API Reference Fix for Right Margin#1558

Merged
tonybove-apple merged 1 commit intoapple:mainfrom
tonybove-apple:api-ref-fix-right-margin
Jul 26, 2022
Merged

API Reference Fix for Right Margin#1558
tonybove-apple merged 1 commit intoapple:mainfrom
tonybove-apple:api-ref-fix-right-margin

Conversation

@tonybove-apple
Copy link
Copy Markdown
Contributor

This PR changes the API Reference documentation to remove the right margin so that text and examples don't appear to be extending beyond it. See attached figures for examples.

  • _static/css/norightmargin.css: Overrides the right margin setting to set the maximum width to none. As a result, the right margin changes depending on the side of the window, and text that is supposed to wrap continues to wrap correctly. See attached figures for examples.

  • conf.py: Add the html_static_path and html_css_files options. This enables the css/norightmargin.css CSS override.

Figures:

Current #1: Converters > Unified: Right margin in Parameters section.
API-ref-right-margin-currrent1

Fixed #1: Converters > Unified
API-ref-right-margin-fixed1

Current #2: Models > compression_utils > palettize_weights: Right margin in Parameters section.
API-ref-right-margin-currrent2

Fixed #2: Models > compression_utils > palettize_weights
API-ref-right-margin-fixed2

Current #3: Mil Ops > tensor_transformation: Right margin for some class headers.
API-ref-right-margin-currrent3

Fixed #3 Mil Ops > tensor_transformation
API-ref-right-margin-fixed3

The next PR provides the generated HTML.

@tonybove-apple tonybove-apple added the docs Contains documentation or clarification regarding docs (type) label Jul 25, 2022
@tonybove-apple tonybove-apple self-assigned this Jul 25, 2022
@tonybove-apple tonybove-apple merged commit 75e1d28 into apple:main Jul 26, 2022
@tonybove-apple tonybove-apple deleted the api-ref-fix-right-margin branch July 26, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Contains documentation or clarification regarding docs (type)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants