I found it a little confusing to have three options for the location on the node settings because both the 'Full View' and 'Node links' are options that both can be selected though they cannot be displayed at the same time.

I think it would make more sense to do it like the similar Facebook Social module (see attached picture) with the Full View / Links and Teaser as separate options. I'm happy to do a patch if you agree.

CommentFileSizeAuthor
Screenshot-1.png12.46 KBacouch

Comments

stillcut’s picture

stillcut commented

First of all, thank you for adding the node location.
It's really a step forward.
Next step will be to implement a more sensible location settings.
By example, we can't make an "align to left" in the node location.

Below, you can see screenshots of what I got and what i would like.

http://www.fileden.com/files/2007/9/22/1449796//WhatWeGot.png

http://www.fileden.com/files/2007/9/22/1449796//WhatWeWant.png

stillcut’s picture

stillcut commented
Version: 6.x-1.x-dev » 6.x-1.4
dddave’s picture

dddave commented

Dude, my anti virus software goes havoc on your site. You might want to look into this.

stillcut’s picture

stillcut commented

Fileden.com is not my site but a wellknown online storage site.
I think that you should look for an antivirus with less false positive alerts.

acouch’s picture

acouch commented
Version: 6.x-1.4 » 6.x-1.x-dev

@stillcut The alignment is something that, it is up to the module maintainer, but would be handled by css. This ticket is addressing the logic for placing in the content vs. links on the node and whether or not it shows up in teasers.

timdiacon’s picture

timdiacon commented

off post topic as mentioned but... remove

.tweetbutton {
float:right;
}

From the tweetbutton stylesheet and you'll get what you want.

chia’s picture

chia commented
Status: Active » Fixed

removed the .css file
updated in dev branch

Status: Fixed » Closed (fixed)

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