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 670500 - Close button has an ugly square border under Ambiance/Radiance
Close button has an ugly square border under Ambiance/Radiance
Status: RESOLVED NOTGNOME
Product: sushi
Classification: Core
Component: general
0.2.x
Other Linux
: Normal normal
: ---
Assigned To: Sushi maintainer(s)
Sushi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-02-21 05:01 UTC by Vadim Rutkovsky
Modified: 2012-02-29 12:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (644 bytes, patch)
2012-02-21 08:43 UTC, Vadim Rutkovsky
rejected Details | Review

Description Vadim Rutkovsky 2012-02-21 05:01:24 UTC
Original Report: https://bugs.launchpad.net/ubuntu/+source/gnome-sushi/+bug/937141

DistroRelease: Ubuntu 12.04
Package: gnome-sushi 0.2.1-2

When Sushi is used under Ambiance or Radiance, a border appears around the titlebar close button that just ends up looking ugly. See screenshots for a comparison with how it is supposed to look under Adwaita.

Adwaita look: https://bugs.launchpad.net/ubuntu/+source/gnome-sushi/+bug/937141/+attachment/2762845/+files/Normal%20Sushi%20look.png
Radiance look: https://bugs.launchpad.net/ubuntu/+source/gnome-sushi/+bug/937141/+attachment/2762846/+files/Sushi%20under%20Radiance.png
Comment 1 Vadim Rutkovsky 2012-02-21 08:43:58 UTC
Created attachment 208095 [details] [review]
Proposed patch
Comment 2 Cosimo Cecchi 2012-02-27 17:05:08 UTC
Review of attachment 208095 [details] [review]:

I don't see why Unico should render a border there when we already specify

border-style: none;
border-image: none;
border-width: 0;
Comment 3 Cosimo Cecchi 2012-02-27 17:05:59 UTC
I think this is a bug in Unico really. It should respect the regular CSS properties and not try to render a border there.
Comment 4 Vadim Rutkovsky 2012-02-28 10:41:56 UTC
Thanks, Cosimo, found a Unico bug - https://bugs.launchpad.net/unico/+bug/920832
Comment 5 Andrea Cimitan 2012-02-28 12:09:30 UTC
Not a bug in Unico, but a bug in the theme (well, just a missing special-case) instead.