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

Hardcoded skin in samigo standalone servlet

    XMLWordPrintable

    Details

    • Previous Issue Keys:
      SAM-2246

      Description

      So it looks like samigo includes this header.inc, and in standalone mode it's designed to just use the standard css. If the site uses a different skin, then this can't be used. Some of this problem is mentioned on SAK-23407, and this is how lessons links into it. The problem is that it's not easy to figure out the site context or what skin it should be using because the intention is that it generally should be running standalone. Maybe someone has some creative ideas for fixing this?

      I was thinking that perhaps the portal (headscripts) it could just omit it's css and the header could 'fix up' the css of this iframe.
      http://stackoverflow.com/questions/217776/how-to-apply-css-to-iframe

      Or perhaps the page that requests the Begin Assessment page can send some data to pass along to the beginAssessment.

      For instance Lessons passes a good amount of request data to /samigo-app/servlet/Login but it seems like much of it is discarded, even if some was passed along it could fix this.

      // supply the standalone versions tool styles
      request.setAttribute("html.head",
      "<link href=\"/library/skin/tool_base.css\" type=\"text/css\" rel=\"stylesheet\" media=\"all\" />\n" +
      "<link href=\"/library/skin/default/tool.css\" type=\"text/css\" rel=\"stylesheet\" media=\"all\" />\n" +
      "<link href=\"/samigo-app/css/tool_sam.css\" type=\"text/css\" rel=\"stylesheet\" media=\"all\" />\n");
      request.setAttribute("html.body.onload", "");

        Gliffy Diagrams

          Zeplin

            Attachments

            1. SAM-2246.patch
              1 kB
              Ramon Garcia Martinez

              Issue Links

                Activity

                  People

                  Assignee:
                  ramongar Ramon Garcia Martinez
                  Reporter:
                  jonespm Matthew Jones
                  Votes:
                  1 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration