When using a thumbnail block (e.g. [bg|bg1|3|117|sort]) within other content I realized that the block always has a left border to the other content, though I wanted it to be inline.

The reason is that the css property "margin" of the tag "ul" is taken over from the global css and not modified within the styles placed by brilliant_gallery.module.

As a quick but working solution I just added "margin: 0" to the $style_ul variable to reset the css margin for html lists, I have defined. Enclosed you find a patch for the 5.x-3.0 version of brilliant gallery.

Regards,
Christian.

CommentFileSizeAuthor
#1 brilliant_gallery-5.x-3.1_ulmargin.patch1.07 KBmeichr
brilliant_gallery.module.patch1.06 KBmeichr

Comments

meichr’s picture

meichr commented
Title: Thumbnail gallery gets same css margin as html lists - should be 0 to flow seemlessly within its context » Thumbnail gallery block's ul css margin and padding explicitly set to 0
Version: 5.x-3.0 » 5.x-3.1
StatusFileSize
new brilliant_gallery-5.x-3.1_ulmargin.patch1.07 KB

Enclosed is the patch ported to version 5.x-3.1 of brilliant gallery.

With this version in addition to margin also padding is set explicitly to 0.

Use this patch, if you want to place a gallery thumbnail block (e.g.[bg|bg1|3|117|sort]) within
the main content and also ul/li tags in the text are used to create lists.

The thumbnail images then will have no margin/padding to the other content and nicely flow into the main content.

A different solution would be to uncomment the div tag in the brilliant_gallery.module and to
reset ul/li margin and padding in the brilliant_gallery.css file or in the theme's css files. This
approach has not been done for simplicity.

Regards,
Christian.

meichr’s picture

meichr commented
Assigned: Unassigned » meichr
Status: Needs review » Fixed

This improvment has been committed to release 5.x-4.4 at the latest. Thanks.

Status: Fixed » Closed (fixed)

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