Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0.1
    • Component/s: demos, site
    • Labels:
      None

      Description

      To better align our Demos to IIA concepts, I'm thinking to re-enable JNLP links for our Demos, so they can run even with Java Web Start as Applications.

      And maybe changing the text View with something like "Run as Applet", and add bottom another link like "Run as Application" ...
      Note that this could help even to improve the experience to all users that has/had troubles running our Applets (some thread on Pivot-2.0 and the IcedTea plugin on Linux, and some problems on some Dell machines with Windows, etc).

      Note that those links was removed by PIVOT-500 issue ( https://issues.apache.org/jira/browse/PIVOT-500 ), but with latest Java 6 Update 21 and upper (currently 6 Update 24) there isn't that problem, so now it's safe to re-enable them.

        Issue Links

          Activity

          Hide
          Sandro Martini added a comment -

          Of course then we could do the same even fot Demos running from our Web Site ... but let's see later this.

          Show
          Sandro Martini added a comment - Of course then we could do the same even fot Demos running from our Web Site ... but let's see later this.
          Hide
          Sandro Martini added a comment -

          I'm thinking to set in jnlp files the Java 6 Update 24 (or at least the Update 21) as a requirement to run our Java Web Start Demos ... do you think it's good for most platforms (I've just seen that even Apple has this version now) ?

          And maybe update to the Java 6 Update 23 (or at least Update 21) as a recommended requirement for Pivot, as reported in some out txt file, ok ?

          Tell me.

          Bye

          Show
          Sandro Martini added a comment - I'm thinking to set in jnlp files the Java 6 Update 24 (or at least the Update 21) as a requirement to run our Java Web Start Demos ... do you think it's good for most platforms (I've just seen that even Apple has this version now) ? And maybe update to the Java 6 Update 23 (or at least Update 21) as a recommended requirement for Pivot, as reported in some out txt file, ok ? Tell me. Bye
          Hide
          Sandro Martini added a comment -

          Reverted the change in index.xsl under demos/xsl/ , so now jnlp link are generated in demos.war , tried all under a local Tomcat and Web Start Demos are working so if there aren't objections in a few days I'll commit the (revert) change.

          On re-enabling them even in our Web Site probably we have to wait just after the 2.0.1 final release.

          Show
          Sandro Martini added a comment - Reverted the change in index.xsl under demos/xsl/ , so now jnlp link are generated in demos.war , tried all under a local Tomcat and Web Start Demos are working so if there aren't objections in a few days I'll commit the (revert) change. On re-enabling them even in our Web Site probably we have to wait just after the 2.0.1 final release.
          Hide
          Sandro Martini added a comment -

          Resolved

          Show
          Sandro Martini added a comment - Resolved
          Hide
          Sandro Martini added a comment -

          Note that (moved here from PIVOT-406):
          in demo.jnlp.xsl, the root param (set from build.xml) has to be fixed because for web start it need the full codebase (and currently in DesktopApplicationContext this is not read ... for this there will be a dedicated issue for Pivot 2.1).
          But in demo.html.xsl that parameter is right, so it could not be changed there (and nor in large-data.xml).

          A quick solution for 2.0.1 (without having to touch code), could be to set by hand in build.properties a variable, and use it inside build.xml , see the patch in attach.

          For a better solution (for Web Start, because on Applets it already works), in 2.1 maybe handle it using the codebase (read from Web Start BasicService) and overwrite the origin parameter. See the related issue.

          Show
          Sandro Martini added a comment - Note that (moved here from PIVOT-406 ): in demo.jnlp.xsl, the root param (set from build.xml) has to be fixed because for web start it need the full codebase (and currently in DesktopApplicationContext this is not read ... for this there will be a dedicated issue for Pivot 2.1). But in demo.html.xsl that parameter is right, so it could not be changed there (and nor in large-data.xml). A quick solution for 2.0.1 (without having to touch code), could be to set by hand in build.properties a variable, and use it inside build.xml , see the patch in attach. For a better solution (for Web Start, because on Applets it already works), in 2.1 maybe handle it using the codebase (read from Web Start BasicService) and overwrite the origin parameter. See the related issue.
          Hide
          Sandro Martini added a comment -

          reopen just to put the short-term slution (in attach)

          Show
          Sandro Martini added a comment - reopen just to put the short-term slution (in attach)
          Hide
          Sandro Martini added a comment -

          short-term workaround

          Show
          Sandro Martini added a comment - short-term workaround

            People

            • Assignee:
              Sandro Martini
              Reporter:
              Sandro Martini
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development