The facebook skin sometimes gives you a cutoff version on the bottom dropshadow.

Also, the resize is operating on the wrong element.

Patch fixes both of these.

CommentFileSizeAuthor
#3 facebook skin.JPG56.32 KBandrewsuth
popups.facebook.js_.patch1.02 KBsirkitree

Comments

starbow’s picture

starbow commented

Great, I will test this out. Thanks.

starbow’s picture

starbow commented
Status: Needs review » Fixed

In alpha5

andrewsuth’s picture

andrewsuth commented
StatusFileSize
new facebook skin.JPG56.32 KB

Without controlling the size of the facebook skin, I get a slightly odd looking display of the node using Chrome. See the attached image from Chrome output.

Using Firefox, it doesn't display at all but I have no problems with any of the other skins.

Does anybody else get these results?

Status: Fixed » Closed (fixed)

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

andrewsuth’s picture

andrewsuth commented
Status: Closed (fixed) » Active

I'll update this to active as I think the issues I found should be looked into.

pribeh’s picture

pribeh commented

On two macs here, get the same issue in Safari, and in firefox it doesn't display at all for me.

toemaz’s picture

toemaz commented

@andrewsuth did you ever manage to fix the facebook popup style on Chrome? I have the same skin issue and I'm looking how to solve it.

toemaz’s picture

toemaz commented
Status: Active » Closed (fixed)

@andrewsuth solution to the css problem:

td.popups-t, td.popups-b {
  height: 2px;
}