Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BO: Easier way to go to Maintenance page while shop is turned off #5612

Merged
merged 1 commit into from
May 23, 2016

Conversation

kpodemski
Copy link
Contributor

Questions Answers
Branch? develop
Description? I had some troubles finding maintenance page so I decided to make alert about enabled maintenance mode clickable
Type? improvement
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? n/a
How to test? Turn on maintenance mode, go to Maintenance page from link at the top of the page

Important guidelines

@@ -308,8 +308,9 @@
{if isset($maintenance_mode) && $maintenance_mode == true}
<span class="shop-state" id="maintenance-mode">
<i class="material-icons">build</i>
<span class="label-tooltip" data-toggle="tooltip" data-placement="bottom" data-html="true"
title="<p class='text-left text-nowrap'><strong>{l s='Your shop is in maintenance.'}</strong></p><p class='text-left'>{l s='Your visitors and customers cannot access your shop while in maintenance mode.%s To manage the maintenance settings, go to Preferences > Maintenance.' sprintf='<br />'}</p>">{l s='Maintenance mode'}</span>
<a class="label-tooltip" data-toggle="tooltip" data-placement="bottom" data-html="true"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

except if I'm wrong, paragraph element is not allowed in a link

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

paragraph in span is also not a good idea but this paragraph is only for a tooltip, right?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree with you paragraph in span is also a bad idea :/ let's validate product interest with @vincentbz before any git update, I dont want you to work for nothing :)

@mickaelandrieu
Copy link
Contributor

mickaelandrieu commented May 21, 2016

Definitively a feature I agree with :)

Can you take a look @vincentbz ?

@mickaelandrieu mickaelandrieu added the Waiting for QA Status: action required, waiting for test feedback label May 21, 2016
@vincentbz vincentbz added QA ✔️ Status: check done, code approved Waiting for QA Status: action required, waiting for test feedback PM ✔️ Status: check done, behavior approved and removed Waiting for QA Status: action required, waiting for test feedback QA ✔️ Status: check done, code approved labels May 23, 2016
@vincentbz
Copy link
Contributor

Hi @kpodemski ,

Thanks for this PR. I just need to test if there is no conflict with another PR made by our team.

@nihco2 nihco2 merged commit ab4f9aa into PrestaShop:develop May 23, 2016
@vTerenti vTerenti added QA ✔️ Status: check done, code approved and removed Waiting for QA Status: action required, waiting for test feedback labels May 23, 2016
@kpodemski kpodemski deleted the maintenance-link branch July 4, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PM ✔️ Status: check done, behavior approved QA ✔️ Status: check done, code approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants