Merge lp://qastaging/~gdesklets-desklet-team/gdesklets/develbook-control-array into lp://qastaging/~gdesklets-desklet-team/gdesklets/develbook

Proposed by Joe Sapp
Status: Merged
Approved by: Joe Sapp
Approved revision: 10
Merged at revision: 11
Proposed branch: lp://qastaging/~gdesklets-desklet-team/gdesklets/develbook-control-array
Merge into: lp://qastaging/~gdesklets-desklet-team/gdesklets/develbook
Diff against target: 206 lines (+72/-12)
5 files modified
ChangeLog (+6/-0)
ctrl-use.xml (+30/-3)
info.xml (+2/-2)
tmp.xml (+32/-5)
xibook.xml (+2/-2)
To merge this branch: bzr merge lp://qastaging/~gdesklets-desklet-team/gdesklets/develbook-control-array
Reviewer Review Type Date Requested Status
Bjoern Koch Approve
gDesklets Core Team Pending
Review via email: mp+24055@code.qastaging.launchpad.net

Description of the change

Added information on using Control arrays

To post a comment you must log in.
10. By Joe Sapp

Merged main branch

Revision history for this message
Bjoern Koch (h.humpel) wrote :

Looks good.

But I am just wondering if we should skip the Control IDs as we don't need them anymore:

<programlisting><![CDATA[
...
<control id="myctrl" interface="ITime" length="1"/>

<script>
 Dsp.lbl1.value = str(myctrl[0].time)
</script>
...
 ]]></programlisting>

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: