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 option to relocate translate window to bottom (top?). #57

Open
yupm opened this issue May 30, 2023 · 2 comments · May be fixed by #62
Open

Add option to relocate translate window to bottom (top?). #57

yupm opened this issue May 30, 2023 · 2 comments · May be fixed by #62
Labels
enhancement Update to existing features/codebase

Comments

@yupm
Copy link
Contributor

yupm commented May 30, 2023

For monitors in portrait mode, the image is squeezed between two sidebars.
I think being able to change the placement of the translate window would be good for user experience.
Not sure how hard it is but I am willing to give a try.

Screenshot_20230531_001626

@blueaxis
Copy link
Owner

blueaxis commented May 31, 2023

I am willing to give a try

Feel free to do so. I think its would be better if there is an option to place it on top/bottom/right.

Also, please see this guide for commit messages, so I don't end up squashing the merge like last time. I don't strictly follow the reference sheet, but I'd enforce one rule: use imperative, present tense verb as the first word of the message. Thanks!

@blueaxis blueaxis added the enhancement Update to existing features/codebase label Jun 3, 2023
@yupm
Copy link
Contributor Author

yupm commented Jun 17, 2023

I am sorry I wanted to do it this week, but fell sick from food poisoning. Probably from the onsen egg combini bento.
JLPT exams on July 2nd so I will be free to focus on this only from July 3rd..

@yupm yupm linked a pull request Jul 23, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Update to existing features/codebase
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants