Your message dated Wed, 19 May 2010 11:47:09 +0000
with message-id <[email protected]>
and subject line Bug#550466: fixed in fluxbox 1.1.1-8
has caused the Debian Bug report #550466,
regarding allow centering fbrun popup [+patch]
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
550466: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=550466
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: fluxbox
Version: 1.1.1-2
The attached patch adds a "-centered" option which centers the fbrun popup on
the screen. The reason for this is that fluxbox will otherwise try to position
this very small window somewhere that is not occupied by other windows, which
requires you to then search for the window in unoccupied areas of the screen.
It's just a minor inconvenience.
Just for the record: The fbrun manpage then also needs adjustment.
Thanks!
Uli
--- fluxbox-1.1.1/util/fbrun/main.cc 2008-01-15 10:50:53.000000000 +0100
+++ main.cc 2009-10-10 11:40:44.000000000 +0200
@@ -60,6 +60,7 @@ void showUsage(const char *progname) {
" -display [display string] Display name"<<endl<<
" -pos [x] [y] Window position in pixels"<<endl<<
" -nearmouse Window position near mouse"<<endl<<
+ " -centered Window position centered on screen"<<endl<<
" -fg [color name] Foreground text color"<<endl<<
" -bg [color name] Background color"<<endl<<
" -na Disable antialias"<<endl<<
@@ -74,6 +75,7 @@ int main(int argc, char **argv) {
bool set_height = false, set_width=false; // use height/width of font by default
bool set_pos = false; // set position
bool near_mouse = false; // popup near mouse
+ bool centered = false; // popup centered on screen
bool antialias = true; // antialias text
string fontname; // font name
string title("Run program"); // default title
@@ -105,6 +107,9 @@ int main(int argc, char **argv) {
} else if (strcmp(argv[i], "-nearmouse") == 0) {
set_pos = true;
near_mouse = true;
+ } else if (strcmp(argv[i], "-centered") == 0) {
+ set_pos = true;
+ centered = true;
} else if (strcmp(argv[i], "-fg") == 0 && i+1 < argc) {
foreground = argv[++i];
} else if (strcmp(argv[i], "-bg") == 0 && i+1 < argc) {
@@ -213,6 +218,20 @@ int main(int argc, char **argv) {
}
}
+ if (centered) {
+ Display* dpy = FbTk::App::instance()->display();
+ unsigned int root_w = WidthOfScreen(DefaultScreenOfDisplay(dpy));
+ unsigned int root_h = HeightOfScreen(DefaultScreenOfDisplay(dpy));
+ x = (root_w-fbrun.width())/2;
+ y = (root_h-fbrun.height())/2;
+ /* TODO: this will center the popup on the root window, but in
+ the case of Xinerama it should rather center it on one screen.
+ Question remains on which of them ... maybe use the one the
+ pointer is in, like the code for 'nearmouse' does? Alternatively,
+ allow defining a 'main' screen, that should then be applied to
+ the 'nearmouse' handling, too. */
+ }
+
if (set_pos)
fbrun.move(x, y);
--- End Message ---
--- Begin Message ---
Source: fluxbox
Source-Version: 1.1.1-8
We believe that the bug you reported is fixed in the latest version of
fluxbox, which is due to be installed in the Debian FTP archive:
fluxbox_1.1.1-8.diff.gz
to main/f/fluxbox/fluxbox_1.1.1-8.diff.gz
fluxbox_1.1.1-8.dsc
to main/f/fluxbox/fluxbox_1.1.1-8.dsc
fluxbox_1.1.1-8_amd64.deb
to main/f/fluxbox/fluxbox_1.1.1-8_amd64.deb
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Denis Briand <[email protected]> (supplier of updated fluxbox package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: RIPEMD160
Format: 1.8
Date: Wed, 19 May 2010 12:42:11 +0200
Source: fluxbox
Binary: fluxbox
Architecture: source amd64
Version: 1.1.1-8
Distribution: unstable
Urgency: low
Maintainer: Dmitry E. Oboukhov <[email protected]>
Changed-By: Denis Briand <[email protected]>
Description:
fluxbox - Highly configurable and low resource X11 Window manager
Closes: 550466 554386
Changes:
fluxbox (1.1.1-8) unstable; urgency=low
.
[Paul Tagliamonte]
* Bumped the deb stds version.
.
[Denis Briand]
* Upgrade libmagickcore-extra to libmagickcore3-extra build-dep.
* Add source/format file.
* Fix FTBFS with binutils-gold (Closes: #554386).
* Fix hyphen-used-as-minus-sign in fbrun and fbsetbg man pages.
* Allow centering fbrun popup (Closes: #550466).
Thanks to Ulrich Eckhardt for his patch.
* Debian Maintainer Upload is now allowed.
Checksums-Sha1:
bb5055f34b9aa1723f63b7da3d4aefd008d5e038 1491 fluxbox_1.1.1-8.dsc
c21be657344c15d4a67e2a1e30b6ecf6ab694094 32631 fluxbox_1.1.1-8.diff.gz
9597d75d4da5028c9efd8c36215a535faae941f8 1504886 fluxbox_1.1.1-8_amd64.deb
Checksums-Sha256:
c9777b318a1df562cc0837b5a7d05776f68be0e7803189e2433cdf1d9eb40e61 1491
fluxbox_1.1.1-8.dsc
a65adc252013d52ae638c15d54a72392e75239ba219b64f5f2b186b01c4b468c 32631
fluxbox_1.1.1-8.diff.gz
1a361061a7dec638df6fa9c0b12844201ba0268f17a7c80f47ec184b68b842a4 1504886
fluxbox_1.1.1-8_amd64.deb
Files:
d7e9d2ed96866d77dd092bd95beb9003 1491 x11 optional fluxbox_1.1.1-8.dsc
688cde806de0046cbbf11ee0df7f2043 32631 x11 optional fluxbox_1.1.1-8.diff.gz
fee1a9011d17e42d22a5c66d0edea332 1504886 x11 optional fluxbox_1.1.1-8_amd64.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
iEYEAREDAAYFAkvzzTUACgkQq4wAz/jiZTek5gCeM7+/TcgV7Udx/5hPdbunfeo2
htAAn3oS9fEhPyzH3YrmZGxjw9zTuEIi
=BsNU
-----END PGP SIGNATURE-----
--- End Message ---
