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

Tabbing is broken in the edit view #109

Closed
sindresorhus opened this issue Aug 5, 2019 · 6 comments · Fixed by #141
Closed

Tabbing is broken in the edit view #109

sindresorhus opened this issue Aug 5, 2019 · 6 comments · Fixed by #141
Labels
bug Something isn't working 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt help wanted Extra attention is needed

Comments

@sindresorhus
Copy link
Owner

sindresorhus commented Aug 5, 2019

Issuehunt badges

The Dimensions dropdown has the first focus, if you tab you get Width, then Height, then the trimmer view. Instead of the trimmer view, it should have been the "pixels/percentage" dropdown.

When the edit view is presented, if you press Shift+Tab, it moves to focus to the trimmer view, if you press Esc then, the trimmer is hidden. We should find a way to prevent the trimmer from disappearing.

I was also thinking, maybe we should make the trimmer view the initial first responder instead of the "Dimensions" dropdown. You would then be able to press Space to view the video, which is nicer, I think. We could make it so that when you tab once, the dimension dropdown is focused.


IssueHunt Summary

sergeykuryanov sergeykuryanov has been rewarded.

Backers (Total: $80.00)

Submitted pull Requests


Tips


IssueHunt has been backed by the following sponsors. Become a sponsor

@sindresorhus sindresorhus added bug Something isn't working help wanted Extra attention is needed labels Sep 5, 2019
@issuehunt-oss
Copy link

issuehunt-oss bot commented Sep 6, 2019

@issuehunt has funded $80.00 to this issue.


@issuehunt-oss issuehunt-oss bot added the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Sep 6, 2019
@NehaKushwah993
Copy link

Hi, I think I can fix this issue. But first I don't know how can I create pull request because I am not able to create any sub_branch or branch. Please let me know that process so I will start my process for other issues as well.

@NehaKushwah993
Copy link

Hi, I have fixed this issue :

  • Instead of the trimmer view, it should have been the "pixels/percentage" dropdown.
  • we should make the trimmer view the initial first responder instead of the "Dimensions" dropdown.
  • if you press Shift+Tab, it moves to focus to the trimmer view, if you press Esc then, the trimmer is hidden.

These 3 issues are fixed, please take a look in my pullRequest.

-Neha

@NehaKushwah993
Copy link

Hi, did you look in to this?

@NehaKushwah993
Copy link

Hey, should I send you it's video?

@issuehunt-oss
Copy link

issuehunt-oss bot commented Nov 6, 2019

@sindresorhus has rewarded $72.00 to @sergeykuryanov. See it on IssueHunt

  • 💰 Total deposit: $80.00
  • 🎉 Repository reward(0%): $0.00
  • 🔧 Service fee(10%): $8.00

@issuehunt-oss issuehunt-oss bot added 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt and removed 💵 Funded on Issuehunt This issue has been funded on Issuehunt labels Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants