Skip to content

[4.0][a11y] Improve accessiblity in mod_finder#23686

Merged
wilsonge merged 2 commits intojoomla:4.0-devfrom
chmst:a11y-mod-finder-role-search
Mar 21, 2019
Merged

[4.0][a11y] Improve accessiblity in mod_finder#23686
wilsonge merged 2 commits intojoomla:4.0-devfrom
chmst:a11y-mod-finder-role-search

Conversation

@chmst
Copy link
Copy Markdown
Contributor

@chmst chmst commented Jan 26, 2019

Summary of Changes

Improve a11y by adding role="search" to the form tag.

Testing Instructions

Add a finder module to your website.
Inspect the souce code before and after patch.

@zwiastunsw
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 49cee0b


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

1 similar comment
@wojsmol
Copy link
Copy Markdown
Contributor

wojsmol commented Jan 26, 2019

I have tested this item ✅ successfully on 49cee0b


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

@wojsmol
Copy link
Copy Markdown
Contributor

wojsmol commented Jan 27, 2019

@zero-24 Please fix rips issues.

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Jan 27, 2019

Rebooted drone @ http://ci.joomla.org/joomla/joomla-cms/13961

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Jan 27, 2019

Drone and Rips is happy now.

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 27, 2019

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 27, 2019
@davidemessia
Copy link
Copy Markdown

I have tested this item ✅ successfully on 49cee0b


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

@zero-24 zero-24 added this to the Joomla 4.0 milestone Feb 23, 2019
@wilsonge
Copy link
Copy Markdown
Contributor

Thanks!

@wilsonge wilsonge merged commit 2376b24 into joomla:4.0-dev Mar 21, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 21, 2019
@chmst chmst deleted the a11y-mod-finder-role-search branch October 5, 2019 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants