TorstenDittmann / OmniaWrite

A text editor engineered for creative writing.
https://OmniaWrite.com
Apache License 2.0
162 stars 34 forks source link

Bugfixes #41

Closed ghost closed 4 years ago

ghost commented 4 years ago

Here is a set of small fixes and improvements I worked on today, for your consideration. These are small enough that I didn't feel compelled to create a GitHub issue.

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/torstendittmann/omniawrite/ntqre7pen ✅ Preview: https://omniawrite-git-fork-antonyboucher-bugfixes.torstendittmann.vercel.app

ghost commented 4 years ago

@TorstenDittmann Good point on scrollbars. I amended the PR with your correction.

What is the best way to test how changes affect mobile devices? Edit: found it -- "Enable device toolbar" in Chrome.

TorstenDittmann commented 4 years ago

@AntonyBoucher That is exactly what I wanted to suggest 😄

It is very accurate for Android used as a PWA. For the native mobile App that is using Capacitor it may be inaccurate when it comes native stuff like saving the export to the file system. Safari on iOS is a complete different story 😄

Thanks for the revision 👍