404: move assett injector css

[protitude]

Pulling in 404 css

[Copilot]

Pull request overview

This PR updates the custom Drupal 404 page so its assets (CSS/JS) are properly attached via the oit/oit_404 library and the controller returns a render array (instead of pre-rendered markup), which helps preserve attachments and render metadata.

Changes:

• Return the 404 page as a render array from OitController::oit404() instead of flattening it into #markup.
• Add core/jquery as an explicit dependency of the oit_404 library (used by js/404.js).
• Scope most 404 CSS rules under #containAll404 to reduce accidental styling bleed.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/Controller/OitController.php	Returns the 404 inline template as a render array to preserve attachments/metadata.
oit.libraries.yml	Adds explicit core/jquery dependency to the oit_404 library.
css/404.css	Adds file header docs and scopes selectors under #containAll404 for 404-only styling.

Suppressed comments (1)

css/404.css:168

• The file header notes scoping to avoid style leakage, but the global .breadcrumb rule will still hide breadcrumbs anywhere this library is attached. If the intent is for this to be 404-only, scope the selector to the 404 path/body class so the rule can’t affect other pages.

/* PAGE HIDES */
.breadcrumb {
  display: none;
}

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.