Viewing 11 posts - 1 through 11 (of 11 total)
  • Author
    Posts
  • #1300138

    Hi,

    I’m having a problem with my client’s new site and Events Calendar plugin. I don’t know if this is related to the problems some others are having but here it goes:

    The single event page is having problems with the looks:

    1) The time is not aligned correctly at the bottom of the page. The “time:” heading is not aligned with the corresponding event time. It looks a bit like this:

    14:00-15:00
    Time:

    I can see from the inspector that if I disable the dd, dt float “left”, it puts the time under the “time:” heading. How can I achieve this or then just align it correctly with the heading.

    2) There is somehow a heading named “event navigation” at the bottom of the page. I don’t know the reason for this even though I have the plugin on many sites. Is there another way to get rid of it than this code:

    .tribe-events-visuallyhidden{
      display: none;
    }

    I get it off with the code but I believe it shouldn’t be there in the first place.

    3) There is no padding in the single event in mobile view. (The text begins right from the edge of the screen) I had a code for this in the older format but it’s not working for some reason.

    More info in the private content.

    • This topic was modified 2 years, 11 months ago by Cloaker.
    #1300319

    Hey Cloaker,
    Thank you for your patience and the link to your site, for the first issue with the time heading, it is caused by this css:

    .tribe-events-event-meta dt:not(:first-child) {
        margin-top: 16px;
    }

    so this is saying that in the tribe-events-event-meta block any dt that is not the first-child will get a 16px margin-top, which is these two elements:
    2021-05-14_063213.jpg
    In this case I believe the best solution would be this:

    .tribe-events-single-section.tribe-events-event-meta.primary dt:not(:first-child) {
        margin-top: 0;
    }

    For your second question the code <h3 class="tribe-events-visuallyhidden"><?php _e( 'Event Navigation', 'avia_framework' ) ?></h3> is in the \wp-content\themes\enfold\config-events-calendar\views\single-event.php file on line 145, you can try removing this if you wish. I’m not sure if it is used for anything else, but the code was originally recommended by the plugin’s Themer’s Guide for the Pro version, from what I can tell. But I’m not an expert with this plugin 🙂
    For your third issue, the padding seems to be set to zero by this css in your merged css:

    #top #tribe-events-pg-template {
        max-width: 100%;
        padding: 0;
    }

    which is overriding the plugin padding:
    2021-05-14_071117.jpg
    please try looking for this in your custom css and remove it.

    Best regards,
    Mike

    #1300323

    Hi!

    Thanks for the reply! I got the first problem sorted (used your code and worked perfectly! Still wondering why it was necessary in the first place… My other sites have not had that kind of a problem. Thanks again!

    The second part I decided to fix with my css “display none”… I think the php would change back if the file would be updated? In addition I’m still wondering where this heading comes from – again my other enfold sites do not have it :D

    The third problem is a weird one since I do not have any code in my css that would state padding 0. I even tried adding a code for padding 30, but it didn’t work

    #top #tribe-events-pg-template {
        max-width: 100%;
        padding: 30;
    }

    Again something that my other sites do not have :S

    #1300493

    Hi,
    Thank you for the feedback, your css for the third issue has the same specificity so in order for it to work you can add !important; tp it like this:

    #top #tribe-events-pg-template {
        max-width: 100%;
        padding: 30 !important;
    }

    or you can increase the specificity by adding an ID like this:

    #top #wrap_all #tribe-events-pg-template {
        max-width: 100%;
        padding: 30;
    }

    I’m not sure why you can’t find the zero padding css in your custom css or how it got into your merged css, but you could try disabling your Enfold Theme Options > Performance > JS & CSS file merging and compression and then at the bottom of the same page enable the Enfold Theme Options > Performance > Delete old CSS and JS files option and then check again.

    Best regards,
    Mike

    #1300544

    Hi,

    Now I have these codes in my quick CSS but the problem is that which ever of the two codes (either the padding 30px OR the margin-top 0) are on top of the other cancels the other. So in this instance, I have the margin code working BUT the padding code will not work. If I switch it so that the padding code is above the margin-top code, the margin code (fixing the time alignment) stops working.

    .tribe-events-single-section.tribe-events-event-meta.primary dt:not(:first-child) {
        margin-top: 0;
    }
    
    #top #tribe-events-pg-template {
        max-width: 100%;
        padding: 30px !important;
    }
    
    .tribe-events-visuallyhidden{
      display: none;
    }

    I wonder if the quick CSS in this new Enfold version has a bug or something? That’s because I also have a problem in one of my other sites where the quick css keeps bugging: the scroll-to-top button disappeared from mobile view ALTHOUGH I do have the right CSS code showing it. It has worked before but after the update, it stopped. So the code I have is

    @media only screen and (max-width: 767px) {
        .responsive #top  #scroll-top-link {
        display: block;
    }
    }

    I also added a minor tweak for the appereance but it doesn’t matter if I have it in there or not. Tried both ways.

    a#scroll-top-link, a#scroll-top-link:hover {
    background: #bf4da4;
    border-radius: 100px;
    font-size: 26px;
    }
    #1300556

    Hi,
    Thanks for your feedback, the css for the padding of the page and the margin for the time text should not interfere with each other as they are different elements, what you describe sounds like there is a missing bracket in your css elsewhere, but while you are using the merged css & js in the theme options I can not find the error, please disable this for now.
    Perhaps your issue on this site is similar to the issue you are having on your second site, on your second site the reason the scroll top display block css is not working is because you are using PHP comment out code in CSS code:

    // ----------------------------------------
    // SCROLL PAINIKE KÄNNYKÄLLÄ

    this is the correct way to comment out in CSS

    /* ---------------------------------------- */
    /* SCROLL PAINIKE KÄNNYKÄLLÄ */

    I see on your second site this occurs 10 times, so please correct this and clear your browser cache and the theme css merging as explained above.
    If you have also done this on your first site please also correct.

    Best regards,
    Mike

    #1300558

    Hi, yes that second page comment out mistake was it – silly of me. Didn’t even notice… Too few hours of sleep I guess :) Thanks – and sorry!

    But in the first site there were no such lines. I disabled the merging.

    #1300573

    In addition to the first sites problem, I noticed that this code where I disable some of the sorting options in the shop page, the woocommerce page element breaks.

    div .product-sorting ul ul li:nth-child(2), li:nth-child(5), li:nth-child(7), li:nth-child(8), li:nth-child(9), li:nth-child(10) {
        display: none;
    }

    It does hide the sorting options that I wanted but it also hides at least one of the products and the view of the shop page is broken. There is en empty space where a product (now missing) should have been and some of the paddings are also broken. Please see pictures. (It doesn’t matter how many columns there are chosen from the woocommerce settings)

    Any ideas why it might break? Other sites (again) have worked ok with this code.

    #1300582

    Hi,
    Thank you for disabling the merging, I now see that you have an extra bracket that is breaking the following css and you have a space after the dot which indicates the class:
    2021-05-16_122333.jpg
    Please correct this for your CSS to correctly work.
    As for your new CSS snippet above, it is flawed so it will not work correctly.
    In CSS rules, after each comma, you need to fully specify the path to your target, writing
    , li:nth-child(5), li:nth-child(7), li:nth-child(8), li:nth-child(9), li:nth-child(10)
    will target every li:nth-child(x) on the page.
    Please try re-writing to be more specific.

    Best regards,
    Mike

    #1300586

    Oh my. Went through it like 10 times and just couldn’t catch that… Thank you so much!

    By the way, now it might be safe to re-enable the merging? And one other thing – is the “delete old css…” button in the bottom part of the performance tab something that you have to press once – then save – and then just deselect it again? (for it to work that is…) Just wondering since the button is a slider and not a simple button.

    #1300612

    Hi,
    Glad to help, you should be able to re-enable the merging, but I would recommend waiting to use merging or any caching until you are done creating your site because I would imagine you will be making many changes.
    The delete old css option is a on/off slider and not a button, so clicking it enables/disables.
    I hope that explains it well, unless there is anything else we can help with on this issue, shall we close this then?

    Best regards,
    Mike

Viewing 11 posts - 1 through 11 (of 11 total)
  • You must be logged in to reply to this topic.