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

lower focus on lowered windows #776

Closed
wants to merge 1 commit into from

Conversation

irush-cs
Copy link

After pressing alt+f3 to fully lower a window, the focus window list still keeps that window second, so alt+tab returns to the lowered window.
This patch moved the lowered window to the bottom of the focus list (which is the behavior in my older icewm 1.3....).

@gijsbers
Copy link
Collaborator

This doesn't take into account transients. It would be better to use doLower instead, but there is another problem. wmLower is called from several places. Only when it is called from an action (actionLower) should it call lowerFocusFrame. I'll take this PR as a hint. Thanks.

@gijsbers gijsbers closed this May 19, 2024
@gijsbers gijsbers self-assigned this May 19, 2024
@gijsbers
Copy link
Collaborator

See commit f343387.

Could you perhaps evaluate the translation of icewm into Hebrew?
I'm interested also in how well icewm supports right-to-left languages.

@irush-cs
Copy link
Author

Thanks, the patch in the commit works for me :)

Regarding right-to-left, I usually use English as the main language so I don't encounter everything.
But giving it a quick look, some layouts are still reversed. E.g. in the main menu - the icons should be on the right, left to them should be the text (aligned to the right), and the triangle for the submenu should be on the left.
On the window menus similar issue - the text should be on the right and the keyboard shortcuts on the left, and if there's a submenu - it should also open to the left.
Some translations are also missing or inaccurate, but I'm not sure what they should be... (again, usually not using Hebrew interface)

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

2 participants