From kde-core-devel Thu Apr 23 22:52:17 2009 From: Ted Gould Date: Thu, 23 Apr 2009 22:52:17 +0000 To: kde-core-devel Subject: Re: KNotificationAreaItem Message-Id: <1240527137.11524.62.camel () shi> X-MARC-Message: https://marc.info/?l=kde-core-devel&m=124057146923755 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--=-0IYGaL1vDM+jFvutFnLa" --=-0IYGaL1vDM+jFvutFnLa Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Mon, 2009-04-20 at 12:11 -0600, Aaron J. Seigo wrote:=20 > questions, comments are desired. Aaron, This is cool, I like the update. I have a couple of concerns though. * There are function calls for both popup menu and mouse click. And I guess I don't understand the difference. Would a someone get both the mouse click event and a popup menu event if someone clicked on the icon? * With the mouse click event I'm guessing that the x,y are the x,y inside the icon? Or on the screen? It seems like you'd need the screen coordinates if you wanted to put a menu there, but if you had a complex icon you'd want to know where in the icon was clicked. * If it's the coords in the icon how do you know what size the icon is being rendered at? (especially with named icons) * If it's the screen coords how do you know where your icon is? Do we need to know a screen as well? * It seems like that by having the application draw the menu you'll end up with GTK menus in the KDE panel and vice versa. And despite all of the theming work this will always look a little off. I'd rather see a simple menu protocol so that the menu could be rendered "natively." What are your thoughts on that? Looks like a good upgrade to the notification spec. --Ted --=-0IYGaL1vDM+jFvutFnLa Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iEYEABECAAYFAknw8SEACgkQLE335pRPGp1gEwCg6xgWnQGvt+7wEysxfQNTeooH x6gAoLPFKhys86Sdi+A3Ov5CU3m9SI12 =9CGc -----END PGP SIGNATURE----- --=-0IYGaL1vDM+jFvutFnLa--