Skip to content
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

[Bug] lag on some pages? #124

Closed
tmpm697 opened this issue May 28, 2023 · 10 comments · Fixed by #126
Closed

[Bug] lag on some pages? #124

tmpm697 opened this issue May 28, 2023 · 10 comments · Fixed by #126
Labels
bug Something isn't working

Comments

@tmpm697
Copy link

tmpm697 commented May 28, 2023

info:

chromium --version: Chromium 114.0.5735.45 Arch Linux
mark my search: 1.13.7 -- latest
os: linux

steps to produce:

1. open new tab and type in address bar: aws_network_acl resource terraform
2. click on first page, should be like [this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/network_acl)
3. observe page long loading to display content, chromium sometime gives box asking to continue waiting for kill the tab --> tab not responding

note that the above issue happen sometime in busy work with multiple tab or randomly that i don't know, but it happens quite often for me.

i tried to find an option in settings to disable mark my search on a certain but not found or an option to not inherit search to a cerntain page.

@ator-dev
Copy link
Member

ator-dev commented May 28, 2023

Thanks for the report!

I can reproduce this in all browsers when a specific option is OFF. If I keep "CLASSIC" highlighting on, the page loads quickly. Could you check whether or not you have it enabled please?

image

I will do some more diagnosis, but it may take a while to release a fix, due to student work.

Edit: The options can be found by opening the extension popup (from the extensions icon). The setting should be ON by default.

@tmpm697
Copy link
Author

tmpm697 commented May 29, 2023

Yes, I used the experimental APIs option as it's critical to display searches with different colors.

@ator-dev
Copy link
Member

ator-dev commented May 29, 2023

Thanks again for reporting this, it's very much appreciated (:

Your issue led me down several rabbitholes, which showed me some severe deficiencies in the PAINT highlighting algorithm (the method used when "Use CLASSIC highlighting" is disabled). As such, I have made massive optimisations, some of which will affect regular users, making PAINT an extremely efficient approach. Your AWS page now loads instantly, and several other pages I stress-tested are almost without lag.

I have also fixed the scroll markers. They were not working properly due to an edge case on Chromium when using PAINT.

If you experience any more page freezes, please report them again, as they should not be happening!

Edit: Note that the update will not be released instantly. Please bear with me as I need to make sure I didn't break anything critical, especially since I merged with an unstable branch recently.

@ator-dev
Copy link
Member

1.14.0 now live on Chrome Web Store. You should also notice that MMS loads a lot more quickly in general 🙂

@tmpm697
Copy link
Author

tmpm697 commented May 30, 2023

thanks, just tested, it's fast, but visibility when highlight search engine should default enabled? (both options)

@ator-dev
Copy link
Member

ator-dev commented May 30, 2023

I'm confused - do you mean the "Visibility when highlighting search engine keywords" options should be enabled by default?

"Highlights begin visible" means that if started from a search, highlights display at first (otherwise you can use the eye button or a shortcut to turn them on).

"Highlights are always visible on search pages" means that regardless of the above setting, highlights are always displayed on search pages.

@tmpm697
Copy link
Author

tmpm697 commented May 31, 2023

I meant this "Highlights are always visible on search pages" means that regardless of the above setting, highlights are always displayed on search pages.

Reading two options give a bit confuse, but I want it default just to highlight different colors right a way when using without toggle option in settings --> this can be most wanted if new users start using this plugin I assume.

@ator-dev
Copy link
Member

ator-dev commented May 31, 2023

(Quick note, I described the inverse of the first option, and have now edited)

I want it default just to highlight different colors

That should be true already. The default is that highlights are always shown at first ("Highlights begin visible" is on when installed). The 2nd option is only useful if the 1st one is off.

Was this not true when you installed?

@tmpm697
Copy link
Author

tmpm697 commented May 31, 2023

just checked, enable Highlights begin visible is enough and it seem it's enabled by default, don't know why last my last check I lose colors for a while and need to check both options.

Anyway it's good now, thanks for help.

@ator-dev
Copy link
Member

Strange, thanks for verifying. I'll keep an eye on it (:

Since you mentioned the options being confusing, here's a preview of the new options page I am building. It's planned to release with support for better storage defaults and clearer options with tooltips.

image

By the way, please consider writing a review if you enjoy MMS. It has taken a lot of work and it will always be free. A positive review will help others to discover it.

Thank you for the help 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants