Robmoen has uploaded a new change for review.
https://gerrit.wikimedia.org/r/235134
Change subject: Make edit link and watchstar accessible to ios voiceover
......................................................................
Make edit link and watchstar accessible to ios voiceover
* Give the edit link a element a title attribute
* Make watchstar element a button instead of span
Bug: T87670
Change-Id: Idd36b57e046c7a429aff827aaaf49dbd1f679c20
---
M resources/mobile.editor/init.js
M resources/mobile.watchstar/Watchstar.js
M resources/skins.minerva.base.styles/pageactions.less
3 files changed, 12 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/34/235134/1
diff --git a/resources/mobile.editor/init.js b/resources/mobile.editor/init.js
index 3677be2..0034a29 100644
--- a/resources/mobile.editor/init.js
+++ b/resources/mobile.editor/init.js
@@ -44,7 +44,10 @@
*/
function addEditButton( section, container ) {
return $( '<a class="edit-page">' )
- .attr( 'href', '#/editor/' + section )
+ .attr( {
+ href: '#/editor/' + section,
+ title: $( container ).attr( 'title' )
+ } )
.text( mw.msg( 'mobile-frontend-editor-edit' ) )
.prependTo( container );
}
diff --git a/resources/mobile.watchstar/Watchstar.js
b/resources/mobile.watchstar/Watchstar.js
index f56875b..789a3a6 100644
--- a/resources/mobile.watchstar/Watchstar.js
+++ b/resources/mobile.watchstar/Watchstar.js
@@ -61,7 +61,7 @@
},
tagName: 'div',
className: watchIcon.getClassName(),
- template: mw.template.compile( '<span>{{tooltip}}</span>',
'hogan' ),
+ template: mw.template.compile( '<button>{{tooltip}}</button>',
'hogan' ),
/** @inheritdoc */
initialize: function ( options ) {
var self = this,
diff --git a/resources/skins.minerva.base.styles/pageactions.less
b/resources/skins.minerva.base.styles/pageactions.less
index 92298d3..edb8c13 100644
--- a/resources/skins.minerva.base.styles/pageactions.less
+++ b/resources/skins.minerva.base.styles/pageactions.less
@@ -62,7 +62,8 @@
input,
a,
- span {
+ span,
+ button {
// Needed for non-JavaScript users
position: absolute;
display: block;
@@ -72,6 +73,11 @@
margin: 0 0 8px;
}
+ button {
+ text-indent: inherit;
+ outline: none;
+ }
+
&:first-child {
margin-top: 3px;
}
--
To view, visit https://gerrit.wikimedia.org/r/235134
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idd36b57e046c7a429aff827aaaf49dbd1f679c20
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Robmoen <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
