Problem/Motivation

See #2931790: Consistent media alignment handling for background.

This is the server-side part that attacks media tags during content filtering and asserts that all various kinds of alignment is aggregated to the media token $tag_info['fields']['alignment'] setting and proper alignment css classes added to the media container on content output.

Proposed resolution

This scans the $tag_info array for various alignment methods (inline style, 'align' attribute) and replaces these with alignment setting and class.

CommentFileSizeAuthor
#2 media-2941608-aggregate-alignment-token-filtering-2.patch13.78 KBkaare
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kaare created an issue. See original summary.

kaare’s picture

kaare
him/his
Primary language Norwegian Bokmål
Location Tromsø
CreditAttribution: kaare at Norway Opening Universities commented
Status: Active » Needs review
FileSize
media-2941608-aggregate-alignment-token-filtering-2.patch13.78 KB

And here is the patch.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs review » Reviewed & tested by the community

  • joseph.olstad committed e63534e on 7.x-3.x authored by kaare 
    Issue #2941608 by kaare: Aggregate and uniform alignment handling during...
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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