[rekonq] Review Request: Remove "Ad Bookmark" and "New Bookmark Folder" actions from the bookmark folders' menu

Furkan Üzümcü furkanuzumcu at gmail.com
Mon Aug 22 09:41:04 UTC 2011



> On Aug. 21, 2011, 4:05 p.m., Andrea Diamantini wrote:
> > It seems reasonable to me. But I have no clear opinion about as I don't use the bk toolbar. Others?
> 
> Yoann Laissus wrote:
>     Yes, I think removing those two actions is a good compromise. There are many other way to add a bookmark in a specific folder.

The removed actions exist in the context menu when you right click a bookmark folder.


- Furkan


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


On Aug. 21, 2011, 3:03 p.m., Furkan Üzümcü wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102397/
> -----------------------------------------------------------
> 
> (Updated Aug. 21, 2011, 3:03 p.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> Remove "Ad Bookmark" and "New Bookmark Folder" actions from the bookmark folders' menu, because they take up too many space and they aren't really necessary there.
> 
> 
> Diffs
> -----
> 
>   src/bookmarks/bookmarkstoolbar.cpp 6fa8fde 
> 
> Diff: http://git.reviewboard.kde.org/r/102397/diff
> 
> 
> Testing
> -------
> 
> Compiles and works.
> 
> 
> Thanks,
> 
> Furkan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20110822/6c88bf99/attachment.html>


More information about the rekonq mailing list