Project

General

Profile

Actions

Patch #15565

open

Sidebar css

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

Status:
New
Priority:
Normal
Assignee:
-
Category:
Code cleanup/refactoring
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

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.


Files

vanilla-sidebar.diff (3.69 KB) vanilla-sidebar.diff Ksenia Altbregen, 2013-11-28 13:48
Actions #1

Updated by Daniel Felix over 9 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

Actions #2

Updated by Ksenia Altbregen over 9 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.

Actions #3

Updated by Daniel Felix over 9 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. :-)

Actions #4

Updated by Ksenia Altbregen over 9 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.

Actions

Also available in: Atom PDF