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.

Sign up for GitHub

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

Jump to bottom

[DOC]: Sphinx theme doesn't expand header backgrounds for long headers #543

Closed
2 tasks done
dagardner-nv opened this issue Dec 13, 2022 · 2 comments
Closed
2 tasks done

[DOC]: Sphinx theme doesn't expand header backgrounds for long headers #543

dagardner-nv opened this issue Dec 13, 2022 · 2 comments
Assignees
@dagardner-nv
Labels
doc Improvements or additions to documentation
Milestone
23.01 Documentati...

Comments

@dagardner-nv
Copy link
Contributor

How would you describe the priority of this documentation request

Low (would be nice)

Please provide a link or source to the relevant docs

https://docs.nvidia.com/morpheus/_modules/morpheus.messages.multi_inference_ae_message.MultiInferenceAEMessage.html

Describe the problems in the documentation

For long headers, which are common in the API build where the H1 is the fully qualified Python name, the black background doesn't extend horizontally to match the content.

ex the API docs for morpheus.messages.multi_inference_ae_message.MultiInferenceAEMessage
image

(Optional) Propose a correction

Update the CSS for our Sphinx theme

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • I have searched the open documentation issues and have found no duplicates for this bug report
@dagardner-nv dagardner-nv added the doc Improvements or additions to documentation label Dec 13, 2022
@dagardner-nv
Copy link
Contributor Author

This is coming from docs/source/_static/omni-style.css line 30:

.wy-nav-content {
    max-width: 1200px !important;
}

dagardner-nv added a commit to dagardner-nv/Morpheus that referenced this issue Dec 14, 2022
@dagardner-nv
Replace absolute value with a relative percentage value fixes nv-morp…
b9cef71 
…heus#543
@dagardner-nv dagardner-nv mentioned this issue Dec 14, 2022
Merged
@dagardner-nv dagardner-nv self-assigned this Dec 14, 2022
ghost pushed a commit that referenced this issue Dec 16, 2022
@dagardner-nv
Misc documentation fixes (#547)
987e6bf 
* Includes changes from PR #538 
* Fixes casing for proper nouns (NVIDIA, Docker, Triton, Python, Conda) when not referring to a command
* Document `StreamPair` (docstrings for globals need to appear after the definition)
* Other fixes suggested by Zenobia
* Brings grade up from an A to A+
* Fixes CSS theme issue #543 
* Various fixes to the `getting_started.md` doc fixes #539
* Fixes issue with the `docker/run_container_*.sh` scripts for users who have the Nvidia Container Toolkit installed but does not have nvidia set as the default runtime
* Documents launching a pre-built Morpheus container
* Fix errant entry in docstring for `AppShieldSource` which was showing up in the command line help
* Documentation work-around for #475
* Fix a few remaining references to 'srf'

Authors:
  - David Gardner (https://github.com/dagardner-nv)
  - https://github.com/bsuryadev
  - Bhargav Suryadevara (https://github.com/bsuryadevara)

Approvers:
  - Christopher Harris (https://github.com/cwharris)
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #547
@dagardner-nv
Copy link
Contributor Author

Fixed in #547

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dagardner-nv dagardner-nv

Labels
doc Improvements or additions to documentation
Projects
Morpheus Boards
Status: Done
Milestone
23.01 Documentation Updates
Development

No branches or pull requests
1 participant
@dagardner-nv