search

spazproject / Spaz-Enyo

A version of Spaz targeted for tablet devices
http://getspaz.com
Other
45 stars 10 forks source link

Reduce the number of components in the entry #107

Closed dcousineau closed 12 years ago

dcousineau commented 12 years ago

WAaaay too many, major issues performance wise.

allowHtml: true, layout that way.

dcousineau commented 12 years ago

Column.js line 342: DON'T READ THE DOM

willhoney7 commented 12 years ago

Okay, did this.

Also, the forced column resizing was a result of me having issues getting the scroller to flex naturally in the very beginning. I had to rig a set height.. but it looks like you fixed it :)

dcousineau commented 12 years ago

Well, still working on it. Going to redo what I started @ palm hq, will have a pull request by Sunday (long day today and tomorrow)

willhoney7 commented 12 years ago

Sure, no worries. Runs a lot smoother in browser too.

funkatron commented 12 years ago

It looks like we lost the reposter avatar when we switched to HTML for the body of the entry

willhoney7 commented 12 years ago

Oh we did. Lemme fix that.

willhoney7 commented 12 years ago

Fixed in 95027dc9703404909841

willhoney7 commented 12 years ago

Closing. Can re-open if we need..