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

Pointer hover changes behavior/bounces in relation app (extremely low priority) #181

Closed
JTFouquier opened this issue Jun 8, 2016 · 2 comments
Assignees

Comments

@JTFouquier
Copy link
Collaborator

This is extremely minor although if a quick fix would change it that'd be nice but I imagine it's a bit more tricky and has to do with YPET.

Essentially when you hover over the title and abstract, the "disable" pointer for the mouse changes back and forth which is a little distracting and annoying.

2016-06-08 11 04 59
2016-06-08 11 04 17
2016-06-08 11 03 56

x0xMaximus added a commit that referenced this issue Jul 6, 2016
@x0xMaximus
Copy link
Member

While is probably the desired behavior, it's important to note that now when reviewing completed ER documents, on the Talk Page for example, or active Relation tasks, the user is compeletly unable from being able to copy text b/c they're no longer able to make a selection. The ability to copy text from these abstracts, for relation tasks or reviewing ER work,might be important to our users. @gtsueng thoughts on this?

@gtsueng
Copy link
Collaborator

gtsueng commented Jul 11, 2016

I think it's fine to have selection completely disabled within the box (which is what I assume your fix did) for the relationship app. I worry that facilitating their look up of things in the relationship app (w/ copy/paste features) will encourage them to draw the relationship inferences from other sources. If they really want to look something up, they can always type it in. So far, we haven't heard from users who really desire this feature outside of the actual ER task.

@gtsueng gtsueng closed this as completed Jul 11, 2016
x0xMaximus added a commit that referenced this issue Jul 25, 2016
* migrations-prior-css: (21 commits)
  Addresses #184
  Addresses #189
  Addresses #180
  [Sentry] fixes #139428581 #139428586 #139610697
  [Redirect / URLs] not auth dashboard redirect
  [STATIC] new jquery 3.0.0 in repo
  [Small CSS] $color_error_red usage 4 back on Tree
  Addresses #102. Pull to prod.
  Addresses #102
  Addresses #28
  Quest loading css
  Addresses #11
  Addresses #135
  [Progress] Addresses #135
  Addresses #141
  More cleanup of verbose js imports from externals
  Closes #151
  Addresses #181
  Addresses #180
  Remove dangerous d3 import from group_home page.
  ...

# Conflicts:
#	static/css/style.css
#	static/scss/style.scss
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants