Please test on IE7, IE6 and Safari. I'm on linux right now and Firefox 3 and Opera 9.6 looks good.

CommentFileSizeAuthor
#1 facebook-ff3.png2.33 KBpasqualle
#1 facebook-ie7.png3.05 KBpasqualle
#1 facebook-opera.png2.11 KBpasqualle
#1 facebook-safari.png2.28 KBpasqualle
facebook.zip802 byteslelizondo

Comments

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
StatusFileSize
new facebook-safari.png2.28 KB
new facebook-opera.png2.11 KB
new facebook-ie7.png3.05 KB
new facebook-ff3.png2.33 KB

screenshots from the settings page

windows XP

FF3
IE7
Opera 9.6
Safari 4

lelizondo’s picture

lelizondo commented

great, looks good, can anyone test with ie6

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs review » Needs work

IE6 doesn't support multiple class selectors.
#378792-2: Tab styles IE6 CSS Issue

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented

could it be a legal problem with naming something Facebook?

lelizondo’s picture

lelizondo commented

You're right. I'm going to test this in IE6 and then change the name.

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented

I do not know a working solution for IE6 yet..

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Title: Facebook Tab Style » New tab style: Facebook
Status: Needs work » Needs review
Issue tags: +QT tabstyle
ultimateboy’s picture

ultimateboy commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

Without a properly formatted patch, there is nothing to review.

I'm not against adding quicktab styles to the module, but will only consider it with properly formatted patches.

Thanks.