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

(Github Pull Request From Link) Underlines : and marks it #93

Closed
rugk opened this issue Dec 10, 2015 · 4 comments
Closed

(Github Pull Request From Link) Underlines : and marks it #93

rugk opened this issue Dec 10, 2015 · 4 comments

Comments

@rugk
Copy link

rugk commented Dec 10, 2015

It's best to show it like this:
underlines
markstext

used #12
Firefox 42.0

@jerone
Copy link
Owner

jerone commented Dec 10, 2015

Underlines :

I kind of left this intentionally to let users know that's a link now.

and marks it

This is somewhat a feature from GitHub. If you disable the userscript it lets you select the whole user+branch, I guess for easy coping. Now that this is a link, GitHub's feature is probably unnecessary.

@rugk
Copy link
Author

rugk commented Dec 10, 2015

@jerone commented on 10. Dez. 2015, 21:45 MEZ:

I kind of left this intentionally to let users know that's a link now.

But as you can see it is really only the character : (and this character is not in every PR). I mean should not it be either the whole branch name or not be underlined at all?

This is somewhat a feature from GitHub. If you disable the userscript it lets you select the whole user+branch, I guess for easy coping. Now that this is a link, GitHub's feature is probably unnecessary.

Ah not really in my test:
withoutuserscript

However I also could not reproduce the marking with your userscript anymore after I reenabled this...

@jerone
Copy link
Owner

jerone commented Dec 10, 2015

Ah not really in my test:

You have to click left from the username. It's kind of a bug that this feature doesn't work on children of an .user-select-contain element (probably a left-over from latest site update)...

$(document).on("click", ".user-select-contain", function() {
        var t, n;
        n = window.getSelection(),
        n.rangeCount && (t = n.getRangeAt(0).commonAncestorContainer,
        e(this, t) || n.selectAllChildren(this))
    }))

However I also could not reproduce the marking with your userscript anymore after I reenabled this...

After navigating to another page and back the feature is back again (probably also a bug).

@jerone jerone closed this as completed in 0369357 Dec 10, 2015
@rugk
Copy link
Author

rugk commented Dec 10, 2015

You have to click left from the username.

Eh, what?
I clicked everywhere there and it did not worked. (just tried it again)

However this does not really matter as I do not want to use it anyway. 😃

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

No branches or pull requests

2 participants