• Hi Scott,

    Just to let you know that the all-day multiday events with no description are mistakenly receiving the has_desc CSS class. The single-day events are behaving as expected. It’s happening just with the multi-days ones for some reason. In my case, I do not want to show the hover box when there’s no description. I do that via CSS based on the has_desc class.

    Thanks for keeping the good work as always.

    The page I need help with: [log in to see the link]

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Author room34

    (@room34)

    This was an intentional change a few updates back, because the description block has now been amended to include the date range of multi-day events.

    Thread Starter renatobusatto

    (@renatobusatto)

    Thanks for clarifying. That was an interesting choice considering in this case there is no eventdesc element inside a has_desc. To me. it would make more sense to have a has_date class instead, and more importantly, in this way I could hide the hover block that has no desc using CSS only (considering the :has selector isn’t supported by Firefox yet). Please, would you have a suggestion on how to solve this? I can’t think of a solution without using javascript.

    Thanks once again.

    Plugin Author room34

    (@room34)

    has_desc has become a bit of a misnomer, since it really just means “has content that should appear in a hover box”.

    I’ve just finished making some updates to the function that determines the value for has_desc because of some “over-reach” in this exact change, which was causing the hover box text to be included unnecessarily and redundantly in list view.

    There are a few other conditions I also need to update it to check for, which I’m working on at this moment. But I think given the scenario you’re describing, it would make sense to add a filter to allow for additional logic outside of the function itself. This would allow you to avoid JavaScript… but you’d have to use PHP instead.

    Thread Starter renatobusatto

    (@renatobusatto)

    If you allow me to give my 5 cents suggestion, it would be helpful keeping it simple and adding as many specific css classes as necessary (such as has_desc, has_time, has_date etc). In this way your php code would be simpler and would allow more control using css only.

    Plugin Author room34

    (@room34)

    I appreciate that and there is some element of that already… again the issue is really just that has_desc has become a bit of a misnomer, as it’s really just an indicator of whether or not the descloc element (the event description/location/etc. HTML container, used for the hover block, lightbox, etc.) is present.

    Thread Starter renatobusatto

    (@renatobusatto)

    No worries, thank you again for your hard work in keeping this plugin.

    Plugin Author room34

    (@room34)

    To better clarify what I was saying in my last reply, but again all of this stems from the gradual evolution of the plugin and my perhaps excessive inclination to use short, rather than highly detailed CSS class names:

    .has_desc exists to indicate that there is an adjacent .descloc element, which is the outer container for the event details that appear in a hover box, lightbox, etc. (depending on the settings). .descloc contains .eventdesc as well as other output. .descloc is short for “description/location” as that was all it initially included. But the key point here is that the “desc” in .has_desc is associated with .descloc, not .eventdesc.

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘CSS issue: all-day multiday events with no desc’ is closed to new replies.