Hide code via urlParameter doesn't work anymore

[PLNech]

Hey amazing hydra team! I used to be able to hide code by default in my created works on hydra.ojack.xyz with &show-code=false.
I see that it doesn't work anymdraore in previous links I had shared, and indeed the code for that param seems commented:

hydra/src/views/editor/editor.js

Lines 72 to 79 in f2c53a6

	// this.show()
	// // // TO DO: add show code param
	// let searchParams = new URLSearchParams(window.location.search)
	// let showCode = searchParams.get('show-code')
	// if (showCode === "false") {
	// this.hide()
	// }

Do you know why? I'd love to have this feature again - Happy to brush off my web skills and do a little PR for hydra if you think it's welcome 🤗

[ojack]

it should work with 'showCode=false'!

we should make it work with 'show-code' also

[PLNech]

Ah nice, indeed this works! I'll use this for new creations 🤗

As for supporting the old syntax, good idea it would fix already shared links 😁

Thank you @ojack for the super quick reply 🙏

[ojack]

great! feel free to PR for supporting the old syntax if you would like :)

…

On Mon, Mar 4, 2024 at 3:29 PM Paul-Louis NECH ***@***.***> wrote: Ah nice, indeed this works! I'll use this for new creations 🤗 As for supporting the old syntax, good idea it would fix already shared links 😁 Thank you @ojack <https://github.com/ojack> for the super quick reply 🙏 — Reply to this email directly, view it on GitHub <#294 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABFOVSONFCPW7GZDRVYTMULYWSALDAVCNFSM6AAAAABEFHTMGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZWG4YDMMBRG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>