For a thin designs chatroom are too wide, so

podarok - November 10, 2009 - 11:34
Project:Chat Room
Version:6.x-2.x-dev
Component:User interface
Category:bug report
Priority:normal
Assigned:justinrandell
Status:needs work
Description

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)

AttachmentSize
chatroom.zip113.23 KB

#1

justinrandell - November 10, 2009 - 22:53
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.

#2

justinrandell - November 10, 2009 - 22:54

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

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

#3

podarok - November 11, 2009 - 09:41
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

#4

justinrandell - November 11, 2009 - 09:46
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.

#5

justinrandell - November 11, 2009 - 11:07
Component:Code» User interface
Priority:critical» normal
Assigned to:Anonymous» justinrandell
Status:active» needs review

here's the patch from podarok.

AttachmentSize
css.patch 1.84 KB

#6

justinrandell - November 11, 2009 - 11:13
Status:needs review» needs work

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

AttachmentSize
css.png 176.46 KB

#7

justinrandell - November 11, 2009 - 11:54

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.

AttachmentSize
css.patch 2.07 KB

#8

podarok - November 11, 2009 - 13:46

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

 
 

Drupal is a registered trademark of Dries Buytaert.