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

[BUG] Menu spawns at the height relative to the wrong monitor #1905

Closed
2 tasks done
Blackilykat opened this issue Oct 6, 2023 · 9 comments
Closed
2 tasks done

[BUG] Menu spawns at the height relative to the wrong monitor #1905

Blackilykat opened this issue Oct 6, 2023 · 9 comments
Labels

Comments

@Blackilykat
Copy link

Rofi version (rofi -v)

Version: 1.7.3

Configuration

https://gist.github.com/Blackilykat/2e1322e0334596bd165c81160fa7aad2

Theme

https://gist.github.com/Blackilykat/e1b0379de2a297d8833b00c9c50074cd

Timing report

https://gist.github.com/Blackilykat/c0d2864c2a9b0b209f65328b037a0299

Launch command

rofi -mod drun,run -show drun

Step to reproduce

  • Have multiple monitors with different resolutions, so that there is a blank space in the corner
  • Launch rofi in both monitors
  • The output will be offset in one of them

Expected behavior

The menu spawns in the middle of the used monitor
rofi-fixed

Actual behavior

The menu spawns in the middle of the used monitor, but with the height of the middle of the unused monitor
rofi

Additional information

No response

Using wayland display server protocol

  • No, I don't use the wayland display server protocol

I've checked if the issue exists in the latest stable release

  • Yes, I have checked the problem exists in the latest stable version
@Blackilykat Blackilykat added the bug label Oct 6, 2023
@DaveDavenport
Copy link
Collaborator

element-icon {
    size:           Tem ;
    vertical-align: 0.50;
}

This is not valid? in your theme.

@DaveDavenport
Copy link
Collaborator

DaveDavenport commented Oct 6, 2023

Can you please post the output of rofi -h
(it contains detected monitor layout)

I am not able to reproduce it with 4k+fhd monitor.

@Blackilykat
Copy link
Author

element-icon {
    size:           Tem ;
    vertical-align: 0.50;
}

This is not valid? in your theme.

I'm not sure, but when I don't add >> output.txt to the command it shows up weird. I'm using rounded-purple-dark from newmanls/rofi-themes-collection
error

here's the output of rofi -h
if it helps, I'm using i3wm

@DaveDavenport
Copy link
Collaborator

DaveDavenport commented Oct 6, 2023

thx for the paste.
Aah, that print is fixed in newer version.
I am also using i3, for me it works fine (atleast with latest version, have not tested the old 1.7.3 yet).

@DaveDavenport
Copy link
Collaborator

DaveDavenport commented Oct 6, 2023

Can you try adding this at the bottom, I think it might get the behavior you want:

window {
   anchor: center;
   location: center;
}

This makes sure that the center of rofi is centered on the center of the monitor, even if your window resizes (and you have dynamic size window).

@DaveDavenport
Copy link
Collaborator

@DaveDavenport
Copy link
Collaborator

This has been fixed and should work as expected without extra config entry in 1.7.5.

I am confused about this:

image

@Blackilykat
Copy link
Author

My bad - I saw the latest version was over a year old and assumed it would be the same as the latest on apt without checking version numbers. I reinstalled using the version on github and the issue is fixed, thanks

Copy link

github-actions bot commented Nov 7, 2023

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants