WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#21940 closed defect (bug) (fixed)

Twenty Twelve: image alignment issue in sidebar

Reported by: 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 2 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 @lancewillett2 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.

@johnbierly2 years ago

comment:2 @johnbierly2 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!

comment:3 @lancewillett2 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.

comment:4 @johnbierly2 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.

comment:5 @lancewillett2 years ago

  • Priority changed from normal to low

comment:6 @lancewillett2 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.