Review Request 129589: Avoid opening duplicate Properties dialogs

[Christoph Feck]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129589/#review101182
-----------------------------------------------------------




src/views/dolphinviewactionhandler.h (line 28)
<https://git.reviewboard.kde.org/r/129589/#comment67804>

    Use 'class' forward declaration instead of include, see lines below.



src/views/dolphinviewactionhandler.h (line 217)
<https://git.reviewboard.kde.org/r/129589/#comment67805>

    Spelling



src/views/dolphinviewactionhandler.cpp (line 57)
<https://git.reviewboard.kde.org/r/129589/#comment67806>

    Use member initialization instead of assignment.



src/views/dolphinviewactionhandler.cpp (line 602)
<https://git.reviewboard.kde.org/r/129589/#comment67807>

    Should we also raise when the dialog was already created?


- Christoph Feck


On Nov. 30, 2016, 12:48 a.m., Mario Bereš wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129589/
> -----------------------------------------------------------
> 
> (Updated Nov. 30, 2016, 12:48 a.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Bugs: 372703
>     https://bugs.kde.org/show_bug.cgi?id=372703
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> You could open duplicate Properties dialogs because every time u hit "Properties" button, a new KPropertiesDialog is created. Now it's stored in class variable until it destroys, if you try to open another KPropertiesDialog it will focus back to already opened one.
> 
> 
> Diffs
> -----
> 
>   src/views/dolphinviewactionhandler.h eb375b5 
>   src/views/dolphinviewactionhandler.cpp 858f929 
> 
> Diff: https://git.reviewboard.kde.org/r/129589/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mario Bereš
> 
>

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