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

miscellaneous improvements #118

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

zamfofex
Copy link

@zamfofex zamfofex commented Sep 6, 2024

Thank you for making such a nice text editor for browsers! I had deleted my GitHub account, but I created another one just to contribute to this. (So I hope it is appreciated!) You might remember me from #49

A small (but very relevant) note is that I have only tested this in Firefox, as I do not have easy access to other browsers currently. If this turns out to work poorly in Chrome or other browsers, I am willing to invest the time into resolving that!

@zamfofex
Copy link
Author

@antonmedv: I have just rebased my changes to resolve the conflict after ab52adb. Friendly bump: Is there anything I can do to have this reviewed and merged?

I separated all of my changes into different commits so that it is easier to review them individually. But if you’d like, I could also submit smaller PRs individually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant