JS Conversion Beta: Report issues here!

[denise]

If you're participating in the JQuery conversion beta test, report any issues you find here!

Currently, here are the things that have been implemented (and not implemented):

Implemented

Cut expand and collapse
Logging in
Screen/freeze/delete
Control strip injection for non-supporting journals
Quick reply
Thread expander

Not implemented

Contextual hover
Media embed placeholder expansion
Same-page poll submission
Icon browser
Same-page comment tracking


(This means that if you try to do something on the "not implemented" list while you have the beta features turned on, you'll be routed to the non-Javascript/non-inline version. That isn't a bug and you don't have to report it, but you should report anything else that's weird or that doesn't behave the way it used to!)

Comments

Thread expander box positioning

[queenbarwench]

Late to the party as usual, but noticed that on some layouts, the thread expander is not positioning the comment boxes correctly. I'm not sure of the layout name*, but it's this page I'm currently viewing. I've seen it elsewhere, but I was on an old Firefox version and dismissed it at the time.

On to the bug!

Logged in (Firefox 8.0), I see that the top-level comments are correctly spaced (large gap to accommodate 50% of icon above the comment box), but I hit 'expand all' and boxes are close together with the icons of the child comments all overlapping the comment above. The top-level comments remain correctly spaced wrt the comment above. The same thing is seen when I collapse and expand a single thread.

I checked also in Chrome (15.0.874.120 m) logged out (hence no jquery) and the spacing was fine after expanding all. I logged in and expanded, and the spacing was compressed again. Interestingly, the collapse/ expand actions in Chrome place the icons *behind* the comment above.

Anyway, from my 2 second experiment I concluded probably a jquery-related issue, so here, have a bug report :D

(OS is Windows 7, btw. Just in case it still matters these days...)


* Random side note: might be nice to link direct to the theme page from the style credits box if it's a site style, for folks who decide that they like how others have decorated :)

Re: Thread expander box positioning

[fu]

Belatedly -- yup there was a bug, we're putting out a fix for it soon, thank you for your report (<3)

PS. we have a bug open for having the name of the theme in the styles credits box :D I even think someone may have claimed it, or at least is eyeing it with interest!