-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
cursor: col-resize for lt-column-resizer #480
Comments
In theory that should already be the case. 🤔 |
Ah I see. Looking at the Resizable Columns Table demo, the tag looks like this <div id="ember980" class="lt-column-resizer ember-view">
</div> This works as expected, the resizable cursor is used. Looking at my own webapp, it seems there is a <div style="touch-action: manipulation;-ms-touch-action: manipulation; cursor: pointer;" id="ember1757" class="lt-column-resizer ember-view">
</div> I'm not sure how or why those style attributes are there... is it anything to do with ELT? Or should I be looking elsewhere? |
Do you have ember-hammertime installed by any chance? I would assume that it accidentally adds these styles: Using Touch-action as a fastclick |
Looks like I do! It was installed as a dependency for ember-toggle. Now to figure out how to make it work nicely with ELT... :) |
Closing this because it's a pretty niche issue and isn't really to do with ELT. Thanks for the tip about ember-hammertime For anyone else reading this, there is more discussion here html-next/ember-hammertime#31 |
Currently it seems to be getting a
cursor: pointer;
applied to it, which isn't as UX friendly as a proper resize cursor is.I've applied my own style to
.lt-column-resizer
to add thecursor: col-resize
and it seems fine. I might get a chance to do a PR for it this week before I go on holiday, but if someone else beats me to it that would be great ;)The text was updated successfully, but these errors were encountered: