Skip to content

Add "frontend" as one of the topic to getting started #19250

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

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

weaverryan
Copy link
Member

Hi!

Right here: https://symfony.com/doc/current/page_creation.html#what-s-next

I think adding a link to frontend.rst makes sense - it's title is Front-end Tools: Handling CSS & JavaScript. We are showing routes & controllers & templates, so making CSS/JS a main link I think makes sense.

Thanks!

@carsonbot carsonbot added this to the 6.4 milestone Dec 9, 2023
Copy link
Contributor

@94noni 94noni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about right after /templates? they are linked/related imo

@weaverryan
Copy link
Member Author

what about right after /templates? they are linked/related imo

I'm fine with either - if anyone else has a preference. Putting it after /templates is more thematic. But configuration (i.e. services) is also very important.

@javiereguiluz javiereguiluz merged commit ae0d141 into symfony:6.4 Dec 12, 2023
@javiereguiluz
Copy link
Member

Thanks Ryan. While merging I change the location of the new entry to display it just before templates, which felt slightly better to me. Thanks!

@weaverryan weaverryan deleted the frontend-getting-started branch December 12, 2023 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants