Patch #15565
Sidebar css
| Status: | New | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% | |
| Category: | Code cleanup/refactoring | |||
| Target version: | - |
Description
Related to #13242.
CSS for <ul> and <li> tags in #sidebar spoils our plugin.
I suggest to add a special class for <ul> and <li> elements that need zero margins.
vanilla-sidebar.diff 
(3.69 KB)
History
#1
Updated by Daniel Felix over 8 years ago
Well, I don't get the point what breaks your plugin.
Do you use a ul/li inside the ul/li of the sidebar?
Maybe it would be enough to change #sidebar ul to #sidebar > ul
#2
Updated by Ksenia Altbregen over 8 years ago
Yes, the plugin imports a tree structure into #sidebar.
Well, it has a very complicated css rendering by js.
I only suggest, I do not insist.
#3
Updated by Daniel Felix over 8 years ago
Can you try to change
#sidebar ul to #sidebar > ul
and #sidebar ul li to #sidebar > ul > li
I think this should do the trick and we don't need to add another class to the css. :-)
#4
Updated by Ksenia Altbregen over 8 years ago
Do you mean in application.css?
If so, it will not, whereas there are <ul>'s situated in <form> and only after that in <div id="sidebar"> in some places.