I'd like to have the possibility to a align a video horizontally centered. I attached a patch providing this feature.

CommentFileSizeAuthor
video_filter.patch1.29 KBPisco

Comments

Pisco’s picture

Pisco commented
Status: Active » Needs review
anon’s picture

anon
Primary language Swedish
Location Gothenburg
commented

We should NOT be using the style attribure for this at all.

An idea I have is to use classes for this. 

.video-left  {
  float:left;
}
.video-right {
  float:right;
}

This way, everyone can override video_filters default css with their own.

The administration for this could be like:
- We have a field for classes. (video-left, video-right) and users can easyely add own classes in this field (a comma-separated list)
- In the popup we get all styles from that field, and creates a selectlist

blackdog’s picture

blackdog commented
Status: Needs review » Fixed

I like the idea of having admin defined classes, but in the meantime, I've added support for align center, as a class, since I committed #581370: Use CSS selectors tonight.

Status: Fixed » Closed (fixed)

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