Skip to content

Adds argument to doxygengroup to display just the description.#718

Merged
michaeljones merged 1 commit intobreathe-doc:masterfrom
intelkevinputnam:kp-add-description-only
Feb 14, 2022
Merged

Adds argument to doxygengroup to display just the description.#718
michaeljones merged 1 commit intobreathe-doc:masterfrom
intelkevinputnam:kp-add-description-only

Conversation

@intelkevinputnam
Copy link
Copy Markdown
Contributor

@intelkevinputnam intelkevinputnam commented Jul 27, 2021

Any doxygen directive that uses the _DoxygenContentBlockDirective can now display just the description, like so:

.. doxygengroup:: GROUPNAME
   :description-only:

I frequently build up groups from their components, so its useful to be able to display just the description.

Signed-off-by: Kevin Putnam [email protected]

Copy link
Copy Markdown
Collaborator

@vermeeren vermeeren left a comment

Choose a reason for hiding this comment

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

@intelkevinputnam Could you also update documentation for doxygengroup and doxygennamespace directives?

  • documentation/source/directives.rst (arg only)
  • documentation/source/group.rst (arg + example)
  • documentation/source/namespace.rst (arg + example)

Doc also serves as tests and catching regressions via comparisons. Patch itself seems good to me.

Thanks!

Edit: You probably also want to rebase on master so CI works again.

@intelkevinputnam intelkevinputnam force-pushed the kp-add-description-only branch 2 times, most recently from f30078b to 2ae05ae Compare December 10, 2021 19:16
@vermeeren vermeeren force-pushed the kp-add-description-only branch from 2ae05ae to 22e7138 Compare February 14, 2022 00:03
@vermeeren vermeeren force-pushed the kp-add-description-only branch from 22e7138 to 901ad06 Compare February 14, 2022 00:04
Copy link
Copy Markdown
Collaborator

@vermeeren vermeeren left a comment

Choose a reason for hiding this comment

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

@intelkevinputnam Apologies for the long wait, the changes you made are great, many thanks for those!

Merging now.

michaeljones pushed a commit that referenced this pull request Feb 14, 2022
@michaeljones michaeljones merged commit 5162461 into breathe-doc:master Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code Source code enhancement Improvements, additions (also cosmetics)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to display just the description for any directive based on _DoxygenContentBlockDirective

3 participants