search

heiseonline / shariff

👮 Shariff enables website users to share their favorite content without compromising their privacy.
http://ct.de/-2467514
MIT License
1.4k stars 177 forks source link

Smaller buttons #184

Closed fuzzy76 closed 6 years ago

fuzzy76 commented 8 years ago

Since #146 was closed, I'm re-issuing as a feature request. More specifically, I'd like "textless" buttons like the Drupal module shows.

ghost commented 7 years ago

+1 Yes, smaller buttons would be great!

JoergLueg commented 7 years ago

Hello

How can you display the buttons without the text, so only on icon size. I use the Java version. Can someone help me.

Greetings Jörg

koelling commented 6 years ago

FWIW, I have used the following CSS to remove the text and make the buttons smaller. Didn't notice any problems so far.

.shariff .orientation-horizontal li {
    min-width: auto;
    max-width: 60px;
}

.shariff .orientation-horizontal li .share_text {
    display: none;
}
richard67 commented 6 years ago

Well I think it would be a good feature, but not as a theme like it was suggested in issue #284 .

It should be a new, separate option, like e.g. data-button-size or so. It could be used with any of the themes standard, grey or white then.

Maybe I find time on weekend to implement that and make a pull request here.

richard67 commented 6 years ago

I think I need more time. Maybe someone else wants to do that?

richard67 commented 6 years ago

Guys, I made a pull request (PR) for having small icons with or without the share counters. Please test PR #309 and report the results and your opinion and possible suggestions there with comments. Testing instructions and links to texting packages you can find in the description of that PR.

richard67 commented 6 years ago

I see my solution was released with version 3.0.0 now. Can you check if it fulfills your requirements, and if yes, close this issue?

fuzzy76 commented 6 years ago

If that was directed at me, feel free to close. I don't have time to upgrade the site I'm using Shariff on for a while, so I shouldn't hold up this issue. :)