X
33470

Ticket #33470 Public

ID's must be unique.

Hi. We're using the shortcode, [add_eventon_list], and the plugin is creating multiple id="evcal_list" which fails our web accessibility test.

 

BY: Erik Cunningham - Apr 6,2017 AT 8:07AM - 4 years ago
    • By: Ashish Rai Apr 6,2017 AT 8:25AM - Posted 4 years ago - #33479

      I am not sure what you mean.

      Can you explain a little more?

      It could  be IDs for each event in the list

    • By: Erik Cunningham Apr 6,2017 AT 8:27AM - Posted 4 years ago - #33480

      Thanks for your quick reply.

      “The id attribute specifies a unique id for an HTML element (the value must be unique within the HTML document).”

      https://www.w3schools.com/tags/att_id.asp

    • By: Erik Cunningham Apr 6,2017 AT 8:29AM - Posted 4 years ago - #33482

      Using the “class” attribute instead of “id” should fix the problem from the front end.

    • By: Ashish Rai Apr 10,2017 AT 10:31AM - Posted 4 years ago - #33820
    • By: Ashan Perera Apr 13,2017 AT 7:40AM - Posted 4 years ago - #34156

      Yes i am aware of this and what ID vs class means. WOuld you be able to send us screenshot of exactly where this is happening in your calendar HTML? Thanks

    • By: Erik Cunningham Apr 14,2017 AT 7:49AM - Posted 4 years ago - #34254

      Sure.  Here’s a screenshot of the page and Chrome inspector.

    • By: Ashan Perera Apr 14,2017 AT 8:32AM - Posted 4 years ago - #34266

      Are you using the event list version of the calendar in here? Thats what it looks like what you are using in the screenshot you sent. I will make a note to take a look at this and get this issue resolved in the next eventon update.

    • By: Erik Cunningham Apr 14,2017 AT 11:08AM - Posted 4 years ago - #34290

      I believe so.  We're using the [add_eventon_list] shortcode.

    • By: Ashan Perera Apr 17,2017 AT 11:30AM - Posted 4 years ago - #34409

      Ok i see whats going on. You know this is not a huge issue that is going to cause major damage to your site. I understand that the code is not what is recommend yet it is still functional. This was done long long time ago when it was first built and there has been lot of other codes that rely on that at the moment so we can not just go change the ID to class and that would cause major issues. We will slowly make the change to a class method to align with code standard overtime.

      So please continue to your as it is, while we make this change in the coming month at the same time making sure the change is not going to break styles and scripts relying on this.

      Cheers!

You must login to reply to this ticket

HelpDesk

Welcome to EventON helpdesk.

EventON is the #1 Best selling event calendar plugin for WordPress websites in envato marketplace.

Checkout EventON