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

Add feature to insert emoji into pages directly #10

Merged
merged 9 commits into from
Apr 21, 2019
Merged

Conversation

rugk
Copy link
Owner

@rugk rugk commented Apr 11, 2019

  • insertion basically works
  • config GUI
  • Autoinsert Bug in GitHub Gist (new line or what?)

Fixes #10

@rugk rugk added this to the 1.0 milestone Apr 11, 2019
Still needs translation.
And has an awkward bug where it reloads the option page…
Otherwise they are submit buttons and submit the form/reload the site.
The fallback option requires the clipboardWrite though.
This required much code to handle different cases, when the permission is there already or not, etc.

Also some AddonSettings adjustments, so it all works. (settings are now frozen e.g.)
@rugk rugk marked this pull request as ready for review April 21, 2019 17:27
@rugk
Copy link
Owner Author

rugk commented Apr 21, 2019

The thing with GitHub gists seems to be special to GitHub gists. Needs more testing and can be fixed later…

So issue for this: #20

@rugk rugk merged commit 49e8a19 into master Apr 21, 2019
@rugk rugk deleted the emojidirectinsert branch April 21, 2019 17:38
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.

None yet

1 participant