-
-
Notifications
You must be signed in to change notification settings - Fork 201
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
Render pages content in order #651
Render pages content in order #651
Conversation
cc: @utzig |
ed79525
to
6768147
Compare
When rendering Doxygen pages one expects content to appear in the written order. As of today `para` sections content was grouped separately depending on its type, causing out-of-order content in some pages. Signed-off-by: Gerard Marull-Paretas <[email protected]>
6768147
to
a46be23
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@utzig @vermeeren can you review? thanks |
@michaeljones can you review? thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gmarull Looks good, thanks!
@vermeeren @michaeljones would you mind creating a new release? Thanks! |
@gmarull v4.28.0 is now released containing this PR. |
When rendering Doxygen pages one expects content to appear in the
written order. As of today
para
sections content was groupedseparately depending on its type, causing out-of-order content in some
pages.
Fixes #648