Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Keyboard, mouse, and other inputs #46

Closed
8 of 9 tasks
themoeway-bot opened this issue Mar 5, 2023 · 6 comments
Closed
8 of 9 tasks

Keyboard, mouse, and other inputs #46

themoeway-bot opened this issue Mar 5, 2023 · 6 comments
Labels
kind/discussion The issue is a discussion

Comments

@themoeway-bot
Copy link
Collaborator

toasted-nutbread opened issue FooSoft/yomichan#267 on 2019-10-26


This issue is intended to be an overview of issues features related to inputs. There are some changes I want to make to text-scanning inputs, and I'm going to list all other input-related issues I can find.

Add support for pen devices using pointer events

Customizable scanning modifier keys

Customizable webpage hotkeys

UI changes

@themoeway-bot
Copy link
Collaborator Author

siikamiika commented on 2019-12-01


Just adding a mention of gestures here as well in addition to https://www.github.com/FooSoft/yomichan/issues/292#issuecomment-560111090

@themoeway-bot
Copy link
Collaborator Author

siikamiika commented on 2019-12-01


Another issue: the search page query parser and glossary clicking don't currently support touch events. I could copy and paste code from frontend.js, but that would mean a lot of duplication. Instead, it could be useful to refactor scanning to a new class and use it from frontend.js and other places as needed. Also mentioned in https://www.github.com/FooSoft/yomichan/pull/279#issue-339144238, but Frontend might be too tied to popups for subclassing.

@themoeway-bot
Copy link
Collaborator Author

siikamiika commented on 2019-12-04


@toasted-nutbread Are you working on something that would conflict if I extract the scanning functionality from frontend.js or general objections?

@themoeway-bot
Copy link
Collaborator Author

toasted-nutbread commented on 2019-12-04


The only thing I've somewhat started on is pointer events, so I don't have anything that should conflict.

@themoeway-bot
Copy link
Collaborator Author

siikamiika commented on 2019-12-07


Swipe gestures could work only on some parts of the popup or search page, similar to what's done in https://www.github.com/FooSoft/yomichan/commit/e99c8c40a1b87e307d5365fe619fa1a71996b500 and how Chrome for Android switches between tabs.

@themoeway-bot
Copy link
Collaborator Author

siikamiika commented on 2019-12-10


Regarding #300 and #161, I think we could reuse the scanning modifiers but for now give the caret position adjustment responsibility to the browser or another extension. The copy event (https://www.github.com/FooSoft/yomichan/issues/300#issuecomment-563918308) shouldn't focus the popup immediately, but there could be another hotkey for that. Automatically selecting the scanned text might also interfere with the feature.

@djahandarie djahandarie added the kind/discussion The issue is a discussion label Mar 5, 2023
@Kuuuube Kuuuube converted this issue into discussion #869 Apr 29, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
kind/discussion The issue is a discussion
Projects
None yet
Development

No branches or pull requests

2 participants