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 text selection action that opens results in a popup #11

Open
gtbkul opened this issue Jul 17, 2024 · 3 comments
Open

Add text selection action that opens results in a popup #11

gtbkul opened this issue Jul 17, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@gtbkul
Copy link

gtbkul commented Jul 17, 2024

Hi!
First I'd like to thank you so much for making this app! It has been very helpful! Sadly I have no programming skills and would like to request the following feature to make this app better:

I think it will be very useful to implement the functionality as seen for the Translate You app in the video attachment. It pops up in the text selection pop-up and allows us to quickly see a translation in a pop-up window, without disrupting the workflow. Here is the github for the Translate You app that can be seen in the video attachment demonstrating the feature: https://github.com/you-apps/TranslateYou

2nd_feature_request.mp4

Thank you!!

@petlyh petlyh added the enhancement New feature or request label Jul 19, 2024
@petlyh
Copy link
Owner

petlyh commented Jul 19, 2024

Hi. Thanks for your feature request.

I'll probably implement request 1 and 3. However, with regards to your second feature request, the ACTION_TRANSLATE intent is made for translator apps and is for various reasons not really suitable for a dictionary app. I will however add support for PROCESS_TEXT intents, which will make JS-Dict show up in the text selection popup and search for the selected text in JS-Dict when tapped. That will at least save 1 or 2 taps over sharing the text to JS-Dict.

By the way, I'm going to split up your issue. In the future, please open a separate issue for each feature request.

@petlyh petlyh self-assigned this Jul 19, 2024
@gtbkul
Copy link
Author

gtbkul commented Jul 20, 2024

Hi!

Thanks for your reply and being able to implement feature 1 and 3.

Regarding the second feature, I think there is some misunderstanding about my request. I did not mean to request a ACTION_TRANSLATE or any translation service. What I intended by that example is that there is an app (Translate You) that not only has PROCESS_TEXT intents, but also that instead of opening the Translate You app it just opens a pop up as you can see in the video in my last post corresponding to feature 2. I wished to request that when JS-dict is tapped in the text selection popup, it can be handled in a way that it opens a similar pop up with condensed UI instead of opening the full app. This would be very seamless as we will not need to leave our Japanese article/page to go to the JS-dict all and would instead be able to access Jisho through that pop-up. I tried to make a video demonstration for what I meant, please find it as follows:

re_2nd_feature_concept.mp4

(Please excuse my bad editing)
(I hope you are able to copy this feature from the Translate You app as it is open source: https://github.com/you-apps/TranslateYou)

Please let me know if there is any confusion about the feature request description, or if I am misunderstanding something.

Thanks!!

PS. I'm sorry about putting all requests in one issue, I'm new to github and not familiar with proper etiquette here, I'll keep it on mind for future!

@petlyh
Copy link
Owner

petlyh commented Sep 20, 2024

Hi. Sorry for the very late reply, I've been away for some time.

You're right, I seem to have misunderstood your request. It looks like Translate You uses a PROCESS_TEXT intent that launches a popup activity instead of the main app activity. It might be difficult to implement a standalone popup like that since this app uses Flutter, though I'll have to look into it.

There's also some other details to figure out, like how much information about each result should be shown in the popup, and whether more info about a result should be shown in the popup or in the main app when the result is pressed, etc. At the very least, I think the UI would need to be simplified a bit compared to the main app in order for it to make sense in a popup, though input is welcome on this.

@petlyh petlyh changed the title Text selection pop-up action button, reaults cache, and furigana for kanji Add text selection action that opens results in a popup Sep 20, 2024
@petlyh petlyh removed their assignment Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants