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 771082 - Icon for go-down-symbolic has vanished
Icon for go-down-symbolic has vanished
Status: RESOLVED FIXED
Product: adwaita-icon-theme
Classification: Core
Component: symbolic
git master
Other Linux
: Normal normal
: ---
Assigned To: adwaita-icon-theme maintainer(s)
adwaita-icon-theme maintainer(s)
: 771409 771414 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2016-09-08 21:28 UTC by Marcus Lundblad
Modified: 2017-09-26 16:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot of gtk3-icon-browser (30.44 KB, image/png)
2016-09-08 21:28 UTC, Marcus Lundblad
  Details
screenshot (36.47 KB, image/png)
2016-09-08 22:05 UTC, Daniel Boles
  Details
symbolic: re-render go-down (11.18 KB, patch)
2016-09-09 08:43 UTC, Jakub Steiner
committed Details | Review
symbolic: go-down-symbolic (1.34 MB, patch)
2016-09-20 22:46 UTC, Jakub Steiner
none Details | Review
invisible icon (563.27 KB, image/png)
2016-09-20 22:56 UTC, Jakub Steiner
  Details

Description Marcus Lundblad 2016-09-08 21:28:05 UTC
Created attachment 335143 [details]
Screenshot of gtk3-icon-browser

It seems the icon "go-down-symbolic" has turned all blank.
Comment 1 Daniel Boles 2016-09-08 22:05:59 UTC
Created attachment 335144 [details]
screenshot

confirmed at all sizes, and i'm using 3.21.91 in debian unstable fwiw
Comment 2 Bjørn Lie 2016-09-09 08:35:35 UTC
Seeing the same in openSUSE with 3.21.91

So confirmed from here too.
Comment 3 Jakub Steiner 2016-09-09 08:43:28 UTC
Created attachment 335158 [details] [review]
symbolic: re-render go-down

- go-home and user-home had the bounding rectangle name go-down,
  thus the export script picked one of the blank rects to crop.
  somebody is being sloppy with the metadata! boo boo
Comment 4 Jakub Steiner 2016-09-09 08:43:56 UTC
Attachment 335158 [details] pushed as cf64e98 - symbolic: re-render go-down
Comment 5 gnome.vrb 2016-09-11 21:40:49 UTC
(In reply to Jakub Steiner from comment #3)
> Created attachment 335158 [details] [review] [review]
> symbolic: re-render go-down
> 
> - go-home and user-home had the bounding rectangle name go-down,
>   thus the export script picked one of the blank rects to crop.
>   somebody is being sloppy with the metadata! boo boo

Hmm. isn't there a way to automatically check if the generated icons are, say not blank ?
Comment 6 gnome.vrb 2016-09-11 22:18:47 UTC
(In reply to Jakub Steiner from comment #4)
> Attachment 335158 [details] pushed as cf64e98 - symbolic: re-render go-down

Still seeing the same issue.

Is src/symbolic/gnome-stencils.svg ( checkin ) relevant to this bug ?
Comment 7 gnome.vrb 2016-09-11 22:26:46 UTC
(In reply to vrishab from comment #6)

> Is src/symbolic/gnome-stencils.svg ( checkin ) relevant to this bug ?

I think "Adwaita/scalable/actions/go-down-symbolic.svg" is missing in this checkin.
Comment 8 Garrett Regier 2016-09-14 20:40:48 UTC
*** Bug 771409 has been marked as a duplicate of this bug. ***
Comment 9 Garrett Regier 2016-09-14 20:41:06 UTC
*** Bug 771414 has been marked as a duplicate of this bug. ***
Comment 10 Garrett Regier 2016-09-14 20:42:11 UTC
Built a fresh checkout and go-down-symbolic is still blank.
Comment 11 Jakub Steiner 2016-09-20 22:46:04 UTC
Created attachment 335951 [details] [review]
symbolic: go-down-symbolic

- user-home had a stray go-down label on a group that might cause
  render issues.
Comment 12 Jakub Steiner 2016-09-20 22:53:48 UTC
Not sure why, but gtk3-icon-browser still renders it blank, despite the exported icon rendering fine with eog.
Comment 13 Jakub Steiner 2016-09-20 22:56:55 UTC
Created attachment 335953 [details]
invisible icon
Comment 14 gnome.vrb 2016-09-21 03:26:06 UTC
(In reply to Jakub Steiner from comment #12)
> Not sure why, but gtk3-icon-browser still renders it blank, despite the
> exported icon rendering fine with eog.

gtk3-icon-browser displays png entries ( not svg ), I guess. So, try after exporting to png and installing the icons in their respective locations.
Comment 15 Jakub Steiner 2016-09-21 10:30:27 UTC
There were issues with stray rectangles named go-down-symbolic which made me think it was all that was to it. But when I build adwaita-icon-theme I see these things -- 

/opt/gnome/bin/gtk-encode-symbolic-svg /home/jimmac/checkout/gnome/adwaita-icon-theme/Adwaita/scalable/$file $size -o /opt/gnome/_jhbuild/root-adwaita-icon-theme/opt/gnome/share/icons/Adwaita/$size/$context; \
	done \
done
Can't load file: Error writing
Can't load file: Error writing
Can't load file: Error writing
Can't load file: Error writing
Can't load file: Error writing
Can't load file: Error writing

I don't fully grok the png fallback for the svgs, but I bet that's where the issue lies.
Comment 16 Jakub Steiner 2016-09-21 10:32:40 UTC
All generated go-down-symbolic.symbolic.png sizes render fine as a black arrow in eog btw.
Comment 17 gnome.vrb 2016-09-21 11:06:01 UTC
Latest git works fine. 

Relevant git commit: https://git.gnome.org/browse/adwaita-icon-theme/commit/?id=00e21c452e65a911b05195648b1212d12d76d576
Comment 18 gnome.vrb 2016-09-21 11:18:14 UTC
Hi Jakub, wanted to mention a few things.

1. From reading the bug report, I must say that I don't fully understand, what is this bug all about. Things like "stray rectangles named go-down-symbolic" sound totally clueless to me. Is there a way to make sure, this problem can be avoided in the future ?

2. Build ( make ) should have failed when "Comment #15" is encountered. How did the build get past that ?

3. Do you think, having a sanity check script, which checks the svg and generated png images for filesize > 0 ( or something like that ), would be a good idea ? Maybe add the check to the build ( make ) script, so it would make your job easier next time.
Comment 19 Lapo Calamandrei 2016-09-21 13:30:34 UTC
can't reproduce with 00e21c452e65a911b05195648b1212d12d76d576, the go-down-symbolic shows up as expected in the icon browser and go-down-symbolic.svg is correct here.