← Back to branch summary

~launchpad-pqm/launchpad/devel

« back to all changes in this revision

Viewing changes to lib/lp/app/javascript/choiceedit/choiceedit.js

  • Committer: Jeroen Vermeulen
  • Date: 2011-09-26 06:30:07 UTC
  • mto: This revision was merged to the branch mainline in revision 14049.
  • Revision ID: jeroen.vermeulen@canonical.com-20110926063007-1fb5eelnidpnra9a
Fix lots of lint in recently-changed files.

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/lp/app/javascript/choiceedit/choiceedit.js
43
43
ChoiceSource.NAME = CHOICESOURCE;
44
44
 
45
45
/**
46
 
 * Dictionary of selectors to define subparts of the widget that we care about.
47
 
 * YUI calls ATTRS.set(foo) for each foo defined here
 
46
 * Dictionary of selectors to define subparts of the widget that we care
 
47
 * about.
 
48
 *
 
49
 * YUI calls ATTRS.set(foo) for each foo defined here.
48
50
 *
49
51
 * @property InlineEditor.HTML_PARSER
50
52
 * @type Object
106
108
 
107
109
    /**
108
110
     * Y.Node (img) displaying the editicon, which is exchanged for a spinner
109
 
     * while saving happens. Should be automatically calculated by HTML_PARSER.
 
111
     * while saving happens. Should be automatically calculated by
 
112
     * HTML_PARSER.
 
113
     *
110
114
     * Setter function returns Y.one(parameter) so that you can pass
111
115
     * either a Node (as expected) or a selector.
112
116
     *
176
180
     */
177
181
    _bindUIChoiceSource: function() {
178
182
        var that = this;
 
183
        var clickable_element;
179
184
        if (this.get('clickable_content')) {
180
 
            var clickable_element = this.get('contentBox');
 
185
            clickable_element = this.get('contentBox');
181
186
        } else {
182
 
            var clickable_element = this.get('editicon');
 
187
            clickable_element = this.get('editicon');
183
188
        }
184
189
        clickable_element.on("click", this.onClick, this);
185
190
 
203
208
        var items = this.get("items");
204
209
        var value = this.get("value");
205
210
        var node = this.get("value_location");
206
 
        for (var i=0; i<items.length; i++) {
 
211
        var i;
 
212
        for (i = 0; i < items.length; i++) {
207
213
            if (items[i].value == value) {
208
214
                node.set("innerHTML", items[i].source_name || items[i].name);
209
215
            }
445
451
        var items = this.get("items");
446
452
        var value = this.get("value");
447
453
        var li;
448
 
        for (var i=0; i<items.length; i++) {
 
454
        var i;
 
455
        for (i = 0; i < items.length; i++) {
449
456
            if (items[i].disabled) {
450
457
                li = Y.Node.create('<li><span class="disabled">' +
451
458
                    items[i].name + '</span></li>');
487
494
            var target = e.currentTarget;
488
495
            var value = target._value;
489
496
            var items = that.get("items");
490
 
            for (var i=0; i<items.length; i++) {
 
497
            var i;
 
498
            for (i = 0; i < items.length; i++) {
491
499
                if (items[i].value == value) {
492
500
                    that.fire("valueChosen", items[i].value);
493
501
                    that.destroy();
524
532
     */
525
533
    _positionCorrectly: function(e) {
526
534
        var boundingBox = this.get('boundingBox');
 
535
        var client_width = document.body.clientWidth;
 
536
        var offset_width = boundingBox.get('offsetWidth');
527
537
        var selectedListItem = boundingBox.one('span.current');
528
538
        valueX = this._mouseX - (boundingBox.get('offsetWidth') / 2);
529
539
        var valueY;
535
545
        } else {
536
546
             valueY = this._mouseY - (boundingBox.get('offsetHeight') / 2);
537
547
        }
 
548
        if (valueX > client_width - offset_width) {
 
549
            valueX = client_width - offset_width;
 
550
        }
538
551
        if (valueX < 0) {
539
552
            valueX = 0;
540
553
        }
541
 
        if ((valueX >
542
 
             document.body.clientWidth - boundingBox.get('offsetWidth')) &&
543
 
            (document.body.clientWidth > boundingBox.get('offsetWidth'))) {
544
 
            valueX = document.body.clientWidth - boundingBox.get('offsetWidth');
545
 
        }
546
554
        if (valueY < 0) {
547
555
            valueY = 0;
548
556
        }
557
565
    },
558
566
 
559
567
    /**
560
 
     * Return the absolute position of any node
 
568
     * Return the absolute position of any node.
561
569
     *
562
570
     * @private
563
571
     * @method _findPosition
569
577
            do {
570
578
                curleft += obj.get("offsetLeft");
571
579
                curtop += obj.get("offsetTop");
572
 
            } while ((obj = obj.get("offsetParent")));
 
580
                obj = obj.get("offsetParent");
 
581
            } while (Y.Lang.isValue(obj));
573
582
        }
574
583
        return [curleft,curtop];
575
584
    }
635
644
                    return (Y.Lang.isValue(item.value));
636
645
                });
637
646
            }
638
 
            for (var i = 0; i < v.length; i++) {
 
647
            var i;
 
648
            for (i = 0; i < v.length; i++) {
639
649
                if (!Y.Lang.isValue(v[i].value) &&
640
650
                    v[i].name.indexOf('<img') == -1) {
641
651
                    // Only append the icon if the value for this item is