Skip to content

Conversation

@zmousm
Copy link
Contributor

@zmousm zmousm commented Jan 6, 2025

No description provided.

Copy link
Collaborator

@Myshor Myshor left a comment

Choose a reason for hiding this comment

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

First of all... it would be nice to have some screens and/or URLs to see how the site looks before fixes and with fix.
2nd thing... I checked some pages in Firefox and found your fix not working so good... here is my URL:
https://techdocs.akamai.com/ion/docs/set-up-adaptive-acceleration so you will need to make some changes to it.
obraz

Last but not least: I checked the page https://techdocs.akamai.com/ion/docs/set-up-adaptive-acceleration on Chromium based browser without your fix and it works fine:
obraz

So even adding some fix here will need to check both kind of browsers if it's working fine in both.

@Myshor
Copy link
Collaborator

Myshor commented Jan 7, 2025

So... best thing here would be sharing URLs you were taking to fix so doing the new review we can check them in both kind of browsers and give a tip how to make fix working for both or even push the fix that will work fine in both. Now I do not even want to make any changes in this PR as I can make more damage than fix. 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants