-
Notifications
You must be signed in to change notification settings - Fork 60
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
Patch 3 #15
base: master
Are you sure you want to change the base?
Patch 3 #15
Conversation
Added extraction of links from selection Added message "No links" Updated locale ru Added filteringDomains (addNodes)
Hi, please reply. Patch will be merged into the master branch? |
Thank you for the code. I would like a little time to review. When the review is all done and any issues are resolved, I will merge it into the master branch. (In general it is easier for me to review smaller chunks, such as one feature or one bug fix per pull request, but overall this looks like not a large change.) |
I tested the new functionality in Firefox 57, and the first three features work well. I started a new mini-site with testing pages. For example, there is a simple page without links. Would you please elaborate on filteringDomains? Also, would you please make two changes? First, it searches now by either the link URL or link text, but the user prompt is no longer accurate.
How about this?
This would be easy for me to change, but you would need to update the translation too. Second, searching by selection may not be intuitive because it not stated in the user interface. If the user highlights a portion, he may be surprised not all links are returned. If the user wants only a portion of links to be returned, he may not know that highlighting enables this functionality. Therefore, I propose to add a checkbox in The default state of the checkbox is either
|
I knew that this question will be! :) That's my fault...
I agree. It can be done
Create extra controls... |
Added filtering on text
Added extraction of links from selection
Added message "No links"
Updated locale ru
Added filteringDomains (addNodes)