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 781988 - Missing format symbols under "format cells"
Missing format symbols under "format cells"
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: GUI
1.12.x
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2017-04-30 17:38 UTC by chris dunn
Modified: 2017-07-04 13:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (914 bytes, patch)
2017-07-03 05:55 UTC, Jean Bréfort
committed Details | Review
screen shot (34.32 KB, image/png)
2017-07-04 06:42 UTC, Andreas J. Guelzow
  Details

Description chris dunn 2017-04-30 17:38:43 UTC
When I select "format cells" and go to "border", all of the expected
options are there, but on the right side of the panel under "Style:"
where the heavier lines and double line etc are displayed, I'm no
longer seeing any of the symbols, just a small terminal symbol in each
of the spaces, except "None" which displays correctly.
Comment 1 Morten Welinder 2017-05-01 13:55:11 UTC
Jean, I understood you see this too -- could you have a look, please?
Comment 2 Jean Bréfort 2017-05-01 14:56:01 UTC
Yes, I'm seeing it, but I don't understand what's going on.
Comment 3 Andreas J. Guelzow 2017-06-02 04:13:56 UTC
For me the lines are all there but on the left side there is a horizontal row of 5 buttons (assuming a cell selection with more than one row and column), the first one is no borders at all, the second one is outline on the very outside, etc.: The third and 5th symbol is missing but is just showing a terminal symbol with a red circle having a red diagonal line.
Comment 4 Technical Support 2017-06-27 18:45:39 UTC
If it helps, I see the same problem as reporter. The line style images are failing to load into the buttons, though clicking a line style button works as expected.

(Tested with Gnumeric 1.12.32 - the latest version in the Ubuntu repository)
Comment 5 Jean Bréfort 2017-07-02 14:44:23 UTC
Made some exploration. The culprit seems to be GdkPixbuf which is not anymore able to load pixdata (I don't know why). Removing the preprocess='to-pixdata' properties in gnm.gresource.xml seems to fix the issue.
Comment 6 Morten Welinder 2017-07-02 17:50:26 UTC
gtk+ is still using to-pixdata for whatever that is worth:

tests/resources.gresource.xml:    <file preprocess="to-pixdata" alias="icc-profile.pixdata">icc-profile.png</file>
tests/resources.gresource.xml:    <file preprocess="to-pixdata" compressed="true" alias="icc-profile-compressed.pixdata">icc-profile.png</file>

I don't find anything about it being deprecated.
Comment 7 Jean Bréfort 2017-07-02 18:00:12 UTC
It does not work the way we use it in the format dialog. I don't know how it works in other situations.
Comment 8 Jean Bréfort 2017-07-03 05:55:09 UTC
Created attachment 354813 [details] [review]
Proposed patch
Comment 9 Morten Welinder 2017-07-03 10:49:11 UTC
Do we understand that patch?  I.e., is there a known problem with gtk_image_new_from_resource?

Other than that, perhaps add a comment and commit.
Comment 10 Jean Bréfort 2017-07-03 11:15:46 UTC
There is another related bug: #781583. I added a comment there.
Comment 11 Jean Bréfort 2017-07-03 11:24:14 UTC
The same issue occurs at wbc-gtk.c:695 but a comment says that this code is currently unreachable. So may be we don't need to change it.
Comment 12 Jean Bréfort 2017-07-03 11:30:03 UTC
Review of attachment 354813 [details] [review]:

Patch committed with a comment.
Comment 13 Morten Welinder 2017-07-03 23:44:48 UTC
Patch not pushed yet.
Comment 14 Jean Bréfort 2017-07-04 05:26:03 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Once that release is available, you may want to check for a software upgrade provided by your Linux distribution.
Comment 15 Andreas J. Guelzow 2017-07-04 06:42:52 UTC
Created attachment 354866 [details]
screen shot

This did not do it. At least I am still missing symbols in that dialog: see attached screenshot.
Comment 16 Jean Bréfort 2017-07-04 12:45:00 UTC
Looks like hyphen and underscore are not anymore equivalent in ui files. Quite a different issue actually.
Comment 17 Morten Welinder 2017-07-04 13:00:10 UTC
We have multiple users of gtk_image_new_from_resource.

We might need to create a gnm_image_new_from_resource to hide the issue.


dialogs/dialog-printer-setup.c: state->icon_rd = gtk_image_new_from_resource ("/org/gnumeric/gnumeric/images/right-down.png");
dialogs/dialog-printer-setup.c: state->icon_dr = gtk_image_new_from_resource ("/org/gnumeric/gnumeric/images/down-right.png");
io-context-gtk.c:               gtk_box_pack_start (box, gtk_image_new_from_resource ("/org/gnumeric/gnumeric/images/gnumeric_splash_1.4.png"),
wbc-gtk.c:      image = gtk_image_new_from_resource ("/org/gnumeric/gnumeric/images/sheet_move_marker.xpm");
Comment 18 Jean Bréfort 2017-07-04 13:57:37 UTC
Only images stored as pixdata c an issue
Comment 19 Jean Bréfort 2017-07-04 13:59:55 UTC
Oops! I was writing that only images stored as pixdata ar an issue, and this excludes icons for which things do work. The only other potential issue concerns sheet_move_marker.xpm which is not used in any reachable code.