Skip to content
This repository has been archived by the owner on Mar 7, 2023. It is now read-only.

Fix preston position #123

Merged
merged 2 commits into from Aug 30, 2021
Merged

Fix preston position #123

merged 2 commits into from Aug 30, 2021

Conversation

NeOMakinG
Copy link

Questions Answers
Description? Preston on left was wrongly positionned
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes PrestaShop/PrestaShop#25527.
How to test? Install the module and see if every steps are fine visually

@NeOMakinG NeOMakinG added this to the 6.0.7 milestone Aug 11, 2021
@PierreRambaud
Copy link
Contributor

The problem is fixed with this PR without changing the css: #122 🤔

@NeOMakinG
Copy link
Author

The problem is fixed with this PR without changing the css: #122 🤔

If you compare the Update bourbon PR, you can see on line 55 on tooltips.scss that there is a mistake, I think you had cache or something like this

@hibatallahAouadni
Copy link

I couldn't test it cause the module doesn't work cause this PR #120 😭
So, we need to test this PR #122 before and merge it, then we can test this one and merge it!

@matks
Copy link
Contributor

matks commented Aug 30, 2021

@hibatallahAouadni #122 is merged 😉

@khouloudbelguith
Copy link

Hi @NeOMakinG,

It is ok ✔️
I checked with

  • PS1769 (PHP7.1)

  • 177x branch (PHP7.1)

  • 178x branch (PHP7.2)

  • develop branch (PHP7.4)

  • I tried with desktop (chrome + firefox)

  • I tried with mobile / tablet

https://drive.google.com/file/d/1R2fvyYY32fS-b6AgxO21gDDc2EvjkXj8/view?usp=sharing
Thanks!

@PierreRambaud PierreRambaud merged commit 9eb061b into PrestaShop:dev Aug 30, 2021
@PierreRambaud
Copy link
Contributor

Thank you @NeOMakinG

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
6 participants