From 7a7c756b0b0460c0a752ff55154c5b352069b6bf Mon Sep 17 00:00:00 2001 From: Thomas Steur <tsteur@users.noreply.github.com> Date: Mon, 25 Sep 2017 14:59:54 +1300 Subject: [PATCH] No longer show "this is demo.piwik.org" message in top control (#12111) * no longer show "this is demo.piwik.org" message in top control * update key --- lang/en.json | 2 +- plugins/CoreHome/javascripts/top_controls.js | 10 -------- plugins/CoreHome/stylesheets/layout.less | 5 ---- .../CoreHome/templates/_headerMessage.twig | 24 +++---------------- 4 files changed, 4 insertions(+), 37 deletions(-) diff --git a/lang/en.json b/lang/en.json index 7d9815d9bf..0be70f5a08 100644 --- a/lang/en.json +++ b/lang/en.json @@ -432,7 +432,7 @@ "YearsDays": "%1$s years %2$s days", "Yes": "Yes", "YouAreCurrentlyUsing": "You are currently using Piwik %s.", - "YouAreViewingDemoShortMessage": "You are viewing the demo of Piwik", + "YouAreViewingDemoMessage": "You are viewing the demo of %1$sPiwik%2$s", "YouMustBeLoggedIn": "You must be logged in to access this functionality.", "YourChangesHaveBeenSaved": "Your changes have been saved." }, diff --git a/plugins/CoreHome/javascripts/top_controls.js b/plugins/CoreHome/javascripts/top_controls.js index 7c6a707067..611bc655d4 100644 --- a/plugins/CoreHome/javascripts/top_controls.js +++ b/plugins/CoreHome/javascripts/top_controls.js @@ -48,16 +48,6 @@ function initTopControls() { $('.top_controls').css('opacity', '1'); } - var header = $('#header_message.isPiwikDemo'); - if (header.length) { - // make sure isPiwikDemo message is always fully visible, move it to the right if needed - var lastSelector = $('.top_controls .piwikTopControl:last'); - - var overlap = getOverlap(header[0], lastSelector[0]); - if (header[0] !== lastSelector[0] && overlap !== 0) { - header.css('right', (Math.abs(overlap) + 18) * -1); - } - } } } diff --git a/plugins/CoreHome/stylesheets/layout.less b/plugins/CoreHome/stylesheets/layout.less index dac9dd0925..4be7e63dc7 100644 --- a/plugins/CoreHome/stylesheets/layout.less +++ b/plugins/CoreHome/stylesheets/layout.less @@ -112,7 +112,6 @@ nav { .top_controls { height: auto; - .isPiwikDemo, .piwikTopControl { position: static !important; float: none; @@ -135,10 +134,6 @@ nav { left: 30px; } } - - .isPiwikDemo { - margin-top: 8px; - } } } diff --git a/plugins/CoreHome/templates/_headerMessage.twig b/plugins/CoreHome/templates/_headerMessage.twig index ee3b5af8f8..91858adf66 100644 --- a/plugins/CoreHome/templates/_headerMessage.twig +++ b/plugins/CoreHome/templates/_headerMessage.twig @@ -10,17 +10,13 @@ </span> {% endset %} -{% if isPiwikDemo or (latest_version_available and hasSomeViewAccess and not isUserIsAnonymous) or (isSuperUser and isAdminArea is defined and isAdminArea) %} +{% if (latest_version_available and hasSomeViewAccess and not isUserIsAnonymous) or (isSuperUser and isAdminArea is defined and isAdminArea) %} <div piwik-expand-on-hover id="header_message" - class="piwikSelector borderedControl {% if isPiwikDemo or not latest_version_available %}header_info{% else %}{% endif %} {% if isPiwikDemo %}isPiwikDemo{% else %}piwikTopControl{% endif %} {% if latest_version_available %}update_available{% endif %}" + class="piwikSelector borderedControl {% if not latest_version_available %}header_info{% else %}{% endif %} piwikTopControl {% if latest_version_available %}update_available{% endif %}" > - {% if isPiwikDemo %} - <a class="title" style="cursor:default;"> - {{ 'General_YouAreViewingDemoShortMessage'|translate }} - </a> - {% elseif latest_version_available %} + {% if latest_version_available and not isPiwikDemo %} <a class="title" href="?module=CoreUpdater&action=newVersionAvailable" style="cursor:pointer;"> {{ 'General_NewUpdatePiwikX'|translate(latest_version_available) }} <span class="icon-warning"></span> @@ -32,13 +28,6 @@ {% endif %} <div class="dropdown positionInViewport"> - {% if isPiwikDemo %} - {{ 'General_DownloadFullVersion'|translate("<a rel='noreferrer' href='https://piwik.org/'>","</a>","<a rel='noreferrer' href='https://piwik.org'>piwik.org</a>")|raw }} - <br/> - {% if isSuperUser and isAdminArea is defined and isAdminArea %} - <br/> - {% endif %} - {% endif %} {% if latest_version_available and isSuperUser %} {% if isMultiServerEnvironment %} {{ 'CoreHome_OneClickUpdateNotPossibleAsMultiServerEnvironment'|translate("<a rel='noreferrer' href='https://builds.piwik.org/piwik-" ~ latest_version_available ~ ".zip'>","</a>")|raw }} @@ -52,13 +41,6 @@ <br /> {% endif %} - {% if isPiwikDemo and isSuperUser and isAdminArea is defined and isAdminArea %} - <br/> - {{ updateCheck|raw }} - <br/> - <br/> - {% endif %} - {{ 'General_YouAreCurrentlyUsing'|translate(piwik_version) }} </div> </div> -- GitLab