I think it would be a good idea to add a hover state for basic links in the new Seven theme. It helps provide a visual queue for users and is I believe a good practice for web design in general.

The attached patch just ads a simple underline to text links.

CommentFileSizeAuthor
seven-link-hover.jpg58.08 KBeojthebrave
eojthebrave-seven_link_hover.patch280 byteseojthebrave

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Issue tags: +Needs design review

I like this. What say other designers?

Zarabadoo’s picture

Zarabadoo commented

You will have no arguments from me. It is definitely a nice touch.

Bojhan’s picture

Bojhan commented

Would find it strange if mark didn't design for this.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Bojhan: Meaning "Let's verify this change with Mark" or meaning "No duh"? ;)

If the latter, please feel free to mark RTBC.

Bojhan’s picture

Bojhan commented

Yhea, the former - I will verify.

Status: Needs review » Needs work

The last submitted patch failed testing.

sun.core’s picture

sun.core commented
Status: Needs work » Needs review
eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented

I think the testing bot was borked. Still applies fine and looks like it passes the tests.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community
xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
commented
Issue tags: +Quick fix, +Usability

This is a great usability addition as well as a design nice touch.

When working with Photoshop to create interfaces, it's often forgot to add usability touches like this.

yoroy’s picture

yoroy commented

Agreed with using a simple underline. RTBC indeed.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Usability, -Needs design review

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Quick fix, +Usability, +Needs design review

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.