-
Notifications
You must be signed in to change notification settings - Fork 278
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
Text selection inside links #109
Comments
Fore surely, it's "must have" feature |
This one is not really doable. :-/ |
+50 to this feature! |
You can try to report it upstream (I couldn't fin existing ticket for this issue) and post link to issue in comments. |
Is it possible to implement such behavour on a top of Gecko engine? |
@unxed, I guess it has to be implemented in engine itself, AFAIK only Presto allowed this behavior... |
@Emdek, I found this blog post today. It is about an extension for Blink-Chropera that supposedly allows selection inside hyperlinks. Considering the origin of Blink, could a similar workaround be coded for Otter? Would it be too much trouble? Do you think it may have a negative impact on performance? |
@beastie1, interesting. |
From my experience, I think the Old Opera works like this:
|
Source can be easily deobfuscated, I'll post a gist with the deobfuscated version today. |
Haven't tested it. If javascript-based solution is acceptable, I can write something like this. By the way, should script-based link text selecting be optional? Maybe there should be some options in about:config. |
@resetnow, I would like to avoid adding option for that. |
@Emdek, I think this script will work almost always. However, native solution would be better, but looks like there's no way to implement it without modifying webkit sources. |
@resetnow, maybe Opera developers will someday implement that in Blink. ;-) |
The blog linked above seems to be updated with this extension https://gist.github.com/Griever/5005165 I haven't tested it. I also want to mention I find this a quite serious usability issue (more important than missing mail client for example). I see how it's not feasible to maintain own WebKit fork (assuming it's reasonably fixable in there) at this stage, but I wouldn't close this just yet (unless the js workaround works properly). |
@rplnt, this version looks more promising. |
Since we have User Script for some time already we could do it as bundled and enabled by default User Script. |
@resetnow proposed to do it in his comment. He seems to be active on Github so maybe he could do it? |
The script mentioned above (https://gist.github.com/Griever/5005165) works just fine for me, at least with QtWebEngine. It's also MIT licensed, is this license compatible with Otter? |
According to the GNU project, the MIT license is a name given to either the X11 license and the Expat. Both are compatible with the GPL v3 that Otter uses. https://www.gnu.org/licenses/license-list.html#X11License |
If we are going to provide official user scripts, it may be a good idea to indicate which scripts are provided by default and are officially supported, and which are not. |
Right, somehow I remembered that there was only some totally unreadable JS code to solve that... |
this one? |
@unxed, no, I remember something obfuscated or at least minified, either someone gave me link outside this ticket or I've found it on the net. |
@unxed, nope, and there is no point for looking it up now. ;-) |
A previous version of the extension mentioned above (which is now removed) was obfuscated. |
You're talking about the one mentioned here. |
@resetnow, @beastie1, yeah, probably that one. |
Opera 12 has an ability to select text placed inside A tag. It would be nice to have the same option included in Otter Browser.
The text was updated successfully, but these errors were encountered: