Bug 877228 - Clearing the search field in workspace should keep position at last selected message
Summary: Clearing the search field in workspace should keep position at last selected ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Zanata
Classification: Retired
Component: Component-UI
Version: 2.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 2.1
Assignee: Patrick Huang
QA Contact: Ding-Yi Chen
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-16 01:04 UTC by Hedda Peters
Modified: 2013-02-26 04:06 UTC (History)
2 users (show)

Fixed In Version: 2.1-SNAPSHOT (20121205-0926)
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2013-02-26 04:06:20 UTC
Embargoed:


Attachments (Terms of Use)

Description Hedda Peters 2012-11-16 01:04:57 UTC
Description of problem:
When using the simple search field in the workspace, clearing that search again will drop you back at page 1 of the document. This search function needs to accomodate two different purposes though:

1) Finding a certain message within a document. It does that just fine. 
2) Navigating to a certain position within a document (e.g. because that's a chapter I need to review during proofreading; there is a whole bunch of related messages I need to compare in context, etc.) The search function doesn't serve this purpose, as it will not drop you at the position of the search result you selected.


Steps to Reproduce:
1. Open a document with several pages
2. Use simple search bar at the top to search for a term that appears later in the document, hit Enter
3. Click on the desired message from the search result from the list
4. Clear the search at the top and hit Enter again
  
Actual results:
The workspace reloads and drops you at page 1 again

Expected results:
The message you selected from the search result should keep the focus and the workspace should drop you at the appropriate page of the document, showing you all messages before/after the selected one.


Additional info:
Current workaround is using the browsers search function Ctrl-F. But since changing the default view to only 5 messages per page this became impractical, too, for obvious reasons.

Comment 1 Hedda Peters 2012-11-16 01:16:28 UTC
Sorry, default view is 10 messages per page. Doesn't change the above though.

Comment 2 Patrick Huang 2012-11-30 01:51:49 UTC
committed into master:
https://github.com/zanata/zanata/commit/6377e4bf6bf50205311b85005a299387a3bf2791

Comment 3 Ding-Yi Chen 2012-12-05 04:02:18 UTC
VERIFIED with Zanata version 2.1-SNAPSHOT (20121205-0926)


Note You need to log in before you can comment on or make changes to this bug.