Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Action cursor #372

Merged
merged 1 commit into from
Mar 17, 2016
Merged

Action cursor #372

merged 1 commit into from
Mar 17, 2016

Conversation

pete-the-pete
Copy link
Contributor

This adds a selector to highlight actionable areas by showing the pointer cursor for elements with the data-ember-action attribute.

This should probably be temporary, to help show where more specific classes could be added.

Without the class
arrow

With the class
pointer
cc @HospitalRun/core-maintainers

@pete-the-pete
Copy link
Contributor Author

I can/will handle the conflicts and merge the branch if accepted.

@jglovier
Copy link
Member

@pete-the-pete thanks! 🎉

jglovier pushed a commit that referenced this pull request Mar 17, 2016
@jglovier jglovier merged commit 90ead43 into HospitalRun:master Mar 17, 2016
jglovier pushed a commit that referenced this pull request Mar 17, 2016
@jglovier jglovier mentioned this pull request Mar 17, 2016
@jglovier
Copy link
Member

I just realized using the attribute selector with data-ember-action results in lots of other elements besides the index rows matching the cursor declaration, including elements that aren't actually clickable. I've got a banch open for some cleanup, so I'll add a classname to the rows and target based on the class.

jglovier pushed a commit that referenced this pull request Mar 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants