Patch #15565

Sidebar css

Added by Ksenia Altbregen over 8 years ago. Updated over 8 years ago.

Status:NewStart date:
Priority:NormalDue 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 Magnifier (3.69 KB) Ksenia Altbregen, 2013-11-28 13:48

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.

Also available in: Atom PDF