When both Service Links and Forward are enabled, Share would be much easier to use if it styled the "Social Web" and "E-mail" links as tabs. I would also like to be able to rename the "Social Web" and "E-mail" text to something else in the Share settings.

Very nice module. Thanks!

Comments

snowball43’s picture

I'm a little confused by 'Share would be much easier to use if it styled the "Social Web" and "E-mail" links as tabs.', share does this if in the Forward settings form type is set to share_form.

The ability to change the texts "Social Web" and "E-mail" is a good idea. I think they are wrapped in the t() function but I could be wrong. Having an option say on a Share settings page to set those probably is the best option that I can see.

Also, Forward form type 'link' does work with Share, it just puts the link in with the other links with no tabs. If form type is 'share_form', then the tabs are applied. Didn't think it made much sense to create a tab just for link, and if you turn off 'group' on the Forward settings page, the link is displayed next to the 'share' link.

Christefano-oldaccount’s picture

StatusFileSize
new13.66 KB

Attached is a cropped screenshot of what I see in Firefox. They remind me of what graphics looked like in MS-DOS, not Drupal $tabs or jQuery tabs.

Can you separate the styling to its own stylesheet?

Christefano-oldaccount’s picture

StatusFileSize
new12.96 KB

This is how it looks in Safari.

snowball43’s picture

Interesting, it must have to do with the theme.

http://beta.written.org/betterway/ is how it looks with the theme that the module was themed in. I'll theme it under Garland when I get a chance. Share module does use the theme function so you can theme the share popup. That demo site also has added jquery effects not found in the module.

Christefano-oldaccount’s picture

Category: feature » bug

The jQuery effects are nice. Do you need the jQuery Update module for that to work?

Yes, Garland is the theme I was using when I took those screenshots.

niklp’s picture

Looks freaking awesome on the written.org page - care to share the extras? I suppose the jQuery is very easy, but I haven't learnt it yet :)

greenskin’s picture

The style should be better now.

You can also turn on the effects and set the tab names via Share settings page.

greenskin’s picture

Status: Active » Fixed
Christefano-oldaccount’s picture

This is just about perfect now. Thanks!

greenskin’s picture

I haven't tested the module without the jQeury Update module. If you want to test this and let me know, that would be great. I just make it a habit to install the jQuery Update module.

greenskin’s picture

Status: Fixed » Closed (fixed)