search

openzim / openedx

Open edX (to zim) scraper
GNU General Public License v3.0
8 stars 7 forks source link

all introduction videos must be placed correctly in the middle #121

Closed IPECORE closed 4 years ago

IPECORE commented 4 years ago

the videos are on the left and too small, all 10 videos_middle

satyamtg commented 4 years ago

The videos are properly centered after the fix in #115. @IPECORE you can see the screenshot in that PR. Also, for the videos being small, we had a discussion regarding that in one of the issues. The point being that when we have low quality option enabled, videos would look blurred when big. That's why I fixed it to the height of the low quality preset.

@rgaudin what do you suggest in this regard?

rgaudin commented 4 years ago

I've updated the live demo with latest version. @IPECORE can you please take another look? As discussed in #115, this is now centered but not expanded