Tapestry
  1. Tapestry
  2. TAPESTRY-1180

synchronizeWidgetState doesn't allow position to be used

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1.1
    • Fix Version/s: 4.1.1
    • Component/s: JavaScript
    • Labels:
      None

      Description

      tapestry.widget.synchronizeWidgetState calls on Dojo's createWidget, which takes a parameter specifying where the widget should be placed in relation to the DOM node. However, synchroniseWidgetState does not take this parameter, so the positioning functionality is unavailable.

        Activity

        Hide
        Phil McCarthy added a comment -

        Patch to widget.js that allows Widget position to be specified relative to widget DOM node.

        Show
        Phil McCarthy added a comment - Patch to widget.js that allows Widget position to be specified relative to widget DOM node.
        Hide
        Jesse Kuhnert added a comment -

        Fix made, thanks.

        FYI there were two problems with the patch:

        -) It referenced a file with a different name than what is in the apache repo

        -) It referenced the file directly as in "something/<foo>Widget.js" instead of "tapestry/tapestry-framework/src/js/tapestry/widget/Widget.js" .

        Not that I'm not grateful for the fix, just thought you should know for future patches. (if any)

        Show
        Jesse Kuhnert added a comment - Fix made, thanks. FYI there were two problems with the patch: -) It referenced a file with a different name than what is in the apache repo -) It referenced the file directly as in "something/<foo>Widget.js" instead of "tapestry/tapestry-framework/src/js/tapestry/widget/Widget.js" . Not that I'm not grateful for the fix, just thought you should know for future patches. (if any)

          People

          • Assignee:
            Unassigned
            Reporter:
            Phil McCarthy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development