I have unfortunately found another issue. Steps to reproduce
1. Use arrow keys to select a folder such as "Popular urls" (this works
if the folder was selected by searching as well e.g typing 'popu')
2. Use right arrow to enter folder
3. Use left arrow or esc to close submenu. Submenu item in main menu
remains highlighted and we receive a 'QEvent::Close' as expected
4. Type a letter too search
5. Search gets reset by two spurious QEvent::Close events
This does not seem too happen if the folder was simply highlighted but
never activated (I.e no right arrow key)
/Rade
On 26/03/18 10:35, Nikita Melnichenko wrote:
> Ok, so everything from the previous list is solved now. We are almost
> ready to merge. Only two new issues are remaining:
>
> 1. D11706 (the issue with jump back actions not closing bookmark menu
> when triggered).
> 2. This is what I've found recently: quick search bar doesn't appear if
> you open bookmark menu in another panel (actually it's even for
> another tab of the same panel as the tabs have different menus for
> some reason), i.e. only first tab you opened the menu will have
> search bar properly visible. It's probably something to do with
> QWidgetAction usage - I didn't have time to debug it yet. Martin,
> Rade - could you check, please? I'll be busy in the next few days.
>
> Once we resolve these two and other we find (I hope we won't find any),
> I'll submit the CR to the final review and testing (mostly for people
> who hasn't participated in the intermediate reviews).
>
> Thanks,
> Nikita.
>
> Nikita Melnichenko wrote:
>> We need to track the outstanding issues somewhere. Let's use this
>> thread for the purpose, so the list resides in a single place. This is
>> a summary from various CRs:
>>
>> * Accelerator behavior - Solved.
>> * UI element to show what's typed already - Solved.
>> * When bookmark search is started with 's' or 'j', the search is
>> interrupted - Needs a review, D11444
>> * Clear highlighting - Solved.
>> * Folder-related: activation in multiple matches without breaking
>> the search - Needs a review, D11443
>> * Folder-related: swallowing of keys - Needs a review, D11443
>> * Folder-related: opening of submenu resets the search, subsequent
>> keypress search in submenu - No progress.
>> * Not-found handling (search is reset in the middle) - Solved.
>> * Don't search in special bookmarks - No progress.
>> * quickSearchBar should be visible by default so it's discoverable
>> and there is no visual jumping. Ideally with a placeholderText
>> "Type to search...". - In progress
>>
>> Please add items if I missed something.
>>
>
>
https://groups.google.com/d/msgid/krusader-devel/976bb8be-cc80-8230-874f-a054afc9de0d%40melnichenko.name
> <
https://groups.google.com/d/msgid/krusader-devel/976bb8be-cc80-8230-874f-a054afc9de0d%40melnichenko.name?utm_medium=email&utm_source=footer>.