After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 571034 - Add transparency to panel run dialog
Add transparency to panel run dialog
Status: RESOLVED WONTFIX
Product: gnome-panel
Classification: Other
Component: run dialog
2.25.x
Other All
: Normal enhancement
: ---
Assigned To: Panel Maintainers
Panel Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-02-09 10:12 UTC by Andrea Gasparini
Modified: 2009-02-12 03:47 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
preview after the patch (114.20 KB, image/png)
2009-02-09 10:15 UTC, Andrea Gasparini
  Details
Patch (5.96 KB, patch)
2009-02-09 10:18 UTC, Andrea Gasparini
none Details | Review
medium alpha level (14.27 KB, image/png)
2009-02-11 14:18 UTC, Andrea Cimitan
  Details
This with a high alpha level, but a different text color (20.51 KB, image/png)
2009-02-11 14:20 UTC, Andrea Cimitan
  Details
Panel run dialog patched with alpha level setted at 0.70 (50.11 KB, image/png)
2009-02-11 17:27 UTC, Leo Iannacone
  Details
Panel run dialog patched with alpha level setted at 0.75 (55.72 KB, image/png)
2009-02-11 17:27 UTC, Leo Iannacone
  Details
Panel run dialog patched with alpha level setted at 0.80 (55.24 KB, image/png)
2009-02-11 17:28 UTC, Leo Iannacone
  Details

Description Andrea Gasparini 2009-02-09 10:12:48 UTC
Media keys use a different look in case xorg supports compositing, showing a nicer dialog, transparent and round bordered.

Perhaps panel-run-dialog could do the same.
Comment 1 Andrea Gasparini 2009-02-09 10:15:11 UTC
Created attachment 128282 [details]
preview after the patch

panel-run-dialog can look like this, in case of composite enabled Xserver
Comment 2 Andrea Gasparini 2009-02-09 10:18:21 UTC
Created attachment 128283 [details] [review]
Patch

This patch should add the behavior explained above, and looks like the image attached.
Comment 3 Andrea Cimitan 2009-02-09 14:15:39 UTC
This kind of things should not be done for usability reason.
Your preview could look good on the wallpaper, but have you thought about what will happen when the dialog is placed over firefox or a text? The text won't be readable, because you will see both the text of the applet and the black text of firefox/oopenoffice.
You can do those things only when the compositing manager adds some kind of blur effects to the alpha channel.

Closing
Comment 4 Andrea Cimitan 2009-02-10 19:09:22 UTC
I'm reopening the bug, I'm still on my own opinion (that patch introduces a lot of usability issues, it does not follow the HIG, and should NOT be part of GNOME), but the italian community is angry because I'm not the maintainer and they don't want me to close bugs... so that's it.
Vuntz? Do you really want that thing inside the panel? :)
Comment 5 Vincent Untz 2009-02-11 14:01:53 UTC
Let's see a screenshot with this window and firefox in background.

To be honest, that's not the first thing I'd do to fix the UI of this dialog. I'd remove the button "Add a file", hide the "Launch in terminal check box" (and make it work only with some magic keypress), kill the list of apps the way it is and instead do something more like deskbar/do/whatever where you start with nothing but results are displayed.
Comment 6 Andrea Cimitan 2009-02-11 14:18:38 UTC
Created attachment 128466 [details]
medium alpha level

It's the same issue of the terminal, this with a medium alpha level
Comment 7 Andrea Cimitan 2009-02-11 14:20:45 UTC
Created attachment 128467 [details]
This with a high alpha level, but a different text color

I can read this, but will my father or my grandfather read?
Comment 8 Leo Iannacone 2009-02-11 17:27:17 UTC
Created attachment 128480 [details]
Panel run dialog patched with alpha level setted at 0.70

Panel run dialog patched with alpha level setted at 0.70
Comment 9 Leo Iannacone 2009-02-11 17:27:53 UTC
Created attachment 128481 [details]
Panel run dialog patched with alpha level setted at 0.75

Panel run dialog patched with alpha level setted at 0.75
Comment 10 Leo Iannacone 2009-02-11 17:28:38 UTC
Created attachment 128482 [details]
Panel run dialog patched with alpha level setted at 0.80

Panel run dialog patched with alpha level setted at 0.80
Comment 11 Leo Iannacone 2009-02-11 17:46:33 UTC
The previous comments provide the patch displayed above some text lines.

Ok, it makes read labels more hard with 0.70 alpha. But I want to remember that this dialog runs only for a few seconds, in most of cases directly on the wallpaper.

I would make you understand that these shots are extremes cases of use it.

By the way, do you believe that labels are unuseful?
Comment 12 Andrea Cimitan 2009-02-11 17:59:04 UTC
(In reply to comment #11)
> The previous comments provide the patch displayed above some text lines.
> 
> Ok, it makes read labels more hard with 0.70 alpha. But I want to remember that
> this dialog runs only for a few seconds, in most of cases directly on the
> wallpaper.
> 
> I would make you understand that these shots are extremes cases of use it.
> 
> By the way, do you believe that labels are unuseful?
> 

I disagree on that... I don't know why you're saying that the dialog is usually launched without any application running or with every application minimized... that's a lie! :P

Well, it is the opposite. I think it is more common that at least one application is running, and firefox/openoffice/evolution/gedit/pidgin/xchat (with a lot of text) are maybe the most used.
Having no application running occures maybe the first time you login, but after that usually there are more than one application on the desktop.
Comment 13 Vincent Untz 2009-02-12 03:47:36 UTC
Okay. I'm not convinced with the screenshots.

If you guys want transparency, the first step is to fix the dialog :-) In its current version, it's just not appropriate for transparency.

See comment #5 for some ideas on how to fix the dialog.

Closing the bug for now. I'll be happy to review a new patch if we get a better dialog. Did I mention we should fix the dialog?