From 1a291349ac46dea84c6a145ae86ae32d8dedfb80 Mon Sep 17 00:00:00 2001 From: Georg Fischer Date: Wed, 10 Aug 2016 20:06:55 +0200 Subject: [PATCH] increase button size on touch devices reorder share links update imgur url --- scripts/views/shareview.js | 10 +++++----- styles/modules/dialog.css | 10 ++++++++++ 2 files changed, 15 insertions(+), 5 deletions(-) diff --git a/scripts/views/shareview.js b/scripts/views/shareview.js index 5a01825..2e0aaee 100644 --- a/scripts/views/shareview.js +++ b/scripts/views/shareview.js @@ -69,12 +69,12 @@ define( .add( isOnlineCssClass, uploadButtonEl ) .add( isOnlineCssClass, 'loader', statusEl ) .add( isOnlineCssClass, imgLinkLabel, imgLinkEl ) - .add( isOnlineCssClass, imgurLinkEl ) - .add( isOnlineCssClass, redditShareLinkEl ) .add( isOnlineCssClass, twitterShareLinkEl ) .add( isOnlineCssClass, facebookShareLinkEl ) - .add( isOnlineCssClass, pinterestShareLinkEl ) - .add( isOnlineCssClass, vkontakteShareLinkEl ); + .add( isOnlineCssClass, imgurLinkEl ) + .add( isOnlineCssClass, redditShareLinkEl ) + .add( isOnlineCssClass, vkontakteShareLinkEl ) + .add( isOnlineCssClass, pinterestShareLinkEl ); if ( browser.test( 'localforage' ) ) { sharedListEl = elHelper.createEl( 'ul', 'shared-list dialog-list' ); @@ -326,7 +326,7 @@ define( } if ( service === 'twitter' ) { - params.text = loc( 'share.link.description.withname' ) + ' ' + imgUrl + ' ' + config.share.appURL; + params.text = loc( 'share.link.description.withname' ) + ' ' + imgurLink + ' ' + config.share.appURL; } return config.share.sharer[service] + '?' + strHelper.objToQueryStr( params ); diff --git a/styles/modules/dialog.css b/styles/modules/dialog.css index 2783d76..ad8fa08 100644 --- a/styles/modules/dialog.css +++ b/styles/modules/dialog.css @@ -289,6 +289,16 @@ overflow: auto; } +.has-touch .share-dialog .dialog-item .button { + width: 100%; + text-align: center; +} + +.has-touch .share-dialog .entry-button-wrapper .button { + width: auto; + text-align: left; +} + /*pulse keyframe animation is inlined in html*/ .dialog .loader {