-
Notifications
You must be signed in to change notification settings - Fork 76
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
Fix : [FO] FILTERING - Resize screen to mobile #495
Fix : [FO] FILTERING - Resize screen to mobile #495
Conversation
ga-devfront
commented
May 4, 2023
•
edited
Loading
edited
Questions | Answers |
---|---|
Description? | Fix the problem of front office about filtering resize screen to mobile. |
Type? | bug fix |
BC breaks? | no |
Deprecations? | no |
Fixed ticket? | Fixes #483 |
Sponsor company | @PrestaShopCorp. |
How to test? | Test with the explination inside the ticket. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import initResponsiveToggler from './responsive-toggler'; | ||
import initEmitter from './prestashop'; | ||
import { | ||
desktopElement, mobileElement, mobileElementId, contentValue, desktopElementId, | ||
} from './constants/mocks/swapElements-data'; | ||
import ThemeType = Theme.ThemeType; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ga-devfront why this import? seems useless.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on my side then I test without this it fail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.