Review Request: Change Online help icon KHelpcenter

[Albert Astals Cid]

> On April 17, 2012, 1:45 p.m., Sebastian Kügler wrote:
> > This change is wrong, as the menu entry has nothing to do with the semantic meaning of the icon, and the icon is not named according to the icon spec.
> > 
> > So the correct icon is already set here, if its look doesn't match, then that icon would need to be fixed. In this case, I assume you mean to better reflect the "online" part in the name, and I agree that it's not reflected in the name. Question is: does it matter where the help is located? (Surely does if the user is offline, but in general ... I think the "help!" part is important, not the "online" part.

+1


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104621/#review12573
-----------------------------------------------------------


On April 16, 2012, 5:28 p.m., Maarten De Meyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104621/
> -----------------------------------------------------------
> 
> (Updated April 16, 2012, 5:28 p.m.)
> 
> 
> Review request for KDE Runtime and Cornelius Schumacher.
> 
> 
> Description
> -------
> 
> Changes the 'Online help' icon in the navigation to a more fitting one.(imho)
> 
> 
> Diffs
> -----
> 
>   khelpcenter/plugins/onlinehelp.desktop 540f83f 
> 
> Diff: http://git.reviewboard.kde.org/r/104621/diff/
> 
> 
> Testing
> -------
> 
> compiled and run, works fine
> 
> 
> Thanks,
> 
> Maarten De Meyer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120417/5b17c138/attachment.htm>