Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21940 closed defect (bug) (fixed)

Twenty Twelve: image alignment issue in sidebar

Reported by: johnbierly's profile johnbierly Owned by:
Milestone: 3.5 Priority: low
Severity: normal Version: 3.4.2
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

HUGE thanks for fixing the missing left image border in Ticket #21926! The same thing happens with Jetpack Images in the sidebar: the margin cuts off the left image border. Thanks again!

Attachments (1)

sidebar.jpg (126.8 KB) - added by johnbierly 12 years ago.

Download all attachments as: .zip

Change History (7)

#1 @lancewillett
12 years ago

  • Milestone changed from Awaiting Review to 3.5
  • Summary changed from Twenty Twelve image align issue resolved yesterday in Ticket #21926 also happens in sidebar. to Twenty Twelve: image alignment issue in sidebar

Hi, could you upload a screenshot and provide more details of how to repeat this, please? What browser and OS, which exact widget and its content so we can reproduce it.

@johnbierly
12 years ago

#2 @johnbierly
12 years ago

Yes! Sorry I didn't do this sooner. The top image is a simple text widget with "img src" html code, and borders are fine. Bottom picture uses the Jetpack Image Widget and borders are missing. Browser is Google Chrome. Thanks for your time and hard work!

#3 @lancewillett
12 years ago

Hmm, seems like the Jetpack Image Widget wraps the content in a wrapper element: <div style="overflow:hidden;"> ... </div> which means the images styles will get cut off.

Note: it's not a border, it's a drop-shadow with box-shadow. :)

Something like this would fix it:

.widget_image div {
	overflow: visible;
}

But I think it'd be better to fix it in Jetpack rather than just put the bandaid fix in the theme. I'll report it and see how it goes.

#4 @johnbierly
12 years ago

Thanks! I prefer to use text widgets for sidebar images anyway. Thanks again for your time and your help. I'm learning a lot.

#5 @lancewillett
12 years ago

  • Priority changed from normal to low

#6 @lancewillett
12 years ago

  • Resolution set to fixed
  • Status changed from new to closed

The CSS fix in the Jetpack Image Widget will be in with next release of Jetpack (version 1.8) -- which is due sometime today or this week.

See http://plugins.trac.wordpress.org/changeset/603747/jetpack for the related change.

Note: See TracTickets for help on using tickets.