Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-20862

allow categorized display of "more" sites control without dhtmlMore panel

    XMLWordPrintable

    Details

    • Type: Task
    • Status: CLOSED
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.8.0
    • Fix Version/s: None
    • Component/s: Portal
    • Labels:
      None

      Description

      The <select> used in the "more sites" portal navigation is a flat list. Use existing categorization (term, site type, other) to produce a categorized select using <optgroup> to generate an easier to parse list.

      Here are some summary notes from a Matt Jones email:
      --------------------------
      Gonzalo did some initial feasibility research and modified the .vm based on the DHTML more work. This seems like a really quick implementation because of that.

      Yea, the only thing I'm not sure about is the term order.
      We'd for sure want it to be in correct term order, with the project and other sites on the bottom.

      I think the term order can be manually controlled by a property (portal.term.order) but this is annoying having to add new terms every semester.
      All that is controlled in
      ./portal-impl/impl/src/java/org/sakaiproject/portal/charon/site/MoreSiteViewImpl.java

      However, we also have the academic sessions in the database (getAcademicSessions), and it is sorted because of "ORDER BY START_DATE ASC". In the edu-services (the default), it's also sorted by start date, so I think this would be a good change to make too.
      . . .
      // The federated list should be sorted by start date.
      Collections.sort(resultSet, startDateComparator);

      So,

      • Add a new property (portal.use.term.more ?) to get the java code to generate the dhtml more data for this drop down.
      • Optional: We could use a catch all property like: portal.use.more = dhtml/term/standard to make it easier if we plan to add more, and since you can really only have one and have it look good.
      • Optional: Add a site property incase someone really wants the old version, but I think we could just go all in on this.
      • Add some code to the .vm to display either the regular dropdown or the portal.use.term.more dropdown (Gonzalo already did most of this)
      • Add some code to MoreSiteViewImpl to use the sorted output from CourseManagementService.getAcademicSessions as the initial sort order rather than the property (if the property isn't defined)

      See attachment

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  umich-dev UMich Developers
                  Reporter:
                  gsilver Gonzalo Silverio
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration