[MGNLSTK-441] BodyClass 'col' has to be overworked Created: 20/Aug/09 Updated: 23/Jan/13 Resolved: 04/Sep/09 |
|
| Status: | Closed |
| Project: | Magnolia Standard Templating Kit (closed) |
| Component/s: | concept |
| Affects Version/s: | 1.1.1 |
| Fix Version/s: | 1.2 |
| Type: | Bug | Priority: | Major |
| Reporter: | Christian Ringele | Assignee: | Christian Ringele |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
For the possibility of displaying mainArea content over the complete width of a page a bodyClass 'col' was implemented into the BodyClassResolver.java. Now there are two problems about this col: I thing the concept of 'col' and 'nav-col' (and eg 'col-subcol' to 'nav-col-subcol' should be looked at and be clearly defined and implemented into css. |
| Comments |
| Comment by Philipp Bärfuss [ 21/Aug/09 ] |
|
The body class reflects the current situation and not the site configuration. If you have after level 3 a sub-navigation but your page exists on level 2 it won't render a navigation and the correct body class is: col, respectively col-subcol. Otherwise the demo project would simply not work. The effect would be that on the section pages empty space would be added in the left side. So the only 'open' point is that this is not supported by the CSS. But in case you need it you can simply adapt the CSS accordingly. |
| Comment by Christian Ringele [ 21/Aug/09 ] |
|
You are right, its on the current situation and not the configuration. About the css: Originally the idea of 'col' was, that mainContent is spread across the complete page. This more regarding content pages. |
| Comment by Philipp Bärfuss [ 04/Sep/09 ] |
|
Is kind of succeeded by |