I added this to line 49...
$block.toggleClass('opened');

I needed a color change and had no way to attach the CSS. I imagine there are lots of reasons to want to know the state while in CSS, so I placed it on the most parent item.

CommentFileSizeAuthor
#1 feedback-message-style.patch324 bytesdoublejosh
form-toggle-class.patch385 bytesdoublejosh

Comments

doublejosh’s picture

doublejosh commented
StatusFileSize
new feedback-message-style.patch324 bytes

Another small style helper patch. The message should have the same background, etc as the main form.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Patch (to be ported) » Needs work

Both patches make sense, but we need to combine them. :)

Also, most of Drupal uses a .collapsed class (ie the inverse), so would be good to use it here as well.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes
Status: Needs work » Closed (outdated)

The Drupal 6 version is no longer available, and no longer maintained. So I'm going to close this issue as outdated.

(If you'd like to maintain the Drupal 6 version, please post an issue in the queue)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.