? a11y PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar hans2103
hans2103
15 May 2023

Pull Request for Issue this issue .

Summary of Changes

This PR will change the unordered list of com_content > category > blog-links.php into an unordered list.
The list is no sequential information but a list of items where the order is not relevant.
Therefor an unordered list should be used.

<ul></ul> instead of <ol></ol>

Testing Instructions

  • Joomla test environment with demo sample data.
  • Go to Sample Layouts > Blog
  • Scroll to the pagination and notice a single link with title = Welcome to your blog

Actual result BEFORE applying this Pull Request

Scherm­afbeelding 2023-05-15 om 16 05 18

Expected result AFTER applying this Pull Request

Scherm­afbeelding 2023-05-15 om 16 05 37

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2023
Category Front End com_content
avatar hans2103 hans2103 - open - 15 May 2023
avatar hans2103 hans2103 - change - 15 May 2023
Status New Pending
avatar richard67
richard67 - comment - 15 May 2023

@hans2103 Could you remove the icon/emoji from the title of this PR? I remember that icons in PR titles have confused some of our tools in past. Thanks in advance.

avatar hans2103 hans2103 - change - 15 May 2023
Title
:wheelchair: change ordered list in blog-links to unordered list
[a11y] change ordered list in blog-links to unordered list
avatar hans2103 hans2103 - edited - 15 May 2023
avatar brianteeman
brianteeman - comment - 15 May 2023

Is there a reason that you only changed the instance in this view and not all views that are incorrectly using ordered lists?

avatar brianteeman
brianteeman - comment - 15 May 2023

For reference these lists were made into ordered lists by the accessibility team in 2012 https://developer.joomla.org/joomlacode-archive/issue-29827.html

avatar chmst
chmst - comment - 15 May 2023

@brianteeman no need for finger pointing. @brianteeman did not find the issue, but the accessibility team did.

avatar brianteeman
brianteeman - comment - 15 May 2023

it is NOT finger pointing - why always looking at the bad :) :-x

I was simply pointing it out that the code is an ordered list because the accessibility team in the past changed it to an ordered list so presumably they had a reason for doing so and as @angieradtke is still around from that time it might be a good idea to find out why before reverting their changes.

avatar hans2103
hans2103 - comment - 15 May 2023

Is there a reason that you only changed the instance in this view and not all views that are incorrectly using ordered lists?

Then others would be changed too in other pr I guess. I just didn't look further.

avatar hans2103
hans2103 - comment - 15 May 2023

For reference these lists were made into ordered lists by the accessibility team in 2012 https://developer.joomla.org/joomlacode-archive/issue-29827.html

Thank you for finding out when these lists where changed into ordered lists.

Apart from what the team thought was right in the past I think we have to change it now. The list is no sequential information but a list of items where the order is not relevant.
Therefor an unordered list should be used.

Do you agree with that argument?
If yes... can this pr be tested and merged?

And if merged, I will create another pr to apply the same change to other ordered lists.

avatar sandewt
sandewt - comment - 16 May 2023

In addition to the previous.

An unordered list makes here the most sense and is therefore preferred, because:

  • Unordered lists are used when the order of the items is not relevant. List items in unordered lists are marked with a bullet.
  • Ordered lists are used for sequential information and are automatically enumerated by the browser.

source: https://www.w3.org/WAI/tutorials/page-structure/content/

avatar HLeithner
HLeithner - comment - 16 May 2023

In addition to the previous.

An unordered list makes here the most sense and is therefore preferred, because:

  • Unordered lists are used when the order of the items is not relevant. List items in unordered lists are marked with a bullet.
  • Ordered lists are used for sequential information and are automatically enumerated by the browser.

source: https://www.w3.org/WAI/tutorials/page-structure/content/

why makes an unordered list more sense then an ordered? isn't the order the most important thing on a pagination?

As unordered list I would expect a grocery store list but not the index of book which the pagination actually is.

avatar chmst
chmst - comment - 16 May 2023

An ordered list indicates that the user has to follow steps in the given order. Like a cooking receipe.
In the pagination the user can choose a pagenumber as he wants. fist page 10, then page 2, or the 20th link first. (It is a grocery store ...)

avatar hans2103 hans2103 - change - 16 May 2023
Labels Added: a11y PR-4.3-dev
avatar sandewt
sandewt - comment - 16 May 2023

why makes an unordered list more sense then an ordered? isn't the order the most important thing on a pagination?

An ordered list indicates that the user has to follow steps in the given order. Like a cooking receipe.

Now I'm starting to doubt, because these arguments are very convincing.

So went to look at the code from Bootstrap, that uses Joomla.
See code Bootstrap v5.2. They use ul for the pagination: (probably the most basic?)

<nav aria-label="Page navigation example">
  <ul class="pagination">
    <li class="page-item"><a class="page-link" href="#">Previous</a></li>
    <li class="page-item"><a class="page-link" href="#">1</a></li>
    <li class="page-item"><a class="page-link" href="#">2</a></li>
    <li class="page-item"><a class="page-link" href="#">3</a></li>
    <li class="page-item"><a class="page-link" href="#">Next</a></li>
  </ul>
</nav>

Source: https://getbootstrap.com/docs/5.2/components/pagination/

The two questions I'm asking now is why ul and why would Joomla differ from Bootstrap?

avatar hans2103
hans2103 - comment - 16 May 2023

This PR does not change the HTML used for pagination.
This PR only changes the HTML for the list above pagination.
See screenshot... the list with a single item with title "Welcome to our blog".

The pagination is not in this scope.

avatar richard67
richard67 - comment - 16 May 2023

@HLeithner I think you misunderstood the PR, like I did at the first time. The relevant thing is not the pagination, it is the small link shown above it. Hard to see in the screenshots.

avatar sandewt
sandewt - comment - 16 May 2023

The pagination is not in this scope.

Good of you to mention this.

avatar HLeithner
HLeithner - comment - 16 May 2023

This PR does not change the HTML used for pagination. This PR only changes the HTML for the list above pagination. See screenshot... the list with a single item with title "Welcome to our blog".

The pagination is not in this scope.

^^ ok sorry that make sense of course

avatar hans2103
hans2103 - comment - 16 May 2023

I like it that we're on the same track now. Please use the same energy as used in discussing this issue for testing this issue. ;-)

avatar sandewt sandewt - test_item - 16 May 2023 - Tested successfully
avatar sandewt
sandewt - comment - 16 May 2023

I have tested this item successfully on 8157e31


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40600.

avatar RickR2H RickR2H - test_item - 16 May 2023 - Tested successfully
avatar RickR2H
RickR2H - comment - 16 May 2023

I have tested this item successfully on 8157e31


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40600.

avatar RickR2H RickR2H - change - 16 May 2023
Status Pending Ready to Commit
avatar RickR2H
RickR2H - comment - 16 May 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40600.

avatar obuisard obuisard - change - 16 May 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-16 15:18:09
Closed_By obuisard
Labels Added: ?
avatar obuisard obuisard - close - 16 May 2023
avatar obuisard obuisard - merge - 16 May 2023
avatar obuisard
obuisard - comment - 16 May 2023

Thank you Hans @hans2103 for the PR!

avatar brianteeman
brianteeman - comment - 16 May 2023

Should this really have been merged in a patch release.

I understand it is being treated as a bug but unlike most bug fixes this changes the visual output of the site which is a breaking change that a user would not expect in a patch release.

avatar obuisard
obuisard - comment - 16 May 2023

Thanks Brian @brianteeman, I understand the concern. I merged this as it has a really low impact and did not deserve to be ignored for another 5 months. The visual will change, but was not right in the first place. That's why I made this decision.

avatar brianteeman
brianteeman - comment - 17 May 2023

Thanks for the response @obuisard #While I appreciate the explanation I will stick with my view that it is wrong in this specific case. It's been wrong for 12 years without comment, until now, so its hardly urgent and there is a remedy available to all users right now (a template override). There are a huge number of other pull requests that user's have reported that are now waiting for 5+ months for a fix and I just dont see what makes this one special.

avatar obuisard
obuisard - comment - 18 May 2023

Hans @hans2103, I will reverse this PR for now, this fix could still impact sites that rely on the <ol> being used.

avatar richard67
richard67 - comment - 18 May 2023

@obuisard Should this be done in 5.0? Or even 6.0?

avatar chmst
chmst - comment - 18 May 2023

Agree with reverse of this PR. It is a bug but not as urgent. It would be nice to have the fix in 5.0 if possible, together with a fix for all occurencies.

avatar hans2103
hans2103 - comment - 20 May 2023

retried it on branch = 5.0-dev with PR = #40633

Add a Comment

Login with GitHub to post a comment