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

Focus screen: augment the Direction command #1117

Open
ThomasAdam opened this issue Nov 22, 2024 · 2 comments
Open

Focus screen: augment the Direction command #1117

ThomasAdam opened this issue Nov 22, 2024 · 2 comments
Labels
difficulty:gfi Issue is suitable for a first-time contributor/developer type:enhancement Augmenting an existing feature

Comments

@ThomasAdam
Copy link
Member

ThomasAdam commented Nov 22, 2024

It would be nice if we were able to say:

Direction Screen (North) CursorMove 50 50

Which would (in this example), move the cursor to the middle of the screen.

Likewise:

Direction Screen (East) CursorMove 50 50

Would move the cursor to the monitor to the right of wherever the focus window was.

This would also mean you could make use of the FromPointer option to Direction, as in:

Direction FromPointer Screen (SouthEast) CursorMove 50 50

To do this, we can make use of how fvwm3 already stores the configuration of monitors in monitor_q and map those back to the direction coordinates.

Hence, this would become a "focus screen" command.

@ThomasAdam ThomasAdam added type:enhancement Augmenting an existing feature difficulty:gfi Issue is suitable for a first-time contributor/developer labels Nov 22, 2024
@ThomasAdam ThomasAdam added this to the 1.1.1 milestone Nov 22, 2024
@ThomasAdam ThomasAdam added this to FVWM3 Nov 22, 2024
@github-project-automation github-project-automation bot moved this to To do in FVWM3 Nov 22, 2024
@somiaj
Copy link
Collaborator

somiaj commented Nov 22, 2024

With your examples, to me it seems a bit odd to want to add this to the Direction conditional, since there aren't many conditions. Would it be better just to update the CursorMove to have a direction argument instead?

@ThomasAdam
Copy link
Member Author

Well, the whole point of the Direction command was for just this sort of scenario.

@ThomasAdam ThomasAdam removed this from the 1.1.1 milestone Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty:gfi Issue is suitable for a first-time contributor/developer type:enhancement Augmenting an existing feature
Projects
Status: To do
Development

No branches or pull requests

2 participants