Review Request 130157: Bug 321575 - Change window title when searching

[Christoph Feck]

> On June 13, 2017, 9:08 p.m., Christoph Feck wrote:
> > src/dolphinviewcontainer.h, line 96
> > <https://git.reviewboard.kde.org/r/130157/diff/1/?file=496259#file496259line96>
> >
> >     This is bad. If code outside of the class needs to access the m_searchBox member, we don't just make it public, but add proper access methods.
> >     
> >     In this case, only getting the current search string is needed, so add a method something like currentSearchText().
> 
> Christoph Feck wrote:
>     Or just searchText()

Or not searchText(), because it doesn't search text :)


- Christoph


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


On June 13, 2017, 9:10 p.m., XY Quadrat wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/130157/
> -----------------------------------------------------------
> 
> (Updated June 13, 2017, 9:10 p.m.)
> 
> 
> Review request for Dolphin, Elvis Angelaccio and Emmanuel Pescosta.
> 
> 
> Bugs: 321575
>     https://bugs.kde.org/show_bug.cgi?id=321575
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> When you search for a file (e.g. "hello world") the window title of Dolphin will change to "Searching hello world". This commit includes moving the DolphinSearchBox object created in dolphinviewcontainer.h from private to public, so that dolphinmainwindow.cpp can access the input the user typed in.
> 
> 
> Diffs
> -----
> 
>   src/dolphinmainwindow.h d741eb2 
>   src/dolphinmainwindow.cpp 8d58043 
>   src/dolphinviewcontainer.h e50386a 
> 
> Diff: https://git.reviewboard.kde.org/r/130157/diff/
> 
> 
> Testing
> -------
> 
> - Disables it iself after search bar is hidden
> - Works with multiple word searches
> 
> 
> Thanks,
> 
> XY Quadrat
> 
>

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