Skip to content

Add complete TOC and stubs #58

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
Jan 2, 2016
Merged

Add complete TOC and stubs #58

merged 1 commit into from
Jan 2, 2016

Conversation

ddeboer
Copy link
Contributor

@ddeboer ddeboer commented Jan 2, 2016

  • Improve the TOC and make sure it is complete
  • Create stubs for all components

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Hm, why is the diff broken? Contacted GH support about this.

@ddeboer ddeboer force-pushed the doc-stubs branch 2 times, most recently from b4bf9db to e23db7f Compare January 2, 2016 10:12
@sagikazarmark
Copy link
Member

LOL. Good question.

@ddeboer ddeboer force-pushed the doc-stubs branch 2 times, most recently from 5764a49 to e9926a4 Compare January 2, 2016 10:36
@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Okay, have a look at http://php-http.readthedocs.org/en/doc-stubs/ to see the built version.

Is that TOC too long? It may be somewhat intimidating.

@ddeboer ddeboer mentioned this pull request Jan 2, 2016
@sagikazarmark
Copy link
Member

Yes, that was my concern exactly: when we start document every detail, documentation will get too long.

That's why I suggested some kind of separation of the docs, but @dbu is right that it would not make things better: crosslinking, many places for docs.

What I am thinking about now is adding some kind of javascript to make the long blocks (Plugins, Clients, etc) collapsible.

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Decided to just move the components into a components group.

@sagikazarmark
Copy link
Member

Some HTTPlug docs could also be merged ito one page, like virtual package. It is actually documented in the index page as well.

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Right, but this PR should only be about the general structure. I’ll re-organize the HTTPlug docs in a separate PR.

@sagikazarmark
Copy link
Member

Maybe utils could go into clients and adapters section. It has been renamed to client-common anyway.

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Ah, moving it down one level in the hierarchy doesn’t help. The only solution is to use JS or to stuff all items (clients, plugins) together in one file.

@sagikazarmark
Copy link
Member

Yes 😢

@sagikazarmark
Copy link
Member

Actually I think the HTTPlug documentation could be in one file.

Where did the development section go?

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Oops, re-added that.

@ddeboer ddeboer force-pushed the doc-stubs branch 2 times, most recently from 75346e1 to 1956a60 Compare January 2, 2016 11:06
@dbu
Copy link
Contributor

dbu commented Jan 2, 2016

seems like sphinx/rst can do collapsible menus. see http://foshttpcache.readthedocs.org/

i noticed that the 4th level only opens when clicking on the parent (which is named Home instead of being on the parent) http://php-http.readthedocs.org/en/doc-stubs/plugins/ - but that 4th level is only jump points within the page. can we name those "Introduction" rather than "Home" btw? actually, all of those "Home" would better be on the title of the section than as a first sub-section. is this another limitation of mkdocs?

why is "Message" under "Other Components"? i think that should be on the same level as Plugins and Clients. and have the sub pages for the points listed in the intro: Authentication, Stream encoding, Message decorators, Message factory, Cookie

@sagikazarmark
Copy link
Member

seems like sphinx/rst can do collapsible menus

Those are actually one pages, not collapsible menus. We have a problem here when we reach level three, while FOS HTTP cache has only two levels.

Home is the automatic name of index.md. Menu title names are generated based on file name, not on the main title. So we can either workaround this issue by renaming the file to introduction.md or configuring it in the mkdocs config.

When a home section is at the 3rd sublevel, it is because it is included in the folder. If we move it up one level, it will show up there.

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Yeah, Sphinx works better on RTD.

all of those "Home" would better be on the title of the section than as a first sub-section. is this another limitation of mkdocs?

Yes.

why is "Message" under "Other Components"?

Moved it up one level.

@dbu
Copy link
Contributor

dbu commented Jan 2, 2016

do we need the components main heading? right now, level 1 is

  • HTTPlug
  • Components
  • Development

what if it was

  • HTTPlug
  • Clients & Adapters
  • Plugins
  • Message (and move Message Factory in there)
  • Promise
  • Discovery
  • Development

that would kill one level of navigation and provide a more direct approach. i feel that the grouping of "Components" does not tell much.

for the other problems, i guess we either move all plugin docs and all client & adapter docs into one huge .md file or we switch to rst because its better supported. or do we have other viable alternatives?

@sagikazarmark
Copy link
Member

If rst solves our level problem, I am not against it. But again: in FOS HTTP Cache, each heading is one file as I can see, so I don't know whether multi level is supported there.

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

i feel that the grouping of "Components" does not tell much.

Yeah, let’s ditch it.

i guess we either move all plugin docs and all client & adapter docs into one huge .md

I’m in favour of this option. Let’s remember that while we (the maintainers) may not like working on long files, the docs are for our users, so how the docs are rendered is more important.

@sagikazarmark
Copy link
Member

To tell the truth, sphinx syntaxt highlighting is much better IMO (http://foshttpcache.readthedocs.org/en/stable/proxy-clients.html#varnish-client)

@ddeboer
Copy link
Contributor Author

ddeboer commented Jan 2, 2016

Other advantage of Sphinx: spell checker (we use that on FOSHttpCache, too).

@sagikazarmark In FOSHttpCache, we have separated the docs into files, with foldout nav still working: https://github.com/FriendsOfSymfony/FOSHttpCache/tree/master/doc, for instance this one.

The syntax highlighting colours come from a custom CSS. You may be able to do that with MKDocs, too.

@sagikazarmark
Copy link
Member

I wonder why mkdocs cannot support it. Well, I think the navigation is a key part of the documentation, so if it works better in sphinx (and it seems generally everything does) then my vote goes to migrating docs to rst. But it's entirely in your hands, I haven't used rst yo far. 😉

@dbu
Copy link
Contributor

dbu commented Jan 2, 2016

i am up to help converting the stuff. most of it looks just the same anyways, it will just be some dig through everything and change some syntax things job. i think we should merge this first and in general be sure there are no open merge requests and wait for github to have fixed the diff view. then we jump into it and push the change through. pretty much every PR done in parallel will be a conflict...

if we are ready, i can have a go at that on monday.

@sagikazarmark
Copy link
Member

👍

@dbu
Copy link
Contributor

dbu commented Jan 2, 2016

i think we should still merge this PR, or i can PR against this branch, as the reorganizations started here are valuable.

@sagikazarmark
Copy link
Member

Let's merge it when @ddeboer thinks it's ready.

ddeboer added a commit that referenced this pull request Jan 2, 2016
Add complete TOC and stubs
@ddeboer ddeboer merged commit 33c4aab into master Jan 2, 2016
@ddeboer ddeboer deleted the doc-stubs branch January 2, 2016 14:05
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