[4.0] breadcrumbs Changing unordered list to ordered list

[yaume]

Un unordered list seems to be semantically wrong. An ordered list could be better

Issue #16312

Documentation Changes Required

Google guides
BreadcrumbList

[brianteeman]

I suspect that this will have to wait until j4 as it potentially will break the layout on an existing site

[yaume]

It'll wait quite easely as it's not very important and it can be overrided

[brianteeman]

updated title

[C-Lodder]

Seem fine for J4.

@yaume - would you mind submitting a PR against the 4.0-dev branch instead?

[zero-24]

@C-Lodder i have just changed the base branch. So @yaume don't need to do anything ;) Just someone review / test & merge against 4.0

[C-Lodder]

I have tested this item ✅ successfully on 2c361e7

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16313.}

[ghost]

I have tested this item 🔴 unsuccessfully on 2c361e7

please see Screenshot below for Reason of unsuccessfully Test:

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16313.}

[C-Lodder]

Hmm, worked for me in FF. What browser are you using @franz-wohlkoenig ?

[ghost]

System information

4.0.0-dev
macOS Sierra, 10.12.5
Firefox 54 (64-bit)

MAMP 4.1.1

• PHP 7.0.15
• MySQLi 5.6.35

[brianteeman]

I have tested this item ✅ successfully on 2c361e7

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16313.}

[ghost]

RTC after two successful tests.

[C-Lodder]

Are you still having issues with this one franz?

[zero-24]

any feedback on this @franz-wohlkoenig ?

[brianteeman]

Once this is merged we can look at making it accessible

[ghost]

Sorry @zero-24 , missed Question. Will have a Look when #17594 is solved.

[ghost]

@C-Lodder Answer on your Comment: Issue is solved.

[zero-24]

merging than. Thanks for the feedback. Thanks @yaume for your contribution!