Review Request 110369: Hide MessageWidget when user changes the URL

[Commit Hook]

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


This review has been submitted with commit 5324fcd315a78f7f2f7ae59bb78a5029e3951674 by Frank Reininghaus to branch master.

- Commit Hook


On May 8, 2013, 11:07 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110369/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 11:07 p.m.)
> 
> 
> Review request for Dolphin, Frank Reininghaus and Aurélien Gâteau.
> 
> 
> Description
> -------
> 
> This patch makes Dolphin hide the MessageWidget as soon as the user manually changes the URL by editing the address bar or entering another folder.
> 
> I, personally, consider the MessageWidget shifting the view down not distracting or annoying, more the fact that it stays there forever until you close it.
> Imho it makes sense to hide the MessageWidget as soon as the URL is changed by the user because usually the message only contains information relevant to the current view as is.
> 
> For example trying to delete a folder that you are not allowed to, as soon as you change into another folder, you don't see the one you're trying to delete and so there is no point in having it still tell you you couldn't.
> Or, if you try to access a folder you're not allowed to, you get an empty view and have to go back/leave manually anyway, so you will have noticed the empty view and the error.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinviewcontainer.h bc58531 
>   dolphin/src/dolphinviewcontainer.cpp de1ae4b 
> 
> Diff: http://git.reviewboard.kde.org/r/110369/diff/
> 
> 
> Testing
> -------
> 
> - Try to delete a folder you're not allowed to and go to another folder ? Message Widget closes
> - Try to access a folder you're not allowed to and go back ? Message Widget closes
> - Try to access a folder that doesn't exist and enter another URL manually ? Message Widget closes
> - Try to access a folder that doesn't exist and go to another location by clicking a parent breadcrumb ? Message Widget closes
> 
> Only issue I have found so far:
>  1. Navigate to ftp://someserverthatdoesntexist.com ? Get "Could not connect message"
>  2. Navigate to ftp://anotherserverthatdoesntexist.com ? The MessageWidget closes but the error message KIO produces is emitted before the MessageWidget is fully closed (when it animates) and so the error is discarded and you do not get an error. (We probably need to cancel the hide animation and/or force a new one to be shown anyways)
> 
> I am not sure if the place I put it is the right one, as I only intended to make the widget hide when changing location via URL bar but turned out it also changes when clicking a folder in the view (whose behavior is fine imho but the code should probably connect to the view url changed signal?)
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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