[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