Jetspeed 2
  1. Jetspeed 2
  2. JS2-546

Fix the sort function on the DB Browser

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1-dev, 2.1
    • Fix Version/s: 2.1.4
    • Component/s: Demo Portlets
    • Labels:
      None

      Description

      A title tip and mouse icon does not appear when hovering over the column name. In fact you don't even know that you can sort the column until you look at the template code. Perhaps this is intentional

      Can not patch the file database-view.vm file because svn reports this to be a binary file.

      I changed line 37 to,

      <div align="center"><a href="#" title="Sort $column" onClick="window.location.href='$columnLink';return false;" style="color: white;">$column</a></div>

        Activity

        Hide
        Ate Douma added a comment -

        Thanks for checking Philip.
        I tested out your change, but although I now can click the column headers, the behavior isn't correct so maybe it is intentional
        The sort functionality isn't implemented correctly, so I've updated this issue to an (still) minor bug and a new subject too.
        I've no time to look into this now but maybe someone else will.

        Show
        Ate Douma added a comment - Thanks for checking Philip. I tested out your change, but although I now can click the column headers, the behavior isn't correct so maybe it is intentional The sort functionality isn't implemented correctly, so I've updated this issue to an (still) minor bug and a new subject too. I've no time to look into this now but maybe someone else will.
        Hide
        David Sean Taylor added a comment -

        Look at the DB Browser demo - sorting is implemented there
        its just not implemented in the administrative portlets
        easy enough to add

        Show
        David Sean Taylor added a comment - Look at the DB Browser demo - sorting is implemented there its just not implemented in the administrative portlets easy enough to add
        Hide
        Ate Douma added a comment -

        David, he's looking at the DB Browser demo.
        I've tested it out, and while sorting is implemented, the behavior isn't correct: it sorts decending first and clicking twice on the column doesn't revert that.

        Show
        Ate Douma added a comment - David, he's looking at the DB Browser demo. I've tested it out, and while sorting is implemented, the behavior isn't correct: it sorts decending first and clicking twice on the column doesn't revert that.
        Hide
        Vivek Kumar added a comment -

        Issue has been fixed

        Show
        Vivek Kumar added a comment - Issue has been fixed

          People

          • Assignee:
            Vivek Kumar
            Reporter:
            Philip Mark Donaghy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development