Coder Social home page Coder Social logo

Comments (26)

ssborbis avatar ssborbis commented on June 30, 2024

For a brief sanity check, does the multi-line search work from the quick menu?

On the topic, I've been thinking a confirm popup would be a good idea for multi-line searches, when enabled. In other words, if you were to do a search on a block of text that had multiple lines, a YesNoCancel box would pop up, asking if you wanted the search to be multi-line ( search each line separately using the same engine ). What do you think?

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

No Mike, it doesn't seem working from anywhere... I think it would be better to have the option to activate it when needed from the settings or the context menu without the popup (and perhaps leading/trailing whitespaces should be removed). Thank you!

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Hmm, ok a few things.

  1. Multi-line search is working for me, as expected, when searching from the context menu
  2. not working from the quick menu
  3. Looks like i already coded a popup menu. The setting multilinesAsSeparateSearchesLimit is the number of lines to search separately before asking for confirmation. In other words, if this is set to 4, then 3 lines won't get a popup, but 5 will ask if you really want to search every line separately
  4. I'm not seeing leading or trailing whitespace in the searches

Please try searching again using the context menu using the example block of lines you provided using these settings
multilinesAsSeparateSearches: true multilinesAsSeparateSearchesLimit: 1

and see what happens.

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Thanks, I've changed the limit from 10 to 1, but I don't see any changing from context menu... mumble mumble...

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Hmm... what browser? Also, it might be useful if you posted your config for me to test. The code seem pretty basic, but maybe there's a conflict I'm not considering.

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Oh, and did you get a confirm dialog when you tried to search multiple lines? Setting multilinesAsSeparateSearchesLimit: 1 should force it to pop up on every multi-line search.

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Chrome, v. 114. No, I don't get a confirm dialog when I try to search multiple lines. Here are my settings, thanks.

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Hi Mike, do you need something else?

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Hi Mike, do you need something else?

Nope, I think i'm seeing the issues. I'll post here when I have a fix to test.

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

The potential fix is in the Master branch if you want to try sideloading. It is only intended to fix the quick menu issue.

In chrome variants, it seems the browser removes newlines from selected text when using the context menu. Separate issue I'll try to address later. For now, the QM appears to be working ( in Chromium, at least )

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Thanks, I'll wait an "official" fix for the context menu, thanks again!

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

The context menu appears to be fixed now as well. I'll post here when I get a release out.

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

1.46.11 going up now to chromestore & AMO

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Reopen if things are buggy. Looks good so far on FF and chromium

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Thank you for your work. The popup opens in both Chrome and Firefox, but after clicking on "ok" the popup does not disappear and often the page freezes and I have to restart the browser... I tried setting different multilinesAsSeparateSearchesLimit

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Thanks for pointing that out. I'm seeing this too. No idea how that glitch popped up.

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Should be fixed in 1.46.12, live now

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Thanks Mike, I hope in a soon fix as I'm experiencing some problems with current version (1.46.11): sometimes CS in context menu doesn't react and the fan begins to rotate, forcing me to restart... :(

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Did you get the update to 1.46.12?

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

I don't see it in the store, or do you mean sideloading?

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

Says "Pending Review". I'm going to republish. Sometimes the chromestore hangs on submissions.

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Okay, thank you

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Ehi Mike, I finally used your last version, seems working great so far, thank you very much! I tried copying the strings of the opening post in Google search field, and 4 browser tabs opened, that is what I desired, I don't know how your extension recognized they were separated string although they were joined together!! A question, is it possible let your extension work from Google Sheets (simple or multiline search)?

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

is it possible let your extension work from Google Sheets (simple or multiline search)?

Not currently. I'll need to look at what's possible with the api

from contextsearch-web-ext.

Parvares avatar Parvares commented on June 30, 2024

Ah okay, thanks again for your great work!

from contextsearch-web-ext.

ssborbis avatar ssborbis commented on June 30, 2024

No prob

from contextsearch-web-ext.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.