-
Notifications
You must be signed in to change notification settings - Fork 156
Seo bundle documentation #438
Seo bundle documentation #438
Conversation
} | ||
|
||
A very basic use case | ||
_____________________ |
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.
Have no clue why that headlines size are jumping from big to small
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.
the cmf doc contains guidelines which headlines to use for what level. afaik underline is never used
Nice that the error occurs in my PR |
For that and the design of the navigation i would need some help:
|
You have to put |
@@ -0,0 +1,7 @@ | |||
SeoBundle | |||
============= |
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.
tagline (line containing =
characters) should be as long as the headline itself. There are more occurences in this PR which needs a fix
Can you please update your line breaks in the PR, so there is a line break after the first word that crosses the 72th character. The very low width you've used now doesn't make it easy to review |
I have chosen a with? I just put some chars in that file, but can't remember to set a with. |
I have no clue how these references work for rst files. Just copied the files. |
Ah but now i understand it on second view. |
Both take care on search engines, which does not like | ||
it to have same content under different routes. | ||
|
||
The SeoBundle uses sonatas SeoBundle and its TwigHelper |
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.
The CMF SeoBundle uses the SonataSeoBundle and ...
continues in #442 |
So now it should work. I just compare with dev.