Why is this css in the panels.css file?

div.panel-pane div.node {
margin: 0;
padding: 0;
}

Its really ugly and quite 'hard' to override. Either you have to do write equally ugly css in your theme, or you would have to implement hook_css_alter and remove the panels.css file all together (and loose all the css in that file).

Is it important?

Comments

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Title: Inappropriate css » Remove CSS for fixing .node negative margins from Garland theme
Category: bug » task

It is probably not important in D7 and can be removed.

In Drupal 5 and 6, Garland put some CSS on the .node class on the assumption that it would always be rendered inside Garland's main area, and it used negative margins which caused it to act *really* wrong inside panes. At the time it was very hard to have theme specific CSS (though it's somewhat easier now). However, since few people are using Garland in Drupal 7, it is probably irrelevant anyway and can be removed.

Changing this to a task.

tisteegz’s picture

tisteegz CreditAttribution: tisteegz commented
Issue summary: View changes

Any update on this issue? I have just run into this now and it seems like it's going to be really ugly to have to sort it out.

thiagodemellobueno’s picture

thiagodemellobueno CreditAttribution: thiagodemellobueno commented

Agreed, this is overly opinionated and breaks content-lists, views, and anything else that might have div.node (lots of things).