New Landing How can we help? Cardinal vimeo height/width issue

Viewing 15 posts - 1 through 15 (of 17 total)
  • Posted in: Cardinal
  • #95886
    themartist
    Member
    Post count: 122

    Ive inserted a Vimeo video in the portfolio page but there are black bars on the side – meaning its nit high enough. Entering height details in the portfolio meta doesn’t seem to do anything. Any suggestions?

    Secondly, subtitle field inside the portfolio meta doesn’t seem to be showing.

    #95985
    Melanie – SUPPORT
    Member
    Post count: 11032

    Hey, does that improve it?

    .player .video-wrapper .video {
    height: 105%; }
    #96090
    themartist
    Member
    Post count: 122

    No – no change. The problem is that the video height is not working. That field that controls video height as well as the field where a portfolio subtitle is inserted are not actually working. Would obviously prefer avoid having a custom css rule created in favor of having the built in theme functionality.

    #96199
    themartist
    Member
    Post count: 122

    any suggestions?

    #96580
    brooklynkid
    Member
    Post count: 37

    I am having the same issues with Vimeo videos. Changing the height and width seem to do nothing, the videos always display small with a black area around them. I tried downloading the latest build from themeforest, but the issue is still there.

    Is there a fix on the way for this issue?

    Attachments:
    You must be logged in to view attached files.
    #96987
    themartist
    Member
    Post count: 122

    guys… please answer this thread.

    #97104
    Swift Ideas – Ed
    Keymaster
    Post count: 15264

    Apologies for the delay.

    EDIT: Just checked, apologies for the issue – resolved for the next update. You can fix by adding this custom css:

    .sf-video-wrap {
    padding-bottom: 56.25%;
    position: relative;
    width: 100%;
    }


    @brooklynkid
    – please update to the latest version as this has been fixed.

    – Ed

    #97109
    themartist
    Member
    Post count: 122

    Hey Ed

    When you mean resolved for the next update, do you mean that the height field will be working after the latest Cardinal 1.5.4 ? My site is full of videos (maybe 40) in different sizes as well – so obviously having them display correctly is really crucial without custom CSS for each clip. It would be nice if that height field worked.

    #97368
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Hi,

    You only have to add it once the custom css options of the theme.

    https://www.dropbox.com/s/qb6oehl48kiex9k/custom_css_options.png

    -Rui

    #97376
    themartist
    Member
    Post count: 122

    Problem is – if different videos have different proportions there couldn’t be one css solution

    #97378
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    I guess the solution provide is a global solution. But will check with the Developer.
    Did you tried the css solution to see if it works with several videos?

    -Rui

    #97395
    themartist
    Member
    Post count: 122

    Thanks Rui

    I didn’t but it definitely wouldn’t work – there is pal, ntsc 1080, 720 x 576 etc etc etc so many different formats that a global padding would mess up some of the videos. I understand the custom css, but this just couldn’t be global.

    Essentially that height field is extremely necessary for anyone who wants to use video – and considering that agency is a big of the theme, I think displaying video correctly should be quite a priority.

    #97493
    Swift Ideas – Ed
    Keymaster
    Post count: 15264

    Hi @themartist,

    I worked on this over the weekend. I’ve set it up so that if you provide a width/height in the asset, then the aspect ratio will be calculated from this.

    It will be in the update this week.

    – Ed

    #97577
    themartist
    Member
    Post count: 122

    Brilliant ED! You guys are great.

    #97583
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Great. Thanks Ed.

    -Rui

Viewing 15 posts - 1 through 15 (of 17 total)

You must be logged in and have valid license to reply to this topic.

License required for one of the following items
Login and Registration Log in · Register