search

silverstripe / api.silverstripe.org

API documentation for the Silverstripe Framework
BSD 3-Clause "New" or "Revised" License
3 stars 14 forks source link

ENH try to stay on current page if switching versions #112

Closed andrewandante closed 11 months ago

andrewandante commented 12 months ago

Fixes #106

Swaps in the version at the front. Bit hacky but should work.

Caveat - haven't been able to test it locally.

GuySartorelli commented 11 months ago

Thank you for providing this solution.

That snippet of code is copy pasted directly from the upstream dependency code-lts/doctum and won't be retained when https://github.com/silverstripe/api.silverstripe.org/pull/108 is merged in. I'm going to close this PR on the basis that if it's fixed in doctum, that PR should resolve it - and if it's not fixed in doctum, that's where it should be fixed.