Opened 9 years ago
Closed 9 years ago
#30059 closed defect (bug) (fixed)
Twenty Fifteen: Social Links Menu and Skype URIs
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.1 | Priority: | normal |
Severity: | normal | Version: | 4.1 |
Component: | Bundled Theme | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
In the main stylesheet there's one rule to support Skype links in the Social Menu, btw they won't work unless you add a "skype" protocol in the kses allowed protocols. Without that, you can't save a link with a "URL" value like, for example, "skype:echo123?call".
Proposed patch uses the "kses_allowed_protocols" filter and also fixes the doubled ":before:before" in the stylesheet.
Attachments (2)
Change History (10)
#3
follow-up:
↓ 4
@
9 years ago
no objections to remove it, if there are security concerns, otherwise I don't see so much difference between "skype:" and "mailto:" links (which are supported)
#4
in reply to:
↑ 3
@
9 years ago
Replying to afercia:
no objections to remove it, if there are security concerns, otherwise I don't see so much difference between "skype:" and "mailto:" links (which are supported)
OK, let's remove Skype. Do you want to cook a quick patch?
I'm kind of ok to remove Skype support rather than adding a Skype protocol.