Skip to content

Commit

Permalink
Feature co2708 cake5 UI fixes (#71)
Browse files Browse the repository at this point in the history
* Fix authentication test for CSS body classes after Cake5 upgrade (CO-2708)

* Hide animated spinner on pagehide (CO-2708)
  • Loading branch information
arlen authored Nov 13, 2025
1 parent e2abf7f commit 6630e11
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
6 changes: 1 addition & 5 deletions app/templates/layout/default.php
Original file line number Diff line number Diff line change
Expand Up @@ -71,11 +71,7 @@
$bodyClasses = $controller_stripped . ' ' .$action_stripped;

// add further body classes as needed
if($this->getRequest()->getSession()->check('Auth.User') != NULL) {
$bodyClasses .= ' logged-in';
} else {
$bodyClasses .= ' logged-out';
}
$bodyClasses .= !empty($vv_user) ? ' logged-in' : ' logged-out';
?>
<body class="<?= $bodyClasses ?>">
<div id="skip-to-content-box">
Expand Down
6 changes: 6 additions & 0 deletions app/webroot/js/comanage/match-onload.js
Original file line number Diff line number Diff line change
Expand Up @@ -187,4 +187,10 @@ $(function() {
}

});

// Dismiss the loading animation on pagehide so that browser caching (specifically Chrome)
// will not retain it in the DOM when a user hits the back-button.
$(window).on('pagehide', function() {
stopSpinner();
});
});

0 comments on commit 6630e11

Please sign in to comment.