fidrelity / microbuilder

4 stars 0 forks source link

number of graphics found #141

Closed egraether closed 12 years ago

egraether commented 12 years ago

brauche für die library im editor bitte bei jedem request auch die anzahl der treffer in der response. Soll bei 'graphics', 'backgrounds' und dem jeweiligen 'search' funktionieren.

fidrelity commented 12 years ago

gäbe mehrere möglichkeiten, die alle mäßig elegant erscheinen: momentan schick ich dir ja ein array von hashes, wobei jeder hash eine grafik ist.

egraether commented 12 years ago

bin für dritte möglichkeit die ich so verstehe:

response : {
  graphics : [
     // graphics ( bei graphiken 14, bei backgrounds 10)
  ],
  size : 1023 // number of graphics found
}

javascript ändern muss ich sowieso

fidrelity commented 12 years ago

ist auf der branch issue141. check mal obs so passt.

egraether commented 12 years ago

die pagination funktioniert nimmer, aber ich bin mir garnicht sicer ob wir das am server brauchen. Ab wieviele graphiken wäre es ein problem mit der response länge?

fidrelity commented 12 years ago

sollte jetzt alles funktionieren. meine änderungen sind auf der branch issue141.

was mir aufgefallen ist: wenn ich auf medium oder large klick, werden 2 requests abgefeuert, nur bei small nicht. is natürlich unnötiger traffic. sollt ich dafür ein issue aufmachen?

egraether commented 12 years ago

Bei mir funktioniert die pagination noch immer nicht. Hab jetzt meine Änderungen auf issue141 gepusht, momentan werden alle Graphiken von der response angezeigt.

Das mit den 2 requests hab ich gefixt.

fidrelity commented 12 years ago

du hast entweder meine änderungen nicht reingepullt oder falsch gemerged, hab altes zeug in deinem pull drin ghabt.

meine response schaut so aus:

{"size":18, "graphics":[{"id":93,...},{"id":92,...},{"id":91,...},{"id":90,...},{"id":89,...},{"id":88,...},{"id":87,...},{"id":86,...},{"id":85,...},{"id":84,...},{"id":...},{"id":82,...},{"id":80,...},{"id":79,...}] }

insgesamt gibts 18 grafiken, davon schick ich dir 14. was passt da nicht?