This module is quite useful. It occurs to me that the option to implement it as a block - and maybe even a collapsible block - would be a huge advantage. That approach might provide a reasonable workaround for some of the viewability issues that various users note as coming up with certain themes and browsers. (I'm having one at the moment with a Zen subtheme on IE6).

Wish I had the programming chops to pull this off. Happy to follow direction, if you want some collaboration (would be a good way for me to get better, I'm sure). Perhaps it's less complicated than I think?

In the meantime, I've been working on IE6 solution. Since IE6 does not support "fixed" positioning, the only solution I can see is to use "absolute" positioning and place it in a space that might otherwise be occupied by a block.

In any case, thanks for building such a useful module. With luck, perhaps providing feedback is at least a small way of contributing to the effort.

CommentFileSizeAuthor
#2 feedback-DRUPAL-5--2.ie6_.patch557 bytessun

Comments

sun’s picture

IE6:

* html div#block-feedback-form {
  position: absolute;
}
sun’s picture

Title: option to implement as block? » IE6 does not properly support position: fixed
Version: 5.x-2.x-dev » 5.x-2.0
Category: feature » bug
Status: Active » Needs review
StatusFileSize
new557 bytes

The same as patch.

Outputting Feedback in a block is treated in #327823: Prevent duplicate feedback forms when placing feedback block.

kay_v’s picture

cool - I will try this out and report results!

sun’s picture

Status: Needs review » Fixed

Thanks for reporting! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.