Skip to content

Conversation

@johnfmorton
Copy link
Contributor

This merge request introduces two updates to the Craft User Manual plugin:

  1. Default Entry Selection Respects Structure Order
  • Previously, when a section was defined as a Structure, the “default” manual entry was chosen based on creation date. As a result, the oldest entry appeared even if it wasn’t intended to be first.
  • The removal of the 'orderBy' => 'dateCreated ASC' allows the default entry to show the expected entry when the section is a Structure. If the section is not a Structure, it falls back to newest by creation date. (Previously, it was oldest by creation date.)
  1. Fix Developer Helper Notification in Default Template
  • In the default template for rendering manual content, a conditional block was preventing the developer helper notification from appearing.
  • That conditional has been corrected so that the helper notification (as previously defined in the template) now displays as intended.

TLDR:

  • The manual opens on the entry specified by the Structure rather than arbitrary creation date.
  • Developers see the helper notification without having to modify the template manually.

FYI: I've added notes in the CHANGELOG.md for these updates, but the actual version number will need to be updated if you accept these changes.

@johnfmorton johnfmorton closed this Jun 4, 2025
@johnfmorton johnfmorton deleted the dev-main branch June 4, 2025 13:21
@johnfmorton johnfmorton restored the dev-main branch June 4, 2025 13:21
@johnfmorton johnfmorton deleted the dev-main branch June 4, 2025 13:21
@johnfmorton
Copy link
Contributor Author

Looking into a new issue. Sorry for the trigger finger merge request.

@johnfmorton johnfmorton restored the dev-main branch June 4, 2025 13:25
@johnfmorton
Copy link
Contributor Author

Reopening after confirming the possible bug. It was not a bug after all. Thanks. -John

@johnfmorton johnfmorton reopened this Jun 4, 2025
@johnfmorton
Copy link
Contributor Author

After submitting the initial pull request, I noticed another user asked about support for a custom control panel trigger. I've added support for that as well.

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.

1 participant