many fix for different browsers with @float:right@ and thin designs (Userlist displayed outta *body*)
userlist at the top of chatroom
and a few little css fixes

in attachement - fully worked version (I can provide patch for cvs if You want)

CommentFileSizeAuthor
#7 css.patch2.07 KBAnonymous (not verified)
#6 css.png176.46 KBAnonymous (not verified)
#5 css.patch1.84 KBAnonymous (not verified)
chatroom.zip113.23 KBpodarok
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Version: 6.x-1.0-beta10 » 6.x-2.x-dev

thanks for the zip, yes, please provide a patch against the 6.x-2.x-dev branch.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

thanks for the zip, yes, please provide a patch.

that looks like the 5.x version - is that what you intended?

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Version: 6.x-2.x-dev » 6.x-1.0-beta10

nope - this is 6.x version
patch for v.2 - hm... in my configuration it was very unstable so I used v.1

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Version: 6.x-1.0-beta10 » 6.x-2.x-dev
Status: Patch (to be ported) » Active

yes, my mistake.

please don't provide a patch against 6.x-1, that code is going the way of the dodo's as soon as possible, please provide the patch against the latest 6.x-2 CVS.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Component: Code » User interface
Assigned: Unassigned »
Priority: Critical » Normal
Status: Active » Needs review
FileSize
css.patch1.84 KB

here's the patch from podarok.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Needs work
FileSize
css.png176.46 KB

ok, this needs work, because the layout has changed in the 6.x-2 branch.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
css.patch2.07 KB

updated patch, still needs work.

not sure how this will interact with imagecache and user-picture support in the list. also, wondering if this has to be configurable.

right now, i think the whole thing can be overridden just in the theme layer.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented

ow
I see
will try to fit this into 2.0 version soon. (Mostly css fixes needed)

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Assigned: » Unassigned