@lightningjs/blits 1.23.1 → 1.23.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v1.23.2
+
+_18 mar 2025_
+
+- Fixed issue with keepAlive pages remaining in cache after navigating back
+
 ## v1.23.1
 
 _17 mar 2025_

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightningjs/blits",
-  "version": "1.23.1",
+  "version": "1.23.2",
   "description": "Blits: The Lightning 3 App Development Framework",
   "bin": "bin/index.js",
   "exports": {

package/src/router/router.js

@@ -328,15 +328,19 @@ const removeView = async (route, view, transition) => {
   }
 
   // cache the page when it's as 'keepAlive' instead of destroying
-  if (route.options && route.options.keepAlive === true && navigatingBack === false) {
+  if (navigatingBack === false && route.options && route.options.keepAlive === true) {
     cacheMap.set(route.hash, { view: view, focus: previousFocus })
   } else if (navigatingBack === true) {
     // remove the previous route from the cache when navigating back
     // cacheMap.delete will not throw an error if the route is not in the cache
     cacheMap.delete(route.hash)
   }
-
-  if (route.options && route.options.keepAlive !== true) {
+  /* Destroy the view in the following cases:
+   * 1. Navigating forward, and the previous route is not configured with "keep alive" set to true.
+   * 2. Navigating back, and the previous route is configured with "keep alive" set to true.
+   * 3. Navigating back, and the previous route is not configured with "keep alive" set to true.
+   */
+  if (route.options && (route.options.keepAlive !== true || navigatingBack === true)) {
     view.destroy()
     view = null
   }