Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various Floating Social Buttons improvements #1635

Merged
merged 5 commits into from
Mar 1, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ This release contains bug fixes for Largo 0.6.

### Fixes

- Increases contrast of the floating social button icons against the background, to improve accessibility. [Pull request #1635](https:/INN/largo/pull/1635/).
- Fixes issue where floating social buttons were not clickable, because of z-index ordering. [Pull request #1635](https:/INN/largo/pull/1635/) for [issue #1576](https:/INN/largo/issues/1576).
- Fixes links in social media buttons not respecting the blog's character set. [Pull request #1635](https:/INN/largo/pull/1635/) for [issue #1283](https:/INN/largo/issues/1283).
- Function `largo_post_social_links` now respects the blog character set option. [Pull request #1635](https:/INN/largo/pull/1635/) for [issue #1283](https:/INN/largo/issues/1283).
- Fixes PHP notices in class `Bootstrap_Walker_Nav_Menu`. [Pull request #1624](https:/INN/largo/pull/1624) and [#1625](https:/INN/largo/pull/1625) for [issue #1623](https:/INN/largo/issues/1623) as part of [issue #1492](https:/INN/largo/issues/1492).
- Fixes a regression in the behavior of the Largo Follow widget. [Pull request #1600](https:/INN/largo/pull/1600) for [issue #1599](https:/INN/largo/issues/1599).
- Fixes issue where post excerpt and featured media were not being used for open graph tags on post types that are `is_singular()` but not `is_single()`. [Pull request #1604)(https:/INN/largo/pull/1604) for [issue #1602](https:/INN/largo/issues/1602).
Expand Down
24 changes: 5 additions & 19 deletions css/editor-style.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion css/editor-style.min.css

Large diffs are not rendered by default.

24 changes: 5 additions & 19 deletions css/style.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion css/style.min.css

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions inc/post-social.php
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ function largo_post_social_links( $echo = true ) {
$output .= sprintf(
$twitter_share,
// Yes, rawurlencode. Otherwise, the link will break. Use html_entity_decode to handle wordpress saving smart quotes as Ӓ entities.
rawurlencode( html_entity_decode( get_the_title(), ENT_QUOTES, "UTF-8" ) ),
rawurlencode( html_entity_decode( get_the_title(), ENT_QUOTES, get_option('blog_charset') ) ),
rawurlencode( get_permalink() ),
$via,
__( 'Tweet', 'largo' )
Expand All @@ -91,8 +91,8 @@ function largo_post_social_links( $echo = true ) {
$output .= sprintf(
'<span data-service="email" class="email share-button"><a href="mailto:?subject=%2$s&body=%3$s%0D%0A%4$s" target="_blank"><i class="icon-mail"></i> <span class="hidden-phone">%1$s</span></a></span>',
esc_attr( __( 'Email', 'largo' ) ),
rawurlencode( html_entity_decode( get_the_title(), ENT_QUOTES, "UTF-8" ) ), // subject
rawurlencode( html_entity_decode( strip_tags( get_the_excerpt() ), ENT_QUOTES, "UTF-8" ) ), // description
rawurlencode( html_entity_decode( get_the_title(), ENT_QUOTES, get_option( 'blog_charset' ) ) ), // subject
rawurlencode( html_entity_decode( strip_tags( get_the_excerpt() ), ENT_QUOTES, get_option( 'blog_charset' ) ) ), // description
rawurlencode( html_entity_decode( get_the_permalink() ) ) // url
);
}
Expand Down
22 changes: 4 additions & 18 deletions less/inc/single.less
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ h2.subtitle {
#floating-social-buttons {
transition: opacity 0.2s;
border: 1px solid @grayLighter;
z-index: 0;
z-index: 1;

.hidden-phone {
display: none;
Expand All @@ -399,7 +399,7 @@ h2.subtitle {
cursor: pointer;
}
& > a {
color: @grayLighter;
color: @gray;
display: block;
padding: 18px;
min-width: 28px;
Expand All @@ -415,31 +415,17 @@ h2.subtitle {
.facebook,
.print,
.email {
color: @grayLighter;
color: @gray;
background-color: @white;
}
.more-social-links {
background-color: @white;
color: @grayLighter;
color: @gray;
a.popped,
&:hover {
background-color: @orange;
color: @white;
}
.popover {
top: 68px;
font-size: 14px;
i {
margin-right: 0.25em;
}
a {
color: @gray;
&:hover {
color: @grayDark;
text-decoration: underline;
}
}
}
}
.social-btn {
padding: 0.5em;
Expand Down