Review Request 108693: Places appears twice when unlocking dock widgets.

[Frank Reininghaus]

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


Thanks, that approach looks better indeed. But now, the panel is called "Bookmarks", and the menu entry still is "Places", right?

I think it might be better to keep the panel name "Places" and just rename the first category inside the panel from "Places" to "Bookmarks". For me, a bookmark sounds like something that the user chose, which is the case only for this first category, and not for the devices that are currently connected. What do you think?

Just for the record, there is also another thing which might be considered problematic then: There is an action "Add to Places" in the context menu, but the panel name 'Places' might not be in the default UI any more. However, I think that we should not rename it to "Add to Bookmarks" because the file dialog still calls the entire thing "Places" and because that might also cause confusion concerning the bookmarks in Konqueror.

Well, resolving this issue in a clean way is a lot more complicated than one might think at first sight (which is probably the reason why nobody tried to work on this before).

- Frank Reininghaus


On Feb. 1, 2013, 5:17 p.m., Daniel Kreuter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108693/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2013, 5:17 p.m.)
> 
> 
> Review request for Dolphin and Frank Reininghaus.
> 
> 
> Description
> -------
> 
> When unlocking the dock widgets, the places text will appear twice.
> I removed all titles of the dock widgets and replaced them with an empty string (removed all for consistency).
> In my opinion, this is the easiest solution I could find so far (I made another proposals on the ML a few hours ago).
> 
> 
> This addresses bug 301405.
>     http://bugs.kde.org/show_bug.cgi?id=301405
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinmainwindow.cpp 8ed31de 
> 
> Diff: http://git.reviewboard.kde.org/r/108693/diff/
> 
> 
> Testing
> -------
> 
> Open dolphin and unlock each dock (places, folders, information and terminal). There should only be the two buttons in the title bar, but no title, Drag & Drop should work just like before.
> 
> 
> Thanks,
> 
> Daniel Kreuter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130203/4fd21603/attachment.htm>