header separator cursor set to crosshair #546

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@cswider
Contributor
cswider commented Sep 28, 2012

Simply changed the hover cursor for the separators to a crosshair

@chromakode
Contributor

Why?

@cswider
Contributor
cswider commented Sep 28, 2012

Well, I thought that the current cursor, the text highlight one, didn't quite fit seeing as you can't input text into the separators or anywhere else in the header.
After seeing other cursors, I thought the crosshair looked the best as it didn't interrupt the function of the header-bottom-right at all and wouldn't be confusing to users.

@chromakode
Contributor

How about cursor: default?

@cswider
Contributor
cswider commented Sep 28, 2012

huh. I guess that looks fine too.
I was also drawing inspiration from the question mark on userkarma; I appreciate details like special cursors for seemingly insignificant things like separators and the such.

@chromakode
Contributor

If the element isn't meant to be interacted with, the default cursor is probably most appropriate. :)

Could you please update your pull request to use the default cursor?

@cswider
Contributor
cswider commented Sep 28, 2012

Alright, that makes sense.
Will do.

@cswider
Contributor
cswider commented Sep 28, 2012

"If the element isn't meant to be interacted with, the default cursor is probably most appropriate."
This was somewhat my thinking in that the text cursor didn't quite fit.

@chromakode
Contributor

Merged: d3f8229

Are you /u/cswider?

@chromakode chromakode closed this Oct 9, 2012
@cswider
Contributor
cswider commented Oct 10, 2012

Yes sir I am.

@chromakode
Contributor

Check your user page for something new. :)

@cswider
Contributor
cswider commented Oct 10, 2012

:D
http://www.youtube.com/watch?v=8tw9w5W7sas
:D :D :D :D :D :D :D :D :D

Thanks! Keep up the good work on your end!

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