Bug 107277 - Needed Menubar, Toolbar icons discussion
Summary: Needed Menubar, Toolbar icons discussion
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:5.4.0
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-19 22:28 UTC by andreas_k
Modified: 2017-10-17 09:32 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
missing icons for the toolbar (83.57 KB, image/png)
2017-04-19 22:34 UTC, andreas_k
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andreas_k 2017-04-19 22:28:56 UTC
as I need for each request a bug report. this is the generic bug report for missing icons.

Some actions are missing icons there are some reason for this
1. no icon is available
2. no icon is needed, because the action wasn't used in a toolbar

so I'd like to discuss here if 1 or 2 isn't correct any more.
Comment 1 andreas_k 2017-04-19 22:34:19 UTC
Created attachment 132702 [details]
missing icons for the toolbar

Missing icon for the actions
- findbar (open the find toolbar when click on it)
- pagedialog (all dialog actions in the format setting has an icon so why not pagedialog. in addition I use this action in the uploaded screenshot
- update all (icon is available and I use the action, so I need an icon for it)
Comment 2 Commit Notification 2017-04-20 07:43:30 UTC
andreas kainz committed a patch related to this issue.
It has been pushed to "master":

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

tdf#107277 add update all icon for writer menu

It will be available in 5.4.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 3 Jan Holesovsky 2017-04-20 08:57:08 UTC
Just for the record - no need to accompany every commit with a bug report if the change is obvious from the commit summary :-)

A commit like:

----- 8< -----
Add icons for X, Y, and Z

The icons were not needed so far, because the actions were not
available in the toolbar previously.
----- 8< -----

is perfectly fine, and does not need a bugreport.  Creating bugreports for ~trivial stuff by commiters themselves just means lots of unnecessary work, and general "catch all" bugreports make no sense either.
Comment 4 Xisco Faulí 2017-04-21 09:41:46 UTC
Moving to NEW as there's already a commit for this bug
Comment 5 Commit Notification 2017-04-22 07:26:28 UTC
andreas kainz committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=93b500ee7c1cbb141bb77b94054f3bee9c3b510a

tdf#107277 add findbar and pagedialog icon to writer

It will be available in 5.4.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 Xisco Faulí 2017-06-09 09:56:34 UTC
Hello,
Is this bug fixed?
If so, could you please close it as RESOLVED FIXED?