Suggestion: Wrap with “Aside” tag would be a better approach

  • Resolved Saswata Baksi

    (@amsaswata)


    3 years, 2 months ago

    When I tried to inside it after content, it injects the div inside the “article” container, so if you can change this div wrap (class=”scriptlesssocialsharing__buttons”) with an “aside” tag it would be a better approach for accessibility and also will be a clean HTML 5 standard.

    Btw thanks for making this awesome lightweight plugin.

    Cheers!

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Robin Cornett

    (@littlerchicken)

    Thank you for the suggestion, I appreciate it!

    If you would like to go ahead and change that for your own use right now, you can use the scriptlesssocialsharing_buttons_container_element filter to return 'aside' instead of 'div' and you’ll be up and running.

    Thread Starter Saswata Baksi

    (@amsaswata)

    Awesome! I’ll try this. However, hope to see this update soon. ??

    Thanks, Robin!

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Suggestion: Wrap with “Aside” tag would be a better approach’ is closed to new replies.