This topic has 7 replies, 2 voices, and was last updated 7 years, 1 months ago ago by Olga Barlow
Hi there!
I found 3 bugs^^.
1. I like the new smart fixed menu. but there is a problem with page scroll animations. You put a “var top_offset” in javascript files… But when I choose the smart menu there is no need for a offset value. The value must be “0”.
Could you please fix that in the next release :)? I put an example in hidden content but I think you used the scroll offset severyl times in your javascript code…
2. Small issue is, that when I want to link to the review section I can not link to it… Please see the link in hidden content. “http://…../#reviews
Thank you very much!
Best,
Tobi
Hello Olga,
I only see your answer as email…
That is correct:
2) Because that tab has different id http://……/#tab_reviews
But it doesnt work. You can try. The tab wont open…
Thank you!
Hello,
1) Developers ask if you are talking about startOffset maybe? Because there is no top_offset in js.
2) We have fixed the #reviews link. Try now.
Regards
Hello Olga,
#reviews link is working now. Please adapt it with the next update for other users ;)…
I tested the smart fixed menu and I think it is not so nice for the shopping experience because when you add a product to the cart you won’t see the “mini cart product add animation”. You see the “8” symbol while adding products to the cart, then it disappears and nothing :). So I think I will stick to the normal fixed menu. I modified the height of the fixed menu by myself so it wont hide so much of the content on smartphones :)…
The issue with the scroll fucntion is, that it is not scrolling far enough – especially on smartphones. My maximum top_offset should be 85px because that’s the maximum height of my fixed menu. Could you change this for all please? Please watch the video in hidden content.
THANKS!
Hello,
Yes, reviews fix will be included.
Ok, we’ll try to improve sticky header.
Regards
Hi Olga,
thank you. But what is with the space when scroll to review? Will you fix this in next update :)?
Hello,
Yes, we’ll fix.
Regards
You must be logged in to reply to this topic.Log in/Sign up