Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GMBUVP-134: Add BackButton component #72

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Conversation

rastislav-chynoransky
Copy link
Contributor

@FrantisekMichalSebestyen
Copy link
Contributor

Could you use blade template to make this?
Something like this (not tested):
<a href="{{ url()->previous() !== url()->current() ? url()->previous() : route(artwork.index) }}"
I am not sure about the general direction of this project or how we should combine vue and blade template but I thought, we use js just for components that require javascript otherwise we use blade

@rastislav-chynoransky
Copy link
Contributor Author

rastislav-chynoransky commented Dec 10, 2024

url()->previous() is a bit tricky and I would avoid using it here because it stores the url in global state therefore can be overwritten by other tabs

@rastislav-chynoransky rastislav-chynoransky merged commit 214ddab into main Dec 10, 2024
2 checks passed
@rastislav-chynoransky rastislav-chynoransky deleted the GMBUVP-134 branch December 10, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants