Morse icon indicating copy to clipboard operation
Morse copied to clipboard

Support punctuations

Open KrasnayaPloshchad opened this issue 2 years ago • 9 comments

If this app could be able to convert a punctuation to morse code or vice versa, this app can present not only a simple phrase like SOS or Hello World but also a complete sentence.

KrasnayaPloshchad avatar Feb 14 '22 05:02 KrasnayaPloshchad

This would be a great improvement.
@FahadSaleem is working on refactoring the code and maybe he can directly implement support for punctuations.

CrazyMarvin avatar Feb 18 '22 23:02 CrazyMarvin

I suppose the punctuations on https://morsecode.world/international/morse.html would be a good start.

CrazyMarvin avatar Sep 27 '23 11:09 CrazyMarvin

Are you interested to help with this too, @fejd? 😅

You have done so much so it's fine if you say no.

CrazyMarvin avatar Oct 16 '23 20:10 CrazyMarvin

Sure, I could take a look. Is the code refactoring still in progress? I was thinking of breaking things out of the main activity and perhaps convert some of that functionality to Kotlin in the process. :)

If I understand the feature request correctly, there should be a mapping of punctuations from the "Punctuation Mark" table in https://morsecode.world/international/morse.html to dots/dahs.

I noticed on https://morsecode.world/international/timing.html that the timing is different depending on the type of space between characters and words. I should probably revisit the sound playback and adapt the pause between sounds based on that. Also, maybe it's useful to share the "playback algorithm" between audio, camera flash and "screen flash"?

fejd avatar Oct 17 '23 17:10 fejd

The refactoring did not start yet unfortunately. So feel welcome to start with it. 😅

Yes, you understood this correctly. 👍

There is another issue about the timing: https://github.com/Crazy-Marvin/Morse/issues/36. This is a bit more complex but it would be awesome if you could tackle it though.

CrazyMarvin avatar Oct 17 '23 18:10 CrazyMarvin

I only have: morse-sec001a4

Noitarud avatar Jan 14 '24 08:01 Noitarud

Do you have time for this or are you busy, @fejd?

CrazyMarvin avatar Jan 27 '24 21:01 CrazyMarvin

@CrazyMarvin Sorry, I'll be a bit busy for the next 3-4 months.

fejd avatar Jan 29 '24 08:01 fejd

@CrazyMarvin Sorry, I'll be a bit busy for the next 3-4 months.

Alright, thanks for the info. 👍

CrazyMarvin avatar Jan 29 '24 08:01 CrazyMarvin