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 699044 - Empty space between rounded window corner and selected window outline
Empty space between rounded window corner and selected window outline
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: overview
3.12.x
Other Linux
: Normal normal
: 3.16
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-04-27 15:37 UTC by Marius Gedminas
Modified: 2015-01-22 14:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot of the visual glitch (197.10 KB, image/png)
2013-04-27 15:37 UTC, Marius Gedminas
  Details
Add an inset shadow to overview windows (934 bytes, patch)
2014-02-13 17:03 UTC, Chris Johns (ter0)
committed Details | Review

Description Marius Gedminas 2013-04-27 15:37:44 UTC
Created attachment 242663 [details]
screenshot of the visual glitch

See attached screenshot.

This only shows up for windows of certain sizes.
Comment 1 Chris Johns (ter0) 2014-02-13 17:03:24 UTC
Created attachment 269042 [details] [review]
Add an inset shadow to overview windows

A small dark gap was caused by the rounded window edges not meeting the
sharp corner of the CSS border on hover. By adding a small inset
box-shadow, this gap is filled in.
Comment 2 Carlos Soriano 2014-08-30 18:47:16 UTC
Review of attachment 269042 [details] [review]:

Looks good given that we can't know the exact radius with the scale applied to window clones.
Do you have commits rights?
Comment 3 Carlos Soriano 2014-08-30 18:47:18 UTC
Review of attachment 269042 [details] [review]:

Looks good given that we can't know the exact radius with the scale applied to window clones.
Do you have commits rights?
Comment 4 Carlos Soriano 2015-01-21 16:28:18 UTC
Fixed on wip/sass
Comment 5 Florian Müllner 2015-01-21 17:13:29 UTC
It's also fixed on master, because the patch got committed a while ago ;-)
Comment 6 Carlos Soriano 2015-01-22 14:16:25 UTC
(In reply to comment #5)
> It's also fixed on master, because the patch got committed a while ago ;-)

whops, I messed up then.