Skip to content

[4.0] Intro image link [a11y]#31576

Merged
chmst merged 2 commits intojoomla:4.0-devfrom
brianteeman:fix_bad_merge
Dec 7, 2020
Merged

[4.0] Intro image link [a11y]#31576
chmst merged 2 commits intojoomla:4.0-devfrom
brianteeman:fix_bad_merge

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

@brianteeman brianteeman commented Dec 4, 2020

This PR reverts an accidental change made in #30784 to make the intro image a link only when the option is set and to ensure that a link has a title. This is an important accessibility issue.

For full testing instructions etc see the original PR where this was introduced #30823

cc @carcam

This PR reverts an accidental change made in joomla#30784 to make the intro image a link only when the option is set. This is an important accessibility issue.

For full testing instructions etc see the original PR where this was introduced joomla#30823
@snehal0904
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 41e565c


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

1 similar comment
@ghost
Copy link
Copy Markdown

ghost commented Dec 5, 2020

I have tested this item ✅ successfully on 41e565c


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

@richard67
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 5, 2020
@chmst chmst added this to the Joomla 4.0 milestone Dec 7, 2020
@chmst chmst merged commit 165cf06 into joomla:4.0-dev Dec 7, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Dec 7, 2020
@chmst
Copy link
Copy Markdown
Contributor

chmst commented Dec 7, 2020

Thanks!

@brianteeman
Copy link
Copy Markdown
Contributor Author

thanks - lets hope it doesnt get overwritten again

@brianteeman brianteeman deleted the fix_bad_merge branch December 7, 2020 09:22
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.

5 participants