bkad / prat

group chat with markdown served over websockets
11 stars 6 forks source link

cursor: pointer for the buttons #98

Closed marcia closed 10 years ago

marcia commented 10 years ago

Test plan: To be honest, crossed fingers. I haven't set up my dev environment, and I've only ever used CSS + LESS, but I'm guessing from looking at this file that this is how I would do this.

cespare commented 10 years ago

The honesty makes me happy.

cespare commented 10 years ago

@marcia This change works fine.

However, there are several random places in the code where we set cursor: pointer for other buttons. Mind going through and getting rid of them, as they'll be pointless (heh-heh) now? (After checking that they're all actually for buttons -- I didn't look super closely.)

marcia commented 10 years ago

I only did a cursory (hah) search, but it looks like the other appearances of cursor: pointer aren't applied to buttons. It's possible that I could make those other pointer-like things (eg. the div.plus-label) into buttons, but before I do that I'd want to set up my environment. :)

bkad commented 10 years ago

thanks!

cespare commented 10 years ago

@marcia Oh, my mistake!