Add backspace on key functionality for deleting of multiple cells

[guergana]

• fixes Cannot delete content of multiple cells #350

[cloudflare-workers-and-pages]

Deploying opendataeditor with    Cloudflare Pages

Latest commit:	379b3ba
Status:	✅  Deploy successful!
Preview URL:	https://93719a6b.opendataeditor.pages.dev
Branch Preview URL:	https://350-add-backspace-key-to-fil.opendataeditor.pages.dev

View logs

[romicolman]

@guergana I have just tested it and it's not working on Mac

[roll]

@guergana
https://github.com/okfn/opendataeditor/blob/main/client/components/Parts/Bars/Menu.tsx#L21

[guergana]

@guergana
https://github.com/okfn/opendataeditor/blob/main/client/components/Parts/Bars/Menu.tsx#L21

Hi, thanks @roll could you provide some context? Why would this be better than capturing the native event?

[roll]

@guergana
Because the library probably covers the differences in native keyboard codes, although not sure about it

Generally speaking, I think it's better to use the lib that abstracts it (if it's already in-use in the project)

[guergana]

@guergana Because the library probably covers the differences in native keyboard codes, although not sure about it

Generally speaking, I think it's better to use the lib that abstracts it (if it's already in-use in the project)

@roll According to the mozilla Javascript docs, it should work as it is now, using the event.key for both Mac and Windows.
We should test this together on your Mac @romicolman