Download & Extend
Responsive Bartik D7 » Issues

Inline Links Need Inline Style - Problem with Taxonomy Term Display

Posted by populist on November 12, 2012 at 5:59pm
Project:Responsive Bartik D7
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

There is a small issue with the handling of CSS around inline links as demonstrated here:

The basic change needed looks like this:

ul.links.inline li a {
-  display: inline-block;
+  display: inline;
   padding: 18px 0 5px;
}

This fixes it as demonstrated:

Login or register to post comments

Comments

#1

Posted by populist on November 12, 2012 at 6:00pm
Status:active» needs review

Here is a patch to get this done.

AttachmentSize
1838602-inline-block-to-inline-for-links-1.patch 248 bytes

#2

Posted by mjohnq3 on December 10, 2012 at 8:12pm
Status:needs review» reviewed & tested by the community

Thanks. This will be incorporated in the upcoming beta release.

#3

Posted by mjohnq3 on December 11, 2012 at 3:22am
Status:reviewed & tested by the community» fixed

Added to beta1 release.

#4

Posted by System Message on December 25, 2012 at 3:30am
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#5

Posted by franskuipers on February 16, 2013 at 12:09pm
Status:closed (fixed)» reviewed & tested by the community

I don't see this patch included in this commitdiff on December 11, 2012.

Current code in -bata1 and 7.x-1.x:

700 ul.links.inline li a {
701  display: inline-block;
702  padding: 18px 0 5px;
703 }

#6

Posted by mjohnq3 on February 16, 2013 at 2:40pm
Status:reviewed & tested by the community» fixed

You're correct. Thanks for the heads up.

Committed (194c16b) and pushed to 7.x-1.x.

#7

Posted by System Message on March 2, 2013 at 2:40pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here