Jdlrobson has uploaded a new change for review.
https://gerrit.wikimedia.org/r/204409
Change subject: Hygiene: Add button partial support to overlays
......................................................................
Hygiene: Add button partial support to overlays
As an example turn the details button in media viewer into a
Button view.
Change-Id: I1e843f50422f508f63765c7865bce255eb6395de
---
M javascripts/Overlay.js
M javascripts/modules/mediaViewer/ImageOverlay.js
M templates/modules/mediaViewer/ImageOverlay.hogan
3 files changed, 12 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/09/204409/1
diff --git a/javascripts/Overlay.js b/javascripts/Overlay.js
index b1de0b4..09ac9a1 100644
--- a/javascripts/Overlay.js
+++ b/javascripts/Overlay.js
@@ -4,6 +4,7 @@
var
View = M.require( 'View' ),
Icon = M.require( 'Icon' ),
+ Button = M.require( 'Button' ),
icons = M.require( 'icons' ),
browser = M.require( 'browser' ),
$window = $( window ),
@@ -40,7 +41,8 @@
*/
className: 'overlay',
templatePartials: {
- header: mw.template.get( 'mobile.overlays',
'header.hogan' )
+ header: mw.template.get( 'mobile.overlays',
'header.hogan' ),
+ button: Button.prototype.template
},
template: mw.template.get( 'mobile.overlays', 'Overlay.hogan' ),
/**
diff --git a/javascripts/modules/mediaViewer/ImageOverlay.js
b/javascripts/modules/mediaViewer/ImageOverlay.js
index 36427a8..f1c233c 100644
--- a/javascripts/modules/mediaViewer/ImageOverlay.js
+++ b/javascripts/modules/mediaViewer/ImageOverlay.js
@@ -23,8 +23,7 @@
* @inheritdoc
* @cfg {Object} defaults Default options hash.
* @cfg {String} defaults.cancelButton HTML of the cancel
button.
- * @cfg {String} defaults.detailsMsg Caption for a button
leading to the details
- * of a media file (e.g. an image) in a preview.
+ * @cfg {Object} defaults.detailsButton options for details
button
* @cfg {String} defaults.licenseLinkMsg Link to license
information in media viewer.
* @cfg {Thumbnail[]} defaults.thumbnails a list of thumbnails
to browse
*/
@@ -36,7 +35,11 @@
additionalClassNames: 'cancel',
label: mw.msg( 'mobile-frontend-overlay-close' )
} ).toHtmlString(),
- detailsMsg: mw.msg( 'mobile-frontend-media-details' ),
+ detailsButton: new Button( {
+ label: mw.msg( 'mobile-frontend-media-details'
),
+ additionalClassName: 'button',
+ progressive: true
+ } ).options,
licenseLinkMsg: mw.msg(
'mobile-frontend-media-license-link' ),
thumbnails: []
} ),
diff --git a/templates/modules/mediaViewer/ImageOverlay.hogan
b/templates/modules/mediaViewer/ImageOverlay.hogan
index 5358f63..59ccb54 100644
--- a/templates/modules/mediaViewer/ImageOverlay.hogan
+++ b/templates/modules/mediaViewer/ImageOverlay.hogan
@@ -6,7 +6,9 @@
</div>
{{{cancelButton}}}
<div class="details">
- <a class="mw-ui-button mw-ui-progressive button"
href="#">{{detailsMsg}}</a>
+ {{#detailsButton}}
+ {{>button}}
+ {{/detailsButton}}
<p class="truncated-text">{{caption}}</p>
<p class="license"><a href="#">{{licenseLinkMsg}}</a></p>
</div>
--
To view, visit https://gerrit.wikimedia.org/r/204409
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1e843f50422f508f63765c7865bce255eb6395de
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
