From f3bc3e94e72d89c5837b84312099d2ba33d1dd42 Mon Sep 17 00:00:00 2001 From: jcass Date: Sun, 8 Jan 2017 08:33:31 +0200 Subject: [PATCH] Tweak popup dialog look and feel for mobile and desktop devices. --- mopidy_musicbox_webclient/static/css/webclient.css | 11 ++++++++--- mopidy_musicbox_webclient/static/index.html | 6 ++---- mopidy_musicbox_webclient/static/js/controls.js | 7 +++++-- .../static/js/functionsvars.js | 13 +++++++++---- mopidy_musicbox_webclient/static/mb.appcache | 2 +- 5 files changed, 25 insertions(+), 14 deletions(-) diff --git a/mopidy_musicbox_webclient/static/css/webclient.css b/mopidy_musicbox_webclient/static/css/webclient.css index 5cbbe85..b5d668c 100644 --- a/mopidy_musicbox_webclient/static/css/webclient.css +++ b/mopidy_musicbox_webclient/static/css/webclient.css @@ -1,6 +1,6 @@ /* * Mopidy Webclient CSS - * (c) Wouter van Wijk 2012-2013 + * (c) Wouter van Wijk 2012-2017 */ /**************************** @@ -578,8 +578,9 @@ a { color: #efefef; } -/* panel workaround to make it responsive wrap push on wide viewports once open */ -@media (min-width: 35em){ +/*tablets and desktop*/ +@media (min-width: 35em) { + /* panel workaround to make it responsive wrap push on wide viewports once open */ .ui-responsive-panel.ui-page-panel-open .ui-panel-content-fixed-toolbar-open.ui-panel-content-fixed-toolbar-display-push, .ui-responsive-panel.ui-page-panel-open .ui-panel-content-fixed-toolbar-open.ui-panel-content-fixed-toolbar-display-reveal, .ui-responsive-panel.ui-page-panel-open .ui-panel-content-wrap-open.ui-panel-content-wrap-display-push, @@ -597,6 +598,10 @@ a { .ui-responsive-panel .ui-panel-dismiss-display-reveal { display: none; } + + .popupDialog { + min-width: 320px; + } } /*smartphones*/ diff --git a/mopidy_musicbox_webclient/static/index.html b/mopidy_musicbox_webclient/static/index.html index 9c94983..cbe6db7 100644 --- a/mopidy_musicbox_webclient/static/index.html +++ b/mopidy_musicbox_webclient/static/index.html @@ -195,9 +195,7 @@ -
- -
+
- Close + Close
diff --git a/mopidy_musicbox_webclient/static/js/controls.js b/mopidy_musicbox_webclient/static/js/controls.js index a22093e..1eb6085 100644 --- a/mopidy_musicbox_webclient/static/js/controls.js +++ b/mopidy_musicbox_webclient/static/js/controls.js @@ -398,8 +398,11 @@ $('#popupShowInfo #uri-cell').val(uri) showLoading(false) $('#popupShowInfo').popup('open') - $('#popupShowInfo #uri-cell').focus() - $('#popupShowInfo #uri-cell').select() + if (!isMobile()) { + // Set focus and select URI text on desktop systems (don't want the keyboard to pop up automatically on mobile devices) + $('#popupShowInfo #uri-cell').focus() + $('#popupShowInfo #uri-cell').select() + } }, console.error) }, diff --git a/mopidy_musicbox_webclient/static/js/functionsvars.js b/mopidy_musicbox_webclient/static/js/functionsvars.js index a06a2a7..4b0432d 100644 --- a/mopidy_musicbox_webclient/static/js/functionsvars.js +++ b/mopidy_musicbox_webclient/static/js/functionsvars.js @@ -42,11 +42,8 @@ var customTracklists = [] // TODO: Refactor into one shared cache var browseStack = [] -var ua = navigator.userAgent +var ua = navigator.userAgent || navigator.vendor || window.opera var isMobileSafari = /Mac/.test(ua) && /Mobile/.test(ua) -var isMobileWebkit = /WebKit/.test(ua) && /Mobile/.test(ua) -var isMobile = /Mobile/.test(ua) -var isWebkit = /WebKit/.test(ua) // constants PROGRAM_NAME = $(document.body).data('program-name') @@ -174,6 +171,14 @@ function scrollToTracklist () { }, 250) } +function isMobile () { + // Checks for known mobile and tablet devices - see http://stackoverflow.com/questions/11381673/detecting-a-mobile-browser + var regexpMobile = /(android|bb\d+|meego).+mobile|avantgo|bada\/|blackberry|blazer|compal|elaine|fennec|hiptop|iemobile|ip(hone|od)|iris|kindle|lge |maemo|midp|mmp|mobile.+firefox|netfront|opera m(ob|in)i|palm( os)?|phone|p(ixi|re)\/|plucker|pocket|psp|series(4|6)0|symbian|treo|up\.(browser|link)|vodafone|wap|windows ce|xda|xiino|android|ipad|playbook|silk/i + var regexpTablet = /1207|6310|6590|3gso|4thp|50[1-6]i|770s|802s|a wa|abac|ac(er|oo|s\-)|ai(ko|rn)|al(av|ca|co)|amoi|an(ex|ny|yw)|aptu|ar(ch|go)|as(te|us)|attw|au(di|\-m|r |s )|avan|be(ck|ll|nq)|bi(lb|rd)|bl(ac|az)|br(e|v)w|bumb|bw\-(n|u)|c55\/|capi|ccwa|cdm\-|cell|chtm|cldc|cmd\-|co(mp|nd)|craw|da(it|ll|ng)|dbte|dc\-s|devi|dica|dmob|do(c|p)o|ds(12|\-d)|el(49|ai)|em(l2|ul)|er(ic|k0)|esl8|ez([4-7]0|os|wa|ze)|fetc|fly(\-|_)|g1 u|g560|gene|gf\-5|g\-mo|go(\.w|od)|gr(ad|un)|haie|hcit|hd\-(m|p|t)|hei\-|hi(pt|ta)|hp( i|ip)|hs\-c|ht(c(\-| |_|a|g|p|s|t)|tp)|hu(aw|tc)|i\-(20|go|ma)|i230|iac( |\-|\/)|ibro|idea|ig01|ikom|im1k|inno|ipaq|iris|ja(t|v)a|jbro|jemu|jigs|kddi|keji|kgt( |\/)|klon|kpt |kwc\-|kyo(c|k)|le(no|xi)|lg( g|\/(k|l|u)|50|54|\-[a-w])|libw|lynx|m1\-w|m3ga|m50\/|ma(te|ui|xo)|mc(01|21|ca)|m\-cr|me(rc|ri)|mi(o8|oa|ts)|mmef|mo(01|02|bi|de|do|t(\-| |o|v)|zz)|mt(50|p1|v )|mwbp|mywa|n10[0-2]|n20[2-3]|n30(0|2)|n50(0|2|5)|n7(0(0|1)|10)|ne((c|m)\-|on|tf|wf|wg|wt)|nok(6|i)|nzph|o2im|op(ti|wv)|oran|owg1|p800|pan(a|d|t)|pdxg|pg(13|\-([1-8]|c))|phil|pire|pl(ay|uc)|pn\-2|po(ck|rt|se)|prox|psio|pt\-g|qa\-a|qc(07|12|21|32|60|\-[2-7]|i\-)|qtek|r380|r600|raks|rim9|ro(ve|zo)|s55\/|sa(ge|ma|mm|ms|ny|va)|sc(01|h\-|oo|p\-)|sdk\/|se(c(\-|0|1)|47|mc|nd|ri)|sgh\-|shar|sie(\-|m)|sk\-0|sl(45|id)|sm(al|ar|b3|it|t5)|so(ft|ny)|sp(01|h\-|v\-|v )|sy(01|mb)|t2(18|50)|t6(00|10|18)|ta(gt|lk)|tcl\-|tdg\-|tel(i|m)|tim\-|t\-mo|to(pl|sh)|ts(70|m\-|m3|m5)|tx\-9|up(\.b|g1|si)|utst|v400|v750|veri|vi(rg|te)|vk(40|5[0-3]|\-v)|vm40|voda|vulc|vx(52|53|60|61|70|80|81|83|85|98)|w3c(\-| )|webc|whit|wi(g |nc|nw)|wmlb|wonu|x700|yas\-|your|zeto|zte\-/i + var uaString = ua.substr(0, 4) + return regexpMobile.test(uaString) || regexpTablet.test(uaString) +} + // A hack to find the name of the first artist of a playlist. this is not yet returned by mopidy // does not work wel with multiple artists of course function getArtist (pl) { diff --git a/mopidy_musicbox_webclient/static/mb.appcache b/mopidy_musicbox_webclient/static/mb.appcache index 738559a..f505d02 100644 --- a/mopidy_musicbox_webclient/static/mb.appcache +++ b/mopidy_musicbox_webclient/static/mb.appcache @@ -1,6 +1,6 @@ CACHE MANIFEST -# 2017-01-07:v1 +# 2017-01-08:v1 NETWORK: *