• Resolved brent3721

    (@brent3721)


    as it is now you can’t add your own background-size css to allow the pinterest to be retina friendly because there’s important set on the background already:

    I’m setting my background size in your plugin to 80×80

    then putting this css in chrome inspector:
    a.pinit-button {
    background-size: 40px 40px !important;
    }

    which resizes it and fixes the pixel density problem, but then when I add it to my stylesheet it doesn’t get applied cause of the importance.

    http://wordpress.org/plugins/jquery-pin-it-button-for-images/

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author mrsztuczkens

    (@mrsztuczkens)

    Hi,

    taking the !important clause off isn’t an option. Why? From time to time I get a support thread with someone saying that something isn’t rendered right. In almost all cases it’s caused by other css clauses overriding my own settings. That’s why the !important is there.

    The only thing that makes sense to me (and doesn’t involve adding additional settings in the admin area of the plugin) is adding a filter to the inline css code, so you hook a function that will add your css right after mine. That should do the trick. What do you think?

    Thread Starter brent3721

    (@brent3721)

    Lots of people use high density displays now so not having support for them makes the plugin unusable for sites that require the support.

    I could hack my template to have a filter, but it seems like a pretty easy change on your end that would allow your plugin to be used by a lot more people.

    You’d just need to include a checkbox that says ‘retina friendly’ that would cut the height and width that are input in half and add the background size line into the CSS with the halved values.

    Of course I understand this is a free plugin so if you don’t have the time I’m happy to make the adjustments on my end. It’s just a shame that such a great plugin doesn’t support retina, all the sites we make have this as a requirement, it’s becoming something that clients are looking for.

    brent
    @
    condensed.io

    Plugin Author mrsztuczkens

    (@mrsztuczkens)

    Hi Brent,

    you’ve convinced me. I’ll add the ‘retina friendly’ checkbox in a couple of days and release a new version of the plugin within a week or so. Thanks for the suggestion.

    Plugin Author mrsztuczkens

    (@mrsztuczkens)

    Hi again,

    new version of the plugin with the ‘Retina friendly’ option is available. Let me know if it’s that you were looking for!

    Thread Starter brent3721

    (@brent3721)

    it works !!! — See it in action here: http://inspiration.condensed.io/

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘could you possibly take the !important declarations off the background css?’ is closed to new replies.