[rekonq] Re: Review Request: Add favorite with 2 clicks in the bookmark icon

[Andrea Diamantini]

> On July 16, 2011, 9:10 a.m., Pierre Rossi wrote:
> > Yeah! that's a really cool addition I think. Small usability details we might need to test and see over time: does the favorite have to be bookmarked by default ? (I know it only takes one click to undo, but it also only takes one click to do...)
> > As an evolution for the right hand side icons, maybe we could have custom context menus for them, or at least add some custom actions to the lineedit's context menu.

The "forced" bookmark question is in fact a weakness of the patch. What do you think about "doubleclick"?


- Andrea


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


On July 13, 2011, 4:37 p.m., Andrea Diamantini wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101945/
> -----------------------------------------------------------
> 
> (Updated July 13, 2011, 4:37 p.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> Provide a "click" mechanism to manage favorites
>     
> With this commit, you can add a favorite by clicking 2 times in the bookmark icon.
> If an URL is a favorite, the favorite icon appears and then you can manage it from there...
> 
> Pushed on remote branch: AddFavoriteWithAClick
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt f623adf 
>   src/urlbar/favoritewidget.h PRE-CREATION 
>   src/urlbar/favoritewidget.cpp PRE-CREATION 
>   src/urlbar/urlbar.h 9ed89da 
>   src/urlbar/urlbar.cpp 078dc6a 
> 
> Diff: http://git.reviewboard.kde.org/r/101945/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andrea
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20110717/f2d71183/attachment.htm