› ›
CRITICAL SECURITY VERSION UPDATE:
https://github.com/daylightstudio/FUEL-CMS/releases/tag/1.4.13

Aria Support for breadcrumbs

almostcompletelyalmostcompletely
edited June 2023 in Feature Requests

Actually, it's 2 things:

1 - I don't think "first_class" and "last_class" fuel_nav() parameters work for the breadcrumb render_type
2 - For screen-reader Aria support, we're supposed to mark the last item on the breadcrumb line with aria-current="page"

I guess a "first_attribute" and "last_attribute" (along the lines of "first_class" and "last_class" parameters) would allow developers to add the aria support in a flexible fashion?

Comments

  • adminadmin

    I've pushed an update to this in the develop branch for you to test out

  • almostcompletelyalmostcompletely

    aria-current="page" is automatic? Neat. Work fine. Thanks.

  • almostcompletelyalmostcompletely

    Looks like aria-current is required for fuel_nav()s too.

    In Menu.php, within _create_open_li(), I've changed

    $str .= $this->_create_link($val);

    to

    $active = (is_array($val)) ? $val['id'] : $val;
$str .= $this->_create_link($val,$active);

    and in _create_link() added aria-current="page" to the outputs

    BTW: The breadcrumbs code still has the numbering debugging integer displaying.

  • adminadmin

    Thanks... I've pushed an update for this.

Sign In or Register to comment.