Skip to content

[docs] remove method toc icons#23004

Merged
ZacBlanco merged 1 commit intoprestodb:masterfrom
ZacBlanco:upstream-docs-remove-toc-icons
Jun 14, 2024
Merged

[docs] remove method toc icons#23004
ZacBlanco merged 1 commit intoprestodb:masterfrom
ZacBlanco:upstream-docs-remove-toc-icons

Conversation

@ZacBlanco
Copy link
Contributor

@ZacBlanco ZacBlanco commented Jun 13, 2024

Description

Removes TOC icons from function definitions

Motivation and Context

See #22887 (comment)

Impact

Slight visual adjustment to page-level table of contents

Test Plan

N/A

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@ZacBlanco
Copy link
Contributor Author

@steveburnett try this one out locally and let me know if it works. Also, if there's anything else different with the new theme that bugs you, let me know and we can try to bundle the changes into this PR

@steveburnett
Copy link
Contributor

@steveburnett try this one out locally and let me know if it works. Also, if there's anything else different with the new theme that bugs you, let me know and we can try to bundle the changes into this PR

Perfection! No other issues.

Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

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

LGTM! (docs)

Pull branch, new docs build, looks great. Thanks for dealing with this so quickly!

@ZacBlanco ZacBlanco marked this pull request as ready for review June 14, 2024 00:54
@ZacBlanco ZacBlanco requested a review from a team as a code owner June 14, 2024 00:54
@ZacBlanco ZacBlanco requested a review from presto-oss June 14, 2024 00:54
@ZacBlanco ZacBlanco merged commit 7f8ff95 into prestodb:master Jun 14, 2024
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