search

victorjonsson / Arlima

Article List Manager - Wordpress plugin suitable for online newspapers that's in need of a fully customizable front page
28 stars 16 forks source link

Preview width not updating when moving teasers between lists. #28

Closed chredd closed 10 years ago

chredd commented 10 years ago

When setting preview widths of teasers (via the filter arlima_tmpl_width), the preview won't update to correct width when moving the teasers between lists.

Watch this clip to see it in action: https://www.youtube.com/watch?v=59aqV2Ootj4&feature=youtu.be

The list Home has a width of 550px and Mega has 1000px.

chredd commented 10 years ago

@victorjonsson @znoid @peterberglund Are you having the same problems at VK? Maybe you don't set the preview width via the filter?

victorjonsson commented 10 years ago

We also use the filter. This is a bug, we'll look into it.

chredd commented 10 years ago

Awesome :+1:

victorjonsson commented 10 years ago

This should now be fixed!

chredd commented 10 years ago

Awesome! I'll try it out as soon as possible.

chredd commented 10 years ago

@victorjonsson When moving a teaser with children, saving while fail with this error in the javascript console: "Uncaught TypeError: Cannot read property 'children' of undefined"

Saving teasers without childrens is working just fine.

victorjonsson commented 10 years ago

I'm having problems reproducing this bug. Could you please give me the stack trace? or which file and on what line number the error occurs.

chredd commented 10 years ago

Problem seems to be only in the javascript: screen shot 2014-04-01 at 08 31 14

I'll create a video where I recreate the issue and get back to you.

chredd commented 10 years ago

https://www.youtube.com/watch?v=nGpWKLkf7T0&feature=youtu.be Ping @victorjonsson

chredd commented 10 years ago

Update: I am unable to recreate this bug on my other dev machine. I'll investigate further and get back to you with findings.

chredd commented 10 years ago

After removing all of my old teasers and creating new ones, this no longer occurs. Closing issue.