Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Clarified the bundle base template bit.#4196
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| @@ -408,6 +408,8 @@ lives in a specific location: | |||
| that's specific to the ``AcmeBlogBundle``. Since the middle, "controller", | |||
| portion is missing (e.g. ``Blog``), the template lives at | |||
| ``Resources/views/layout.html.twig`` inside ``AcmeBlogBundle``. | |||
| Note that the colon is not removed with the "controller", and two colons exist | |||
| between the "bundle"name and template file name. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
bundle name and template filename
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I like the note, but would re-word it to directly talk about the 2 colons:
Yes, thereare 2 colons in the middle of the string when the "controller"
subdirectory part is missing.
I don't get what you are trying to say here, sorry. Can you maybe clarify your change a bit more? Was something confusing you in the current state? |
Yes, it's a bit confusing that there are two colons in a row, it looks like a mistake if you don't have a lot of knowledge. By know it seems logical to me, so maybe I was a bit premature in my edit. |
* 2.3: [#4196] Updating language Clarified the bundle base template bit.
* 2.5: [#4196] Updating language Clarified the bundle base template bit.
Hi guys! I think a little extra clarification herewas kind of nice. So, I merged this, but updated the language a bit in sha:c8f96c0 Thanks! |
No description provided.