Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[HttpKernel] Bootstrapping the HttpKernel Component Documentation #2064

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 3 commits into from
Dec 30, 2012

Conversation

weaverryan
Copy link
Member

Hiya guys!

Here's the long awaited first pass at the HttpKernel component documentation - thanks to @Drak for poking me on this and @richardmiller for this patience! The images used here are with Fabien's permission from his https://speakerdeck.com/fabpot/symfony2-meets-drupal-8 presentation.

If anyone has a few minutes, please read this over - comment about any corrections or anything that you think will need to be added later or changed. I'd like to merge this relatively quickly so that all of us in the community has the opportunity to improve it from here.

Thanks!

The HttpKernel Component provides a structured process for converting
a Request into a Response by making use of the event dispatcher. It's
flexible enough to create a full-stack framework (Symfony), a micro-framework
(Silex) or an advanced CMS system (Drupal).
Copy link
Member

Choose a reason for hiding this comment

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

Why is it indented ?

Copy link
Member

Choose a reason for hiding this comment

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

We do that in all component documents.

@wouterj
Copy link
Member

wouterj commented Dec 26, 2012

You should create some interlinking to this aswell, for instance in the introduction paragraphs of the Event Dispatcher Component, we talked about this component and it's event features, we should add a link this this documentation.

@weaverryan
Copy link
Member Author

Hi guys!

Thanks to @Drak, @wouterj and @stof for the really quick check on this - you guys each had some really good suggestions, I've taken all of them into account.

Also, after a merge, the following changes will need to be made on the 2.1 branch (which I'll do after a merge):

  • document kernel.terminate
  • change "on the Session" wording, where we talk about setting the locale "on the Session" - which will now be "on the Request"

Thanks!


Another common listener is routing. A router listener may process the ``Request``
and determine the controller that should be rendered (see the next section).
In fact, the ``Request`` object has an ":ref:`attributes<component-foundation-attributes>`"
Copy link

Choose a reason for hiding this comment

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

Yes, I prefer the backticking of the classes as it makes the code more readable as instantly recognisable as something special. making then ::class:: links initially and then backticks is 👍

@ghost
Copy link

ghost commented Dec 27, 2012

Looks good to merge now.

@ghost
Copy link

ghost commented Dec 28, 2012

@weaverryan - I think you should get this merged and then progress to the 2.1 docs. We can tweak the documents to perfection afterwards.

weaverryan added a commit that referenced this pull request Dec 30, 2012
[HttpKernel] Bootstrapping the HttpKernel Component Documentation
@weaverryan weaverryan merged commit d834b44 into 2.0 Dec 30, 2012
weaverryan added a commit that referenced this pull request Dec 30, 2012
@weaverryan
Copy link
Member Author

Hi guys!

Thanks again very much for your help on this! I've now merged this into 2.0, made the 2.1 changes at sha: 2d23d72 and merged up to 2.1 and master. If anyone sees any issues, please let me know!

Again, I'm very excited to have these docs and have so many people driving to get them done and done well. There's certainly more to cover from here, which we should probably start to break out into individual documents in this section.

Thanks!

@weaverryan
Copy link
Member Author

Also, I've just seen the first render of the docs at symfony.com, and we're missing the images right now. Let's wait for a full nightly re-render - I believe I've just missed today's re-render by a few hours. So if it's not better in 24 hours, then we'll check into it further.

@weaverryan weaverryan deleted the http-kernel branch December 30, 2012 04:17
@ghost
Copy link

ghost commented Dec 30, 2012

ping @fabpot

@wouterj
Copy link
Member

wouterj commented Dec 30, 2012

The images are shown here (maybe a hard refresh works?)

@weaverryan
Copy link
Member Author

I'm also seeing the images now, so we're good!

@wouterj
Copy link
Member

wouterj commented Dec 30, 2012

Now it is rendered, it looks like a really long page to me. Long scrollbars scaries people, at least me. We can maybe find places to split this into multiple articles?

@ghost
Copy link

ghost commented Dec 30, 2012

Yes, and the images should be a little less than the width of the text, they are a bit imposing as it stands. I'd also have a little spacing before and after the image, they are currently a bit too flush to the text in some places.

@ghost
Copy link

ghost commented Jan 1, 2013

There is also a rendering issue, see attached:

The HttpKernel Component master Symfony

ping @fabpot

@wouterj wouterj mentioned this pull request Feb 2, 2013
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.

3 participants