Bug 90790

Summary: ICONS: Parent icon theme links.txt duplicate records not used in child themes
Product: LibreOffice Reporter: Yousuf Philips (jay) (retired) <philipz85>
Component: LibreOfficeAssignee: Not Assigned <libreoffice-bugs>
Status: RESOLVED FIXED    
Severity: major CC: barta, franciscoadriansanchez, h3734236, kendy, tk
Priority: highest Keywords: regression
Version: 4.5.0.0.alpha0+ Master   
Hardware: Other   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=92105
Whiteboard: target:5.1.0 target:5.0.0.2
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 86696    
Attachments: screenshot

Description Yousuf Philips (jay) (retired) 2015-04-22 20:49:58 UTC
Created attachment 115015 [details]
screenshot

The galaxy links.txt file has the below entry

cmd/lc_formatselection.png sfx2/res/symphony/sidebar-property-large.png

which is to show the same icon that appears as the sidebar properties' sidebar button in the 'Format Selection' button but this does not work in child themes like tango and breeze and instead it shows the same galaxy icon in that position.

This is a regression that started in the 4.5/5.0 branch.

Version: 5.0.0.0.alpha1+
Build ID: badec7478035008f514e0976a94438fe2e32dc40
TinderBox: Linux-rpm_deb-x86@45-TDF, Branch:master, Time: 2015-04-22_00:50:58
Comment 1 tommy27 2015-05-24 11:32:04 UTC
works for me under Win8 using LO  5.1.0.0.alpha1+
Build ID: 83eb114394879cbfd073322a51c47d02553c1fcf
TinderBox: Win-x86@39, Branch:master, Time: 2015-05-22_06:33:51
Locale: en-US (it_IT)

the Breeze and Tango themes display a proper Legend On/Off icon rather than the icon from Galaxy theme like in your screenshot.

are you still seeing this issue?
Comment 2 Yousuf Philips (jay) (retired) 2015-05-26 22:13:20 UTC
The issue is still present in master updated today. The 'Format Selection' button should be the same 'Properties' button found in the sidebar, but instead it shows the 'Properties' button found in Galaxy's icon theme.

Another example of this can be found with :-
1) Open Writer
2) Insert an image
3) The 'Forward One' and 'Back One' button are from Galaxy
Comment 3 Yousuf Philips (jay) (retired) 2015-06-21 16:26:33 UTC
*** Bug 92168 has been marked as a duplicate of this bug. ***
Comment 4 Yousuf Philips (jay) (retired) 2015-06-21 16:26:53 UTC
*** Bug 92105 has been marked as a duplicate of this bug. ***
Comment 5 Commit Notification 2015-06-23 10:38:37 UTC
Jan Holesovsky committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=43a082f0bba799a9fc33071079f7a42932935e05

tdf#90790: Don't take galaxy as a reference icon set, it may be incomplete.

It will be available in 5.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 6 Jan Holesovsky 2015-06-23 10:44:11 UTC
Fixed now, was a packaging issue.
Comment 7 Commit Notification 2015-06-23 14:31:39 UTC
Jan Holesovsky committed a patch related to this issue.
It has been pushed to "libreoffice-5-0":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=41d1bd782ccb888fdfe2566ad7513bdaccba689d&h=libreoffice-5-0

tdf#90790: Don't take galaxy as a reference icon set, it may be incomplete.

It will be available in 5.0.0.2.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.