search

kasperisager / vanilla-bootstrap

Kickstart your Vanilla-powered community forums with a fresh and solid theme that is ready to be customized to your heart's desires.
http://vanillaforums.org/addon/bootstrap-theme
MIT License
174 stars 62 forks source link

.TagCloud padding-left should be removed #169

Closed Stevearzh closed 9 years ago

Stevearzh commented 9 years ago

I found this problem in our bbs, I think it's better to remove .TagCloud's padding-left. Like the image showing below:

http://img.vim-cn.com/93/6e758fd646e9f61af3662185651e55ce2e2f0e.jpg

Sorry for I'm not that into less now, I'm not sure where and how to modify this, I think labels.less maybe the right place to add this rule...

kasperisager commented 9 years ago

The theme is not concerned with plugin-specific styling. It would be better to open a PR for the plugin in question with a proper reset of the tag cloud CSS.

Stevearzh commented 9 years ago

I found that it was browser's(same thing happened to Firefox) default setting for ul, menu and dir tags that made the 40px's padding-left appeared:

http://img.vim-cn.com/0e/575af1367001d984eb5a920e56757277b0a714.jpg

May I add a rule in globals/mixins.less to reset ul, menu and dir's padding?

kasperisager commented 9 years ago

That'd be a nasty hack and it will cause issues with other elements. There's nothing stopping you from putting it in custom.css though :+1: