• Priority: 0
  • Status: Closed
  • Theme: Alysum
  • Assigned To: Fred
  • Private: No
  • Open Date: 22.07.21, 13:39
  • Opened by: Lisa Keeling
  • Closed by: Anonymous Submitter
  • Closed on:
  • Reason: Not a bug

Ticket #21991 - AMP Wishlist, My Alerts not switching to mobile

Hello, On AMP under logged into My Account, the My Alerts and Wishlist links dont seem to be switching to AMP and have a mixed menu (see screenshots). Also, when you hit the menu (3 dashes button) this is bringing up a large white empty box / not the menu. How do we fix this? Screenshot attached.

Comments

Fred 23 Jul 2021, 08:39

Hi, Lisa.
1. Yes Wishlist is not compatible with AMP. We have removed that outdated module from the latest Alysum theme version
2. The screenshot is not attached, please try once more

Lisa Keeling 23 Jul 2021, 09:52

Screenshots attached as discussed. This makes sense why it is not switching to mobile view and showing poor ‘responsive’ version.. Is this still the situation for AMP - both My Alerts + Wishlist? We may need to disable them for desktop if so as 80% of our browsing is done via mobile. No one really uses these functions so it’s not a massive problem. Hopefully it does not require much adjustment from a css perspective to disable for desktop. Let me know and we can figure that out :-)

Fred 25 Jul 2021, 09:06

Could you please give me a link to a page where I can see My Alerts module?

Lisa Keeling 26 Jul 2021, 11:08

Hi Fred, It’s a mobile link from my phone (so should be /m) attached image and link below I have copied and pasted from my phone:

https://dev.londoncopyright.com/my-alerts

TY

Attachments:

IMG_ED10DAB79404-1.jpeg (265.5 KiB)
Fred 27 Jul 2021, 14:56

try following CSS fix:

@media (max-width:768px) {
#header .header-main > .page-width {height:auto}
#header .main-header {flex-direction: column}
}
Lisa Keeling 27 Jul 2021, 16:47

Hi Fred, this CSS change is attempting to make it responsive for mobile right? Rather than AMP - which you said both My Alerts + Wishlists are not AMP compatible. It doesn’t work (screenshots attached) but we wouldn’t want some of the /m being desktop responsive and some converting to AMP - so we may have to disable these for desktop and mobile if there is no AMP solution. You have no plans to enable these modules for AMP again?

Attachments:

IMG_1067.PNG (356.9 KiB)
IMG_1068.PNG (329.4 KiB)
IMG_1069.PNG (336.8 KiB)
Fred 27 Jul 2021, 20:01

Yes, there are no AMP version for both modules. I can add to our todo to “ampify” those modules. But we can do that only after vacation

Lisa Keeling 28 Jul 2021, 10:33

Thank you. Can we disable them for AMP and leave them on desktop in the meantime? If so, how do I do that? If you are going to Ampify them eventually then there is no point in us making CSS changes to adapt desktop again without them.

Fred 28 Jul 2021, 11:07

Yes, you can disable it for a while

Lisa Keeling 28 Jul 2021, 11:36

Hi Fred, how do we do that for AMP and not desktop?

Fred 28 Jul 2021, 11:44

Sorry, I misread your previous message. I read that you want to disable AMP module.
There is no way to disable some module for AMP. You can only hide their links with custom CSS

Lisa Keeling 28 Jul 2021, 11:45

Can you share this CSS so I can add it?

Fred 28 Jul 2021, 21:17

really hard to provide custom work now because of vacation
We recommend to use Developer Tools of your browser to change such small things https://www.youtube.com/watch?v=wcFnnxfA70g

Lisa Keeling 29 Jul 2021, 11:10

Thanks Fred, we do use this - more for inspecting than managing any changes. I will look into it a bit further for that purpose. Hppe youre enjoying your holiday!

Fred 30 Jul 2021, 08:22

Thank you for understanding