Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The project page links to a FitVids jQuery file. Near the top of this file we see two settings:
$.fn.fitVids = function( options ) {
var settings = {
customSelector: null,
ignore: null
};
Currently, there is the ability to pass in options for customSelector
into the FitVid's library, but not the option for ignore
.
Plan:
Create a textfield for the admin settings.
Have these options feed into FitVids to utilize the ignore setting.
Comment | File | Size | Author |
---|---|---|---|
#5 | ignore-setting-2635066-5.patch | 3.14 KB | rahulbile |
|
Comments
Comment #2
mikedotexe CreditAttribution: mikedotexe at Metal Toad commentedPatch
Comment #3
mikedotexe CreditAttribution: mikedotexe at Metal Toad commentedComment #4
mikedotexe CreditAttribution: mikedotexe at Metal Toad commentedComment #5
rahulbile CreditAttribution: rahulbile as a volunteer commentedLooks good @mikedotexe, attached is the updated patch for passing default value if not set as '', oresle it throws js error of it being null.
Comment #7
BWPanda CreditAttribution: BWPanda commentedWorks for me...
Comment #8
mikedotexe CreditAttribution: mikedotexe commentedLooking at some older issues, I think this one never got flipped to "Needs review" so I'm doing that now.
Comment #9
rrrob CreditAttribution: rrrob at Chapter Three commented#5 looks good and works for me.
Comment #11
NickDickinsonWildenot promising a 7.x release... but looks good, merged, thanks