- Issue created by @kunalkursija
- ๐ซ๐ฎFinland lauriii Finland
Thank you for picking this up @shyam_bhatt! I left some feedback on the MR
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
This probably should have the same behavior as core's Toolbar:
core/modules/toolbar/js/escapeAdmin.js
.If not the same behavior, then it should be documented explicitly why this behaves differently compared to how that core functionality works.
- ๐ซ๐ฎFinland lauriii Finland
This probably should have the same behavior as core's Toolbar: core/modules/toolbar/js/escapeAdmin.js.
The behavior cannot be exactly the same as
escapeAdmin.js
because the admin UI could be linking to XB, and in that case we'd want to link user back to the page in the admin UI that they used to navigate to XB. Other than that, it could be the same as what's in Toolbar. - ๐ซ๐ฎFinland lauriii Finland
Assigning to @wim leers because there's an open question to him in the MR.
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
Working on eliminating reliance on the Referer header for "Back" link functionality
Whatโs been done:
- Track Navigation: Iโve written the script to capture the current pageโs URL on every page load.
- Store in Session Storage: The captured URL is then saved to the sessionStorage to track the user's navigation within the same session.
- Trigger on Load: The script is triggered by default on page load to ensure that the URL is captured and store
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
Thanks, @parthbcharya! ๐๐
Both the CSS and JS are causing CI jobs to fail because they don't meet the coding standards. Should be a trivial fix compared to everything else you've done! ๐