selecting "css scrollbox" has no effect (the legal module css refers to non existent identifiers). The T&C's for new user are always displayed in full. (this would appear to be identical to a dev issue cleared over a year ago).
Investigated as far as possible -- theme_legal_display uses prefix/suffix to add code .. this is being ignored. ISTR prefix/suffix only works on textfields, could this be the problem? had to resort to page_alter to get the code running!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | legal-scrollbox-css-setting-2026555-2.patch | 1.48 KB | jojonaloha |
| #1 | legal-scrollbox-css-setting-2026555-1.patch | 691 bytes | jojonaloha |
Comments
Comment #0.0
Infoloko commentedadditional information
Comment #1
jojonaloha commentedAttached patch is a quick fix to this issue. Personally I don't like the idea of calling a theme function in a form alter though.. just leaves a bad taste in the mouth.
Comment #2
jojonaloha commentedUpdated patch also fixes the scroll box when you submit the form and the form doesn't pass validation.
Comment #3
idebr commentedLets leave the cleanup for later and fix the scrollbox first. Patch applies cleanly and fixes the Scrollbox on the user/register form. Thanks Jonathan!
Comment #4
idebr commentedComment #5
ryanbriscall commentedIf you look at the code in the module, it's clearly broken. Confused why it's released in such a state.
These patches (submitted by jojonaloha) fix the issue(s). So, why haven't they merged?
Comment #7
robert castelo commentedThanks jojonaloha, deployed to dev, will be in the next release (soon).