This is my first time to make contribution to a open source project.
I use a lot of note apps in my andorid phone, Joplin really need a good toolbar to extend the markdown editing ability, so I start to develope the toolbar.
I never notice the rule about making a pull request , until I almost developed this toolbar.
After I search in the forum, I notice this toolbar is already mentioned here Mobile Editor
Thanks for sharing, that’s quite a big undertaking for a first task but it seems to work rather well already. Last time I tried that the TextInput selection API had bugs that prevented it to work properly but I assume they’ve improved it since then.
I’ve looked at your branch and that looks pretty good overall. The code is quite readable and well organised. There are typos that would need to be fixed and I think some functions, in particular the render one, should be split into smaller functions, but otherwise it’s a very good start.
For your first PR, I would suggest your narrow the scope down a bit though - for example by creating a toolbar that supports only Bold, Italic, Code and Lists. That would be much easier to review and to get to a point where the feature can be merged.
Once your code is merged, you’re of course free to open more PR to add more buttons to the toolbar. But in general, for PR, the less code there is the better.
Also please check the CONTRIBUTING.md document. If you can do the toolbar with React Hooks and indeed TypeScript that would be perfect.
I’m sorry for how late that I reply the message.
I got tons of works to do recently so that I leave all the things at there (:з」∠).
I will continue to maintain the markdown Toolbar.
).