Details

    • Type: Feature Request Feature Request
    • Status: Reopened Reopened
    • Priority: Critical Critical
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: search
    • Labels:
      None
    • Previous Issue Keys:
      SAK-9535

      Description

      If I have a search that returns 70 results, it seems to want to show only a max of 6 page numbers (e.g. initially shows 1 2 3 4 5 6) and no more than 3 prior to the page that you've selected (e.g. when you're on 7 it shows 5 6 7).

      If so I think it would help to just increase those numbers, e.g. Google shows current page + next 10 + previous 10.

      Other UI changes that would help here are Previous and Next links (also like Google) as it's hard to click on the numbers, and also to unlink the current page so it's easier to see what page number you're on.
      1. pager.html
        5 kB
        Ian Boston
      2. pager.html
        4 kB
        Ian Boston
      1. search design.png
        92 kB
      2. search design 2.png
        94 kB
      3. search design 3.png
        93 kB

        Activity

        Hide
        Ian Boston added a comment -
        Scheduling for 2.5
        Show
        Ian Boston added a comment - Scheduling for 2.5
        Hide
        Ian Boston added a comment -
        Harriet,

        Can you give me some UI design on this.

        Thanks
        Show
        Ian Boston added a comment - Harriet, Can you give me some UI design on this. Thanks
        Hide
        Peter A. Knoop added a comment -
        [Bulk Change] This issue is currently Unresolved, however, it has a Fix Version set. In keeping with the newly added Target Version, Fix Versions should only be set for Resolved Issues, and only after it has been merged to that version specifically. The Fix Version is being reset to Unknown for this issue. Please use the Target Version to indicate when you plan to address this issue.
        Show
        Peter A. Knoop added a comment - [Bulk Change] This issue is currently Unresolved, however, it has a Fix Version set. In keeping with the newly added Target Version, Fix Versions should only be set for Resolved Issues, and only after it has been merged to that version specifically. The Fix Version is being reset to Unknown for this issue. Please use the Target Version to indicate when you plan to address this issue.
        Hide
        Harriet Truscott added a comment -
        Search tool - starting point

        First page now includes 10 pages,

        Current page is in red (#FF0000)

        Subsequent page numbers are in standard link blue, without an underline. Page numbers are well spaced out for people with motor control difficulties.

        'Next page' takes the person to the next page - so from page 1, it takes them to page 2.

        A forward arrow appears between the numbers and the 'newxt page' . Clicking this has the same effect as clicking 'next page'. (Icon is Silk result_set_next)

        Note that the same links would be shown in the same style at the bottom of the page too.
        Show
        Harriet Truscott added a comment - Search tool - starting point First page now includes 10 pages, Current page is in red (#FF0000) Subsequent page numbers are in standard link blue, without an underline. Page numbers are well spaced out for people with motor control difficulties. 'Next page' takes the person to the next page - so from page 1, it takes them to page 2. A forward arrow appears between the numbers and the 'newxt page' . Clicking this has the same effect as clicking 'next page'. (Icon is Silk result_set_next) Note that the same links would be shown in the same style at the bottom of the page too.
        Hide
        Harriet Truscott added a comment -
        This is the final page of results

        The previous 10 pages are displayed.

        The 'previous' link takes you to the page before the one you are currently on. Thus on page 20, clicking 'previous' takes you to page 19.

        The arrow icon (result_set_previous) takes you to the same page as 'previous' link

        It is 'previous' rather than 'previous page' for reasons of space, and because I think that there's a little more confusion about what these links will take you to (i.e. is it next 10 results? next page?) when you start than by the time you've got to the final results page.
        Show
        Harriet Truscott added a comment - This is the final page of results The previous 10 pages are displayed. The 'previous' link takes you to the page before the one you are currently on. Thus on page 20, clicking 'previous' takes you to page 19. The arrow icon (result_set_previous) takes you to the same page as 'previous' link It is 'previous' rather than 'previous page' for reasons of space, and because I think that there's a little more confusion about what these links will take you to (i.e. is it next 10 results? next page?) when you start than by the time you've got to the final results page.
        Hide
        Harriet Truscott added a comment -
        When you're in the middle of your search resutls, you see both 'previous' and 'next page' links, and both arrows. Again, our current page is indicated in red, with others in blue.

        Google puts its numbers above 10 very close together, which allows 21 numbers to fit onto the page.

        I always feel a little bit uncomfortable about how close the Google numbers are - for clarity and for people with motor difficulties, I'd prefer to have 8 numbers either side, better spaced out, as shown on this screen.
        Show
        Harriet Truscott added a comment - When you're in the middle of your search resutls, you see both 'previous' and 'next page' links, and both arrows. Again, our current page is indicated in red, with others in blue. Google puts its numbers above 10 very close together, which allows 21 numbers to fit onto the page. I always feel a little bit uncomfortable about how close the Google numbers are - for clarity and for people with motor difficulties, I'd prefer to have 8 numbers either side, better spaced out, as shown on this screen.
        Hide
        Ian Boston added a comment -
        DHTML Pager implementation with 2 modes. To check the operation before implementation
        Show
        Ian Boston added a comment - DHTML Pager implementation with 2 modes. To check the operation before implementation
        Hide
        Ian Boston added a comment -
        I have attached a pager impl in JS/HTML, can you check that this is what you want for behavior
        Show
        Ian Boston added a comment - I have attached a pager impl in JS/HTML, can you check that this is what you want for behavior
        Hide
        Harriet Truscott added a comment -
        I think the styling is good - it's very clear which page you're on, and it looks pleasant

        'next page' v 'previous' isn't consistent with Google - I'm not saying that Google necessarily made the right decision here, but we do know it's the de-facto standard and what our users will probably expect.

        So, on page N clicking 'next page' should take you onto page N + 1, not page N + 10 as it currently does.

        On page N, clicking 'previous' should take you to page N - 1, not page N - 10. I'd have a lower-case letter starting 'previous' for consistency.

        Clicking on the arrow should have the same behaviour.

        I'd go for a width of 15 and with the 'centre' behaviour.
        Show
        Harriet Truscott added a comment - I think the styling is good - it's very clear which page you're on, and it looks pleasant 'next page' v 'previous' isn't consistent with Google - I'm not saying that Google necessarily made the right decision here, but we do know it's the de-facto standard and what our users will probably expect. So, on page N clicking 'next page' should take you onto page N + 1, not page N + 10 as it currently does. On page N, clicking 'previous' should take you to page N - 1, not page N - 10. I'd have a lower-case letter starting 'previous' for consistency. Clicking on the arrow should have the same behaviour. I'd go for a width of 15 and with the 'centre' behaviour.
        Hide
        Harriet Truscott added a comment -
        Comments left above
        Show
        Harriet Truscott added a comment - Comments left above
        Hide
        Ian Boston added a comment -
        Hmmm, Sorry

        Not certain about the paging (next and previous), after testing here is why.

        If next does current page+1 and previous does current page -1 .....

        why do we remove previous when the first page in the list is page 0 and the current page is page 7 ?

        previous-1 would be 6 which is valid so the control is valid

        at the other end on page 93 (100 pages) we are not displaying next but page+1 is valid.

        If N = CP+1 and P = CP-1 then it makes no sense to hide them except when the page is 0 or 99 and
        the possition of next and previous is wrong and should be beside the current page.


        IMHO, next and previous are rapid scrolling controls that jump so that the current page becomes the page just beyond the last one shown.

        Soooo....

        I've uploaded a CP+-1 version (the second pager)



        Show
        Ian Boston added a comment - Hmmm, Sorry Not certain about the paging (next and previous), after testing here is why. If next does current page+1 and previous does current page -1 ..... why do we remove previous when the first page in the list is page 0 and the current page is page 7 ? previous-1 would be 6 which is valid so the control is valid at the other end on page 93 (100 pages) we are not displaying next but page+1 is valid. If N = CP+1 and P = CP-1 then it makes no sense to hide them except when the page is 0 or 99 and the possition of next and previous is wrong and should be beside the current page. IMHO, next and previous are rapid scrolling controls that jump so that the current page becomes the page just beyond the last one shown. Soooo.... I've uploaded a CP+-1 version (the second pager)
        Hide
        Ian Boston added a comment -
        New version with 2 pagers.

        The first one is a jump pager on the next and prev buttons

        The second one is a page+1 page -1
        Show
        Ian Boston added a comment - New version with 2 pagers. The first one is a jump pager on the next and prev buttons The second one is a page+1 page -1
        Hide
        Ian Boston added a comment -
        New pager versions uploaded
        Show
        Ian Boston added a comment - New pager versions uploaded
        Hide
        Stephen Marquard added a comment -
        Reopening as an unassigned FR because the UI is still in need of improvement.
        Show
        Stephen Marquard added a comment - Reopening as an unassigned FR because the UI is still in need of improvement.
        Hide
        David Horwitz added a comment -
        MAINT TEAM REVIEW: This feature request is currently unassigned and will be reviewed. In line with stated Jira practice [http://confluence.sakaiproject.org/display/MGT/Sakai+Jira+Guidelines] Feature requests that are not going to be implemented will be closed with a status of "wont fix". If you intend implementing this issue please ensure that its up to date and assigned correctly
        Show
        David Horwitz added a comment - MAINT TEAM REVIEW: This feature request is currently unassigned and will be reviewed. In line with stated Jira practice [ http://confluence.sakaiproject.org/display/MGT/Sakai+Jira+Guidelines ] Feature requests that are not going to be implemented will be closed with a status of "wont fix". If you intend implementing this issue please ensure that its up to date and assigned correctly

          People

          • Assignee:
            Unassigned
            Reporter:
            Stephen Marquard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: