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

Delete key not working on Mac #152

Open
mohanpb opened this issue Apr 26, 2019 · 13 comments
Open

Delete key not working on Mac #152

mohanpb opened this issue Apr 26, 2019 · 13 comments
Labels
bug first-timers-only Suitable for fevelopers who are contributing first time in open source development . good first issue help wanted

Comments

@mohanpb
Copy link

mohanpb commented Apr 26, 2019

Describe the bug
I am not able to delete a wrong annotation in Mac using Chrome.

To Reproduce
Steps to reproduce the behavior:

  1. Go to imglab.ml/#. Open an image
  2. Click on Polygon. Try to make an annotation. And then try to delete it.
  3. It fails

Screenshots

Even after pressing delete key multiple times, it is not working.

Console Errors

None

Device detail
(please complete the following information):

  • OS: [e.g. Ubuntu] Mac
  • Browser [e.g. chrome, firefox] Chrome

Additional context

Watch for changes, or Bookmark for easy discovery.
Fund this project for new features and maintenance.
[Showcase] your project with us by raising an issue


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@amitguptagwl
Copy link
Member

Thanks for reporting the issue. Unfortunately, I don't have mac to test it so this fix may take time. However, if you're a developer I'll appreciate if you can debug and fix the issue.

In addition of this, can you please share the console logs? It may help us to fix the issue without having mac.

@amitguptagwl amitguptagwl added bug first-timers-only Suitable for fevelopers who are contributing first time in open source development . good first issue help wanted labels Apr 28, 2019
@gitclem
Copy link

gitclem commented May 7, 2019

There are two issues.

  1. The Mac does NOT have an alt-key. (It has an "option" key in which some applications sometimes treat as an alt key. However, IMHO, it's better to allow the user to generate non-ascii characters.) . This means that there is no Alt-A to do the select-all command on the Mac. (I didn't bother fixing the other Alt-key short cuts.)

  2. The key that is labeled "delete" on the Mac generates a backspace.

These two fixes can be applied to the file workarea.tag.html which fixes the problem for me.

change line 31:
if ( (e.keyCode == 46) || (e.keyCode == 8) ) {// del key is 46, backspace key is 8 (for Mac)

change line 54 (adds Ctl-A as a synonym for Alt-A)
} else if ( (e.key === 'a' && e.altKey) || (e.key === 'a' && e.ctrlKey) ) {//TODO change this

@amitguptagwl
Copy link
Member

Ctrl key is used by browser for most of the operations and Alt key is used for some of the operations like opening the menu, access plugins etc. Hence, we'll have to write some modules which effectively manages keys across browsers and devices.

@gitclem
Copy link

gitclem commented May 24, 2019

I found this library, which seems like it will solve the various OS keyboard compatibility issues: https://wangchujiang.com/hotkeys/

@amitguptagwl
Copy link
Member

Thanks it'll be helpful.

leonardopalma added a commit to leonardopalma/imglab that referenced this issue May 30, 2019
@leonardopalma
Copy link

i have solved the issue tell me if is good solution for your program #152

@amitguptagwl
Copy link
Member

I'm thinking to use the 3rd party lib suggested by @gitclem . Otherwise we'll have to keep patching our code.

@hitesh-19
Copy link

I don't have knowledge about Mac, but if Del. key not working and there is no other alternatives so may be browser have something to del. that thing i am not sure, because i really don't have any knowledge about these type of bugs.

@Zeeshan75
Copy link

fn + delete worked for me

@crayne
Copy link

crayne commented Aug 1, 2020

f+delete is the normal way to do a forward delete on the Mac. The "delete" key does backward delete

@imrahul361
Copy link

Is this issue still open? If yes, Can you please brief me about it?

@gitclem
Copy link

gitclem commented Oct 3, 2020

Is this issue still open? If yes, Can you please brief me about it?

I don't know. I'm no longer employed at the job where I encountered this. So, I haven't used this code in months.

My understanding is the problem is the Mac control/option/command keys work differently from windows control keys (which may be yet different again from Linux) within JavaScript.

I think there needs to be a conditional to check which OS is running, then interpret the keys appropriately. I was a bit of a newbie with git at the time, but now I realize I should have made a Pull Request once I had figured out how to fix the problem...

@mmoollllee
Copy link

Still open. Keyboards without forward delete key need to do something like fn+delete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug first-timers-only Suitable for fevelopers who are contributing first time in open source development . good first issue help wanted
Projects
None yet
Development

No branches or pull requests

9 participants