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

Link hover #45

Closed
wants to merge 1 commit into from
Closed

Link hover #45

wants to merge 1 commit into from

Conversation

Arty2
Copy link

@Arty2 Arty2 commented Aug 14, 2015

Per Dave’s suggestion over at #43 (comment), I’m proposing a PR for colored links on hover.

It’s been 11 years since Jacob Nielsen’s guidelines and a lot of sites nowadays —notably Medium— don’t offer different styles for hover.

They may only be relevant to desktop users, but in my opinion hover styles are indispensable in how they strongly inform the reader that the underlined phrase they put their cursor on, is actually a link.

Sublte transition and color, border-color change to #f05
@crmackay
Copy link
Contributor

I'm ok with a :hover style for links....though I'm not sure that using the same color as the sidenote numbers makes sense.

here is a showcase of some more creative link hover effects if you're interested in playing around with some thing a bit different. They would need to be adapted somehow to fit the aesthetic of tufte-css: http://tympanus.net/Development/CreativeLinkEffects/

@daveliepmann
Copy link
Contributor

I definitely don't want anything unusual or out of the ordinary for hover effects in this project.

I agree that a color change might make sense, but I'm really hesitant to use red. My instinct is to stick to greyscale.

@bolatovumar bolatovumar mentioned this pull request Dec 25, 2015
@daveliepmann
Copy link
Contributor

After refinements to project goals with regards to hyperlinks—specifically, emphasizing readability and unobtrusiveness—hover styles have become less compelling. Thanks for your contribution but I'm going to close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants