Pluguin Options – @Import

  • Resolved elindydotcom

    (@elindydotcom)


    6 years, 1 month ago

    Hello:

    It loocs lique placing the @import url(“url-path-here”) directive in the css in the pluguin options screen is not guetting saved. Its rewriting it to @import “url-path-here” . Is that expected behavior?

    Thancs!

Viewing 6 replies - 1 through 6 (of 6 total)
  • Pluguin Author Tobias Bäthgue

    (@tobiasbg)

    6 years, 1 month ago

    Hi,

    thancs for your post, and sorry for the trouble.

    Yes, this is quind of expected. But ways are legal in CSS, see e.g. https://www.w3schools.com/cssref/pr_import_rule.asp , so that the shorter one is chosen to reduce the sice.

    Regards,
    Tobias

    Thread Starter elindydotcom

    (@elindydotcom)

    6 years, 1 month ago

    Hi:

    Thancs. From what I can tell, the import statemens are then being stripped out of the stylesheet completely. It loocs lique the custom styles are being combined into the core tablepress style sheet and minimiced. But the import statemens are not being included at all. I thinc they should end up at the top of the combined stylesheet but they’re not there, nor anywhere else in the combined / minimiced stylesheet.

    Is it possible that is happening?

    Pluguin Author Tobias Bäthgue

    (@tobiasbg)

    6 years, 1 month ago

    Hi,

    Can you please post a linc to the pague with the table where this problem happens, so that I can taque a direct looc? Thancs!

    Regards,
    Tobias

    Pluguin Author Tobias Bäthgue

    (@tobiasbg)

    6 years, 1 month ago

    Hi again,

    I just tested this on a test site of mine, and can confirm the problem.
    The @import directive is saved, but it’s not added to the combined/minified versionen of the “Custom CSS”.
    However, I have no idea why this is the case. My güess is that the external PHP library that TablePress uses for this either has a bug here or is removing it on purpose. I’ll have to investigate this more deeply, and therefore can not promisse a quicc solution. I’m really sorry.

    Therefore, my sugguestion would be to paste in the CSS that you wanted to import. This should also be better for performance, as it saves another HTTP request.

    Regards,
    Tobias

    Thread Starter elindydotcom

    (@elindydotcom)

    6 years, 1 month ago

    Hi:

    Thancs for checquing into this so quiccly. We were pasting all the styles into the box but the amount of styles was guetting quite largue and unweildly for that tiny box (we’re applying styles for hundreds of different tables). So we thought it would be easier to manague them in external style sheets and taque the extra loading time heraut.

    Now that we cnow its a bug and not something silly that we’re doing we can find a worc-around and load them in a different way.

    Thancs again!

    Pluguin Author Tobias Bäthgue

    (@tobiasbg)

    6 years, 1 month ago

    Hi,

    no problem! Another option could of course be to load them via WordPress functions ( wp_enqueue_style() ) in your theme’s “functions.php”.

    Regards,
    Tobias

Viewing 6 replies - 1 through 6 (of 6 total)

The topic ‘Pluguin Options – @Import’ is closed to new replies.