Skip to content

Fix issues with Glowing Follow#885

Merged
homu merged 1 commit intonew-xkit:masterfrom
hobinjk:glowing-follow-broken
Dec 24, 2015
Merged

Fix issues with Glowing Follow#885
homu merged 1 commit intonew-xkit:masterfrom
hobinjk:glowing-follow-broken

Conversation

@hobinjk
Copy link

@hobinjk hobinjk commented Dec 23, 2015

Improves Glowing Follow's short description. Fixes iframe URL detection
for Tumblr update. Updates class used for follow button detection.

Fixes #844.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would prefer this to be a new XKit.iframe function. (because I think some blogs still might not have the new controls)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the selector would be .btn.follow,.tx-button.follow-button then

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, fixed.

Improves Glowing Follow's short description. Fixes iframe URL detection
for Tumblr update. Updates class used for follow button detection,
adding XKit.iframe.follow_button.

Fixes new-xkit#844.
@hobinjk hobinjk force-pushed the glowing-follow-broken branch from 54a9188 to 5cb778d Compare December 24, 2015 00:35
@nightpool
Copy link
Member

doesn't work on blogs with dark backgrounds because it's just varying the opacity. mind filing a followup issue?

@homu r+

@homu
Copy link

homu commented Dec 24, 2015

📌 Commit 5cb778d has been approved by nightpool

@homu
Copy link

homu commented Dec 24, 2015

⚡ Test exempted - status

@homu homu merged commit 5cb778d into new-xkit:master Dec 24, 2015
homu added a commit that referenced this pull request Dec 24, 2015
Fix issues with Glowing Follow

Improves Glowing Follow's short description. Fixes iframe URL detection
for Tumblr update. Updates class used for follow button detection.

Fixes #844.
@hobinjk
Copy link
Author

hobinjk commented Dec 24, 2015

@nightpool I filed #886 as a follow-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants