跳到内容
新问题

对这个项目有疑问吗？注册一个免费的GitHub帐户以打开一个问题，并联系其维护者和社区。

注册GitHub

单击“注册GitHub”，表示您同意我们的服务条款和隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了？登录到您的帐户
跳到底部

社交图标：添加显示/隐藏标签的功能 #38152

已合并
将6个提交合并到从 2022年1月27日
已合并

社交图标：添加显示/隐藏标签的功能 #38152

将6个提交合并到从 2022年1月27日

对话

自我
复制链接
成员

@自我 自我 评论2022年1月21日 •
已编辑

说明

社交链接（图标）块允许您添加自定义链接标签。很遗憾，您无法显示这些文本标签，这是在中请求的#31605.本公关修复 #31605通过添加打开和关闭链接标签的功能。启用后，如果未指定自定义标签，则显示服务名称。

这是如何测试的？

  1. 使用运行WordPress 5.9 RC3的任何主题进行测试。
  2. 将社交图标块添加到新页面，并添加一些单独的图标。
  3. 在侧栏中，打开“Show labels”，您现在应该可以看到服务名称。
  4. 在一个图标上，添加一个自定义标签，并查看它是否会更新编辑器中的标签。
  5. 查看前端并确认标签显示正确。

屏幕截图

socal-icon-labels公司

更改的类型

新功能

检查清单：

  • 我的代码已经过测试。
  • 我的代码遵循WordPress代码风格。
  • 我的代码遵循可访问性标准。
  • 我已经用键盘和屏幕阅读器测试了我的更改。
  • 我的代码有适当的内联文档。
  • 如果合适的话，我已经包括了开发人员文档。
  • 我已经更新了所有受此PR中任何重构/重命名影响的React Native文件（请手动搜索所有*.native.js文件需要重命名或删除的术语的文件）。
  • 如果合适的话，我已经更新了相关的模式。

@自我 自我补充[类型]增强功能 改进建议。 [Block]社交 影响社交块-用于显示社交媒体帐户标签2022年1月21日
@自我 自我自我分配的2022年1月21日
@自我 自我将此拉入请求标记为已准备好进行审阅 2022年1月22日16:19
@自我 自我更改了标题社交链接：添加显示/隐藏标签的功能 社交图标：添加显示/隐藏标签的功能 2022年1月22日
马马杜卡
马马杜卡 检验过的 2022年1月22日
查看已审阅的更改
复制链接
成员

@马马杜卡 马马杜卡 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

尼克，谢谢你的努力。

我刚刚做了一个快速的代码检查，但将在周一进行更详细的测试。

自我 和其他补充2提交 2022年1月22日13:10
@自我 @马马杜卡
使用零合并运算符。
42亿4555 
合著人：George Mamadashvili<georgemamadashvili@gmail.com>
@恩迪戈 @马马杜卡
在esc_attr上使用esc_html。
255e113美元 
合著者：George Mamadashvili<georgemamadashvili@gmail.com>
@自我
复制链接
成员 作者

谢谢你的评论@马马杜卡我做了这些更改并添加了评论。🙌

@卡罗来纳州人
复制链接
贡献者

卡罗来纳州人 评论2022年1月25日 •
已编辑

哦，我喜欢这个。
问题：title属性带来了什么价值？
已经尝试从Core中删除标题属性。https://core.trac.wordpress.org/tickt/24766

@贾斯穆森
复制链接
贡献者

真不错！

马马杜卡
马马杜卡 检验过的 2022年1月25日
查看已审阅的更改
复制链接
成员

@马马杜卡 马马杜卡 留下了评论 •
已编辑

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

看起来我们必须为链接添加标签显示标签已启用。

而不是添加另一个条件来显示阿里亚·拉贝尔，也许我们可以用不同的方法？

我想技术#1这个职位应该符合我们的要求-https://www.sarasoueidan.com/blog/accessible-icon-buttons/.

你怎么认为？

@自我
复制链接
成员 作者

问题：title属性带来了什么价值？

好的一点，我会删除它。

我认为技术#1这个职位应该符合我们的要求-https://www.sarasoueidan.com/blog/accessible-icon-buttons/.

嗯，这很有趣。因此，我们使标签始终存在，但如果显示标签已关闭。然后我们可以删除阿里亚·拉贝尔如果我理解正确的话。

@马马杜卡
复制链接
成员

正确，我们可以有条件地添加class=“屏幕阅读器-文本”基于显示标签值。

复写的副本@亚里士多德

@自我
复制链接
成员 作者

我已经做了要求的更改。

我还遇到了其他可能的改进，但尚未实现。我认为svg图标不应该有role=“img”。我读到，如果svg有，那么它也应该有一个阿里亚·拉贝尔.有谁比我更了解这件事吗？

马马杜卡
马马杜卡 批准了这些更改 2022年1月27日
查看已审阅的更改
复制链接
成员

@马马杜卡 马马杜卡 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

这看起来很棒。谢谢，@自我.

你介意为role=“img”?

@自我
复制链接
成员 作者

你介意为role=“img”?

☝️@马马杜卡可以。我还会看看是否还有其他role=“img”在其他可能需要更新的块中。

这个公关很适合合并吗？

@马马杜卡
复制链接
成员

我的理解是我们只需要role=“img”如果SVG不是装饰性的并且传达信息。但我不能百分之百肯定社交图标的案例。

这个公关很适合合并吗？

很好！🚢

@自我 恩迪戈合并提交第7页23960进入之内 WordPress（文字出版社）:大旅行箱 2022年1月27日
@自我 自我删除了 尝试/添加标签-社交图标 分支 2022年1月27日12:43
@github-操作 github-操作 机器人程序将此添加到古腾堡12.6里程碑2022年1月27日
@自我 自我提到这个拉请求 2022年1月27日
关闭
@女性化 女性化提到这个拉请求 2022年4月14日
关闭
5项任务
免费注册 在GitHub上加入此对话.已经有帐户了吗？登录以发表评论
审阅者

@马马杜卡 马马杜卡 Mamaduka批准了这些更改

@阿吉博拉 阿吉博拉 等待ajitbohra要求的审查 阿吉博拉是代码所有者

@姆卡兹 姆卡兹 等待mkaz要求的审查

@贾斯穆森 贾斯穆森 等待jasmussen的审查请求

@亚里士多德 亚里士多德 等待亚里士多德的审查请求

受让人

@自我 自我

标签
[Block]社交 影响社交块-用于显示社交媒体帐户 [类型]增强 改进建议。
项目
还没有
里程碑
古腾堡12.6
开发

成功合并此请求可能会解决这些问题。

社交块：添加选项以显示文本标签
4名参与者
@自我 @卡罗来纳州人 @贾斯穆森 @马马杜卡