moutard / SubZoom-Proto1

1 stars 1 forks source link

Images don't appear in a sticker at install #42

Closed HadrienR closed 12 years ago

HadrienR commented 12 years ago

I've noticed in the last version that some jpg images are displayed inside a story but not in the relevant sticker within the story selector.

I've sent an email with the screenshot and that http://lafeverchestnuts.webstarts.com/uploads/chestnut.jpg is the image in my case.

Controller - init - controller.js:23 less: parsed chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/startpage.less successfully. less.js:8 less: css for chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/startpage.less generated in 28ms less.js:8 less: parsed chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/story.less successfully. less.js:8 less: css for chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/story.less generated in 10ms less.js:8 less: parsed chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/loading.less successfully. less.js:8 less: css for chrome-extension://bffhpelggejfbhmanblcndnaikhamehe/media/loading.less generated in 12ms less.js:8 less: css generated in 51ms less.js:8 The database is already up to date engine.js:160 Global store created controller.js:38 Ask if empty controller.js:40 Contoller : already installed - DBSCAN2 controller.js:52 Controller - start controller.js:194 story deleted controller.js:204 [ Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class ] controller.js:218 After dbscan2 controller.js:112 [ Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object ] controller.js:113 1 controller.js:114 [ Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object , Object ] cluster_story.js:13 1 cluster_story.js:14 Class cluster_story.js:73 [ Class ] expand_store.js:13 DB - add stories expand_store.js:14 13The database is already up to date engine.js:160 forFwouts expand_store.js:48 [ Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class ] expand_store.js:49 13The database is already up to date engine.js:160 forFwouts expand_store.js:48 [ Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class ] expand_store.js:49 13The database is already up to date engine.js:160 forFwouts expand_store.js:48 [ Class , Class , Class , Class , Class , Class , Class , Class , Class ] expand_store.js:49 11The database is already up to date engine.js:160 ct - visitItems already loaded sticker.js:212 ct - visitItems corresponding to the clicked story sticker.js:213 [ Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class , Class ] sticker.js:214 parseUrl

HadrienR commented 12 years ago

Actually it does not seem limited to install. in v0.0.2.10, we simply can't get images inside our stickers.

moutard commented 12 years ago

I should have done that before our release, but there are too many informations, and bugs are forgotten. See my email, "stack" bugs.

rkorach commented 12 years ago

That's not a problem because:

I think next week should be focused on these points actually (including the good doc Red wrote about it). And UI polishing when Blue release his updated version (then implement it). And 'PR'.

When this will be done, the product will look like a totally different product. And we'll be happy to work hard on the algorithm right afterwards (and other stuff).

Hope you're all as excited as I am to see this coming! Pink.

On Thu, Aug 9, 2012 at 10:33 PM, Raphael Moutard notifications@github.comwrote:

I should have done that before our release, but there are too many informations, and bugs are forgotten. See my email, "stack" bugs.

— Reply to this email directly or view it on GitHubhttps://github.com/moutard/SubZoom-Proto1/issues/42#issuecomment-7627675.

HadrienR commented 12 years ago

It is a step back from previous versions. This particular feature is already coded but for some reason do not work anymore. I would qualify it as a bug for this reason.

moutard commented 12 years ago

@Red +1 by definition it's a bug

moutard commented 12 years ago

@HadrienR solved for me. See commit 982b99273f47f481b59ac96f2afe11c65d6cd993

HadrienR commented 12 years ago

Works for me too.