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

feh -x --borderless not removing Window border? #137

Closed
ghost opened this issue Jul 26, 2013 · 5 comments
Closed

feh -x --borderless not removing Window border? #137

ghost opened this issue Jul 26, 2013 · 5 comments

Comments

@ghost
Copy link

ghost commented Jul 26, 2013

Hi,

-x & --borderless seem pretty self explanatory, run feh -x or --borderless /path/to/wallpaper and feh opens up without any window border, correct?

If so this is not working 2.9.3...

I still get a Window/Title bar saying;
feh [1of41] - /home/foo/Wallpaper/A Bug.jpg

I just want to see a borderless window, I'm assuming I'm suppose to be able to do this with those commands?

I only run Openbox 3.5.0 as my DE, seems as though the commands don't over ride the Openbox themes to remove the border...

Thank you

rephorm pushed a commit to rephorm/feh that referenced this issue Jul 28, 2013
Compiling feh with -O2 appears to cause the mwmhints struct to get
padded. When casting it to (unsigend char *) and passing that to
XChangeProp, the values were getting corrupted.

For example,
$ feh -x foo.png
$ xprop | grep MOTIF
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x0, 0xeb277ce8, 0x838fd4c8, 0x1
                                             ^^^^^^^^^^
                                             should be 0

This patch copies the values from the struct into an unsigned char * buffer explicitly.
@jim-rees
Copy link

The problem is that mwmhints is not initialized, and contains garbage after the flags field. I have a patch but it's not obvious to me how to attach it to this comment. You can pick it up at http://jim.rees.org/feh.diff .

rephorm pushed a commit to rephorm/feh that referenced this issue Jul 28, 2013
This reverts commit c7acdb9.
@ghost
Copy link
Author

ghost commented Jul 30, 2013

I mentioned this issue on the Openbox mailing list and Brian submitted the feh-mwmhints3.patch

Now I see Jim-rees has submitted a different patch, feh.diff, so which one should we use?

Thanks

@rephorm
Copy link

rephorm commented Jul 30, 2013

Use feh-mwmhints3.patch. It is the same as pull request #139, which includes Jim's patch.

@jim-rees
Copy link

Brian Mattern wrote:

Use feh-mwmhints3.patch. It is the same as pull request #139, which
includes Jim's patch.

Agreed. Thanks.

I'm left wondering why struct mwmhints isn't defined in some system include
file. Or maybe it is, but that would require pulling in all of Motif, which
would be a bad idea.

@derf
Copy link
Owner

derf commented Dec 4, 2013

Confirmed and merged, thanks!

@derf derf closed this as completed Dec 4, 2013
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

No branches or pull requests

3 participants