author | Michael Krelin <hacker@klever.net> | 2006-09-30 18:27:38 (UTC) |
---|---|---|
committer | Michael Krelin <hacker@klever.net> | 2006-09-30 18:27:38 (UTC) |
commit | 01fbc99cb713a0c4656f33949b2949bf7403656b (patch) (side-by-side diff) | |
tree | 2ee9c07db4c6c7c329f97e56615672ca2fc3e462 | |
parent | 4f5e775ff24baf34b3425e93bfeac4a436b4b525 (diff) | |
download | fireflix-01fbc99cb713a0c4656f33949b2949bf7403656b.zip fireflix-01fbc99cb713a0c4656f33949b2949bf7403656b.tar.gz fireflix-01fbc99cb713a0c4656f33949b2949bf7403656b.tar.bz2 |
more code beauty/performance enhancements
git-svn-id: http://svn.klever.net/kin/fireflix/trunk@171 fe716a7a-6dde-0310-88d9-d003556173a8
-rw-r--r-- | content/photoset-props.js | 38 | ||||
-rw-r--r-- | content/photoset-props.xul | 1 |
2 files changed, 18 insertions, 21 deletions
diff --git a/content/photoset-props.js b/content/photoset-props.js index 43dc1b9..7c816be 100644 --- a/content/photoset-props.js +++ b/content/photoset-props.js @@ -1,81 +1,77 @@ var psetprops = { fireflix: null, photoset: null, pripic: null, - settitle: null, setdesc: null, - primarypic: null, photos: new Array(), init: function() { this.fireflix = window.arguments[0]; this.photoset = window.arguments[1]; - this.settitle = document.getElementById('set_title'); - this.settitle.value = this.photoset.title; - this.setdesc = document.getElementById('set_desc'); - this.setdesc.value = this.photoset.description; - this.primarypic = document.getElementById('primary_picture'); - this.primarypic.src = + pull_elements(this,document,[ + 'set_title','set_desc','primary_picture', + 'primary_picture_list' + ]); + this.set_title.value = this.photoset.title; + this.set_desc.value = this.photoset.description; + this.primary_picture.src = this.fireflix.flickr.get_image_url( this.photoset, 't' ); - this.primarypic.hidden = false; - this.picslist = document.getElementById('primary_picture_list'); + this.primary_picture.hidden = false; var _this = this; this.fireflix.flickr.api_call( { method: 'flickr.photosets.getPhotos', auth_token: 'default', photoset_id: this.photoset.id }, function(xr) { var x = xr.responseXML; var xp = x.evaluate( '/rsp/photoset/photo', x, null, XPathResult.ORDERED_NODE_ITERATOR_TYPE, null ); - _this.picslist.removeAllItems(); _this.photos= new Array(); + _this.primary_picture_list.removeAllItems(); _this.photos= new Array(); var n; while(n=xp.iterateNext()) { _this.photos.push( { id: n.getAttribute('id'), secret: n.getAttribute('secret'), server: n.getAttribute('server') } ); - var ni = _this.picslist.appendItem( + var ni = _this.primary_picture_list.appendItem( n.getAttribute('title'), _this.photos.length-1 ); ni.setAttribute('command','cmd_select_picture'); if(n.getAttribute('isprimary')==1) { - _this.picslist.selectedItem = ni; + _this.primary_picture_list.selectedItem = ni; _this.pripic = _this.photos[_this.photos.length-1]; } } - _this.picslist.hidden = false; + _this.primary_picture_list.hidden = false; }, function() { } ); }, on_select_picture: function(ev) { var epic = ev.explicitOriginalTarget; - this.picslist.selectedItem = epic; - var pic = this.photos[this.picslist.selectedItem.value]; + this.primary_picture_list.selectedItem = epic; + var pic = this.photos[this.primary_picture_list.selectedItem.value]; this.pripic = pic; - this.primarypic.src = + this.primary_picture.src = this.fireflix.flickr.get_photo_url( pic.server, pic.id, pic.secret, 't' ); }, on_accept: function() { - this.photoset.title = - document.getElementById('set_title').value; - this.photoset.description = - document.getElementById('set_desc').value; + this.photoset.title = this.set_title.value; + this.photoset.description = this.set_desc.value; this.photoset.server = this.pripic.server; this.photoset.primary = this.pripic.id; this.photoset.secret = this.pripic.secret; this.photoset.dirty = true; return; } }; diff --git a/content/photoset-props.xul b/content/photoset-props.xul index e8f6d13..2efa9e6 100644 --- a/content/photoset-props.xul +++ b/content/photoset-props.xul @@ -3,24 +3,25 @@ <?xsml-stylesheet href="fireflix.css" type="text/css"?> <!DOCTYPE dialog SYSTEM "chrome://fireflix/locale/fireflix.dtd"> <dialog xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" id="photoset_props" buttons="accept,cancel" defaultbutton="accept" title="&photosetprops.title;" onload="psetprops.init()" ondialogaccept="psetprops.on_accept()" > + <script src="util.js" type="application/x-javascript"/> <script src="photoset-props.js" type="application/x-javascript"/> <commandset> <command id="cmd_select_picture" oncommand="psetprops.on_select_picture(event)"/> </commandset> <hbox class="wholething"> <vbox> <menulist id="primary_picture_list" hidden="true" sizetopopup="always"/> <hbox pack="center"> <box width="100" pack="center"> |