[Screenshot] When using "Capture an area of the screen" on a monitor not on topleft position (not 0,0 coordinates) you can't select area. about lximage-qt

[tsujan]

lximage-qt's screenshot code won't be developed and might be removed. See lximage-qt --help.

If you see the same problem in screengrab, please open an issue for it.

[elviosak]

If you see the same problem in screengrab, please open an issue for it.

Didn't know about Screengrab, it works fine there.

lximage-qt's screenshot code won't be developed and might be removed. See lximage-qt --help.

So no PR for it will be accepted? I have a quickfix for it, change:

lximage-qt/src/screenshotselectarea.cpp

Line 38 in 12d299b

setWindowState(windowState() | Qt::WindowFullScreen);

to:

  setWindowFlags(Qt::FramelessWindowHint | Qt::WindowStaysOnTopHint | Qt::X11BypassWindowManagerHint);
  setGeometry(image.rect());

also, should data/lximage-qt-screenshot.desktop.in be removed if it's deprecated?

[tsujan]

So no PR for it will be accepted? I have a quickfix for it, change:

Some LXQt members think that the code should be removed; some others (me included) think that it can be left as it is but with a warning (as in --help). The first group is right in saying that screengrab is LXQt's screenshot utility and we shouldn't have parallel functionalities. The second group (I think Alf agreed with me) say that we shouldn't annoy users by removing what they've got used to.

Since you've found a simple fix for the problem, I see no reason why it shouldn't be merged.

also, should data/lximage-qt-screenshot.desktop.in be removed if it's deprecated?

Good idea! We could keep the code (for now) and remove lximage-qt-screenshot.desktop instead.

Would you please make 2 PRs, one for the fix and the other for removing the desktop entry? Sorry for the confusion!

[elviosak]

Would you please make 2 PRs, one for the fix and the other for removing the desktop entry? Sorry for the confusion!

Ok, done.

[tsujan]

Thanks!