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

Fixes some UI bugs in Firefox #3044 #3058

Merged
merged 3 commits into from
Nov 19, 2017
Merged

Conversation

ffranchina
Copy link
Contributor

Here's my PR to solve the reported problems :)

I've cleaned a bit some CSS and I've refined the usability of the "Upload" button": now the clickable area is exactly equal to the button area.
I've tested the changes into Chromium, Firefox (Quantum, too) and Edge and it behaves as expected!

Let me know if you have questions or requests!

Thank you for your work!

@takluyver
Copy link
Member

Thanks - I've just tested this, and it looks good to me.

Oddly, when I move my mouse over it in Chromium, the pointer (hand) cursor appears momentarily, then it goes back to the default (arrow) cursor. In Firefox it uses the hand cursor like the other buttons. I wouldn't hold up the improvements over this, though.

@ffranchina
Copy link
Contributor Author

Yes, I've noticed it: it's due to the fact that Chrome/Chromium doesn't like to have the cursor changed on some kind of object..! Let me see if I can find a clean way to force it :)

@ffranchina
Copy link
Contributor Author

I had to add a browser-specific rule but I think it's worth it since it's very straightforward :)
Now it behaves uniformly.

@takluyver
Copy link
Member

Thanks, nice one :-)

@takluyver takluyver merged commit 6f23f45 into jupyter:master Nov 19, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants