Uploaded image for project: 'MyFaces Tomahawk'
  1. MyFaces Tomahawk
  2. TOMAHAWK-945

Split the PPR Example into smaller easy to understand Examples with explanations

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.7-SNAPSHOT
    • Fix Version/s: 1.1.7
    • Component/s: PPRPanelGroup
    • Labels:
      None

      Description

      A lot of users have been complaining, that the sandbox PPR example is difficult to understand and they
      did't get it working. Therefore it would be good to split this example into a number of different
      examples for the different features similar to the datatable examples

        Activity

        Hide
        bischoje Jeff Bischoff added a comment -

        Are you volunteering, Ernst?

        Show
        bischoje Jeff Bischoff added a comment - Are you volunteering, Ernst?
        Hide
        ernief Ernst Fastl added a comment -

        Yes I am. I am almost finished, just a nasty issue in common.js with the commandLinks which I still need to figure out
        if anyone knows how this renderMessages javascript works I could attach a patch containing what I have
        so far...

        Show
        ernief Ernst Fastl added a comment - Yes I am. I am almost finished, just a nasty issue in common.js with the commandLinks which I still need to figure out if anyone knows how this renderMessages javascript works I could attach a patch containing what I have so far...
        Hide
        ernief Ernst Fastl added a comment -

        new more easy to understand examples. Just commandLinks are not working due to some method in common.js

        Show
        ernief Ernst Fastl added a comment - new more easy to understand examples. Just commandLinks are not working due to some method in common.js
        Hide
        bierbrauer Gerald Müllan added a comment -

        I will commit the patch so far, but leave this issue open because of the missing common.js fix.

        Show
        bierbrauer Gerald Müllan added a comment - I will commit the patch so far, but leave this issue open because of the missing common.js fix.
        Hide
        bischoje Jeff Bischoff added a comment -

        Okay Ernst, sorry for asking. I just didn't want somebody else to start working on it, if you already were! : )

        Great that you are working on this!

        Show
        bischoje Jeff Bischoff added a comment - Okay Ernst, sorry for asking. I just didn't want somebody else to start working on it, if you already were! : ) Great that you are working on this!
        Hide
        ernief Ernst Fastl added a comment -

        Good idea, I wouldn't have liked that as well. thanks Jeff

        Show
        ernief Ernst Fastl added a comment - Good idea, I wouldn't have liked that as well. thanks Jeff
        Hide
        mmarinschek Martin Marinschek added a comment -

        as of my latest tests, links are working fine now.

        regards,

        Martin

        Show
        mmarinschek Martin Marinschek added a comment - as of my latest tests, links are working fine now. regards, Martin

          People

          • Assignee:
            bierbrauer Gerald Müllan
            Reporter:
            ernief Ernst Fastl
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development