跳到内容
新问题

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

注册GitHub

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

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

FormTokenField:添加帮助文本 #15469

已合并
将3个提交合并到从
2019年5月16日
已合并

FormTokenField:添加帮助文本 #15469

将3个提交合并到从
2019年5月16日

对话

马雷赫拉贝
复制链接
贡献者

@马雷赫拉贝 马雷赫拉贝 评论2019年5月6日 •
已编辑
加载

描述

修复 #15355

我已经使消息可见，并添加了回车键。它的样式与侧边栏中的其他帮助文本相同。

该组件允许开发人员选择使用空格键作为分隔符。我已经考虑过了，并根据配置制作了两个版本的帮助文本。

这是如何测试的？

  • 新建帖子
  • 在侧栏中，打开“标签”面板并浏览新消息

侧栏术语选择器（标记、类别）是在核心中使用此组件的唯一位置。

屏幕截图

标准用法 启用空格分隔符
截图2019-05-06第17 49 59页 截图2019-05-06第17 51 16页

@马雷赫拉贝 马雷赫拉贝语补充[类型]增强功能 改进建议。 [焦点]可访问性（a11y） 影响可访问性并需要相应审查的更改（例如标记更改）。标签2019年5月6日
无尾目
无尾目 检验过的 2019年5月6日
查看已审阅的更改
包/组件/src/form-token-field/style.scss
margin-bottom:$grid-size-small；
}

.components-form-token字段__help{
显示：块；
字体样式：斜体；
复制链接
贡献者

选择隐藏此评论的原因

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

我想得到一些关于斜体风格的设计反馈。就我个人而言，我会避免使用它，因为有些用户觉得它很难阅读。

复制链接
贡献者 作者

选择隐藏此评论的原因

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

我根据的帮助文本的样式将其设置为斜体基本控制它被用作大多数其他控件的基础。如果类似的帮助文本缺少斜体字，则与其他文本不一致。仅举几个例子：

截图2019-05-06，21 18 19

截图2019-05-06，21 18 29

截图2019-05-06，21 19 10

我同意斜体字的问题，但我认为我们应该在全球范围内单独解决BaseControl的所有现有用法。你觉得把这份公关写成斜体字，然后在全球范围内展开讨论以取代所有帮助文本怎么样？一旦我们达成解决方案，我很乐意进行公关。

复制链接
贡献者

选择隐藏此评论的原因

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

是的，我认为斜体和无斜体可以单独讨论-这张公关只是反映了现有的风格.组件-base-control__help.

（通过快速点击，似乎永久链接帮助文本是这类文字中唯一没有斜体的，所以我们应该以这种或那种方式做出决定）。

复制链接
贡献者

选择隐藏此评论的原因

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

意大利语用于整句话时，对于某种阅读障碍来说是很糟糕的。我投票赞成不添加新的斜体。此外，计划创建相关的GitHub问题和Trac票证。谢谢您。

复制链接
贡献者

选择隐藏此评论的原因

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

酷-如果我们可以用其他东西替换斜体，我很乐意在全球范围内去掉它。我只是想确保帮助文本与表单字段标签的显示方式不同，以便用户能够区分它们。但除了使文本倾斜之外，还有其他方法可以做到这一点。我会留意这张票，并在那里尝试一些想法。🙂

复制链接
贡献者

@阿费西娅 无尾目 2019年5月8日 •
已编辑
加载

选择隐藏此评论的原因

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

除了使文本倾斜之外，还有其他方法可以做到这一点

绝对同意。在我死之前的一天，我想在WordPress中看到一个印刷系统🙂

复制链接
贡献者

选择隐藏此评论的原因

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

我已经打开了#15683跟踪不一致的帮助文本，并考虑在其中使用斜体文本。我们将在古腾堡全球解决这一问题。🙂

同时，这不应该阻止这一公关，因为它没有添加任何新的风格-只是复制现有的基本控制帮助文本样式。

@阿费西娅 无尾目补充这个需要设计反馈 需要一般设计反馈。标签2019年5月6日
无尾目
无尾目 请求的更改 2019年5月6日
查看已审阅的更改
复制链接
贡献者

@阿费西娅 无尾目 留下了评论

选择隐藏此评论的原因

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

谢谢@马雷赫拉贝，我留下了一些评论。

回复：斜体文字，我想得到一些设计反馈。为了更好的可读性，我想尽量避免使用它。此外，在我看来，古腾堡并没有在输入字段下面使用斜体来表示这种提示，例如：

2019-05-06第18 24 23页截图

/抄送@喀斯特化的

大戟
大戟 检验过的 2019年5月6日
查看已审阅的更改
复制链接
成员

@大戟 大戟 留下了评论

选择隐藏此评论的原因

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

此组件似乎具有BaseControl组件提供的标签和帮助文本功能：https://github.com/WordPress/gutenberg/blob/1e40b281d1db725e512e383b5e00f3b59e333d5a/packages/components/src/base-control/README.md

是否可以在这里使用基本控件，避免标签和帮助文本自定义样式的需要，并提高代码的重用性？

packages/components/src/form-token-field/index.js
<div id={`components-form-token-suggements-howto-${instanceId}`}className=“screen-reader-text”>
｛__（'用逗号分隔'）｝
</div>
<p id={`components-form-token-suggements-howto-${instanceId}`}className=“components-form-token-field__help”>
复制链接
成员

选择隐藏此评论的原因

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

好像id组件-form-token-suggements-howto-${instanceId}没有在任何地方被引用，我们可以删除id吗？

复制链接
贡献者 作者

选择隐藏此评论的原因

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

此处引用https://github.com/WordPress/gutenberg/blob/master/packages/components/src/form-token-field/token-input.js#L50

@马雷赫拉贝
复制链接
贡献者 作者

我实际上已经尝试过使用BaseControl，但它不太容易使用，因为此组件在包装元素上附加了许多附加属性，而这是不受支持的。

@凯勒 凯勒提到这个拉请求 2019年5月16日
正常开放
凯勒
凯勒 建议的更改 2019年5月16日
查看已审阅的更改
复制链接
贡献者

@凯勒 凯勒 留下了评论 •
已编辑
加载

选择隐藏此评论的原因

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

@马雷赫拉贝这看起来很棒。只是一个小小的设计更改，然后就可以在我这边合并了。目前，第一版标签上写着：

使用逗号或Enter键分隔

我们应该在末尾添加一个句点，以便与其他帮助文本对齐。不幸的是，这很尴尬地将“key.”一词推到了桌面屏幕上的一行。😄

为了避免这种情况，让我们将“using”替换为单词“with”：

屏幕截图2019-05-16上午10 56 07

我们还可以通过第二个标签版本进行更改，其中包括提及空格：

屏幕截图2019-05-16下午1 55 31

@马雷赫拉贝
复制链接
贡献者 作者

真是个好主意！😀消息更新，代码更新。

凯勒
凯勒 批准了这些更改 2019年5月16日
查看已审阅的更改
复制链接
贡献者

@凯勒 凯勒 留下了评论

选择隐藏此评论的原因

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

从设计角度来看，看起来很棒。👍

@凯勒
复制链接
贡献者

回复：斜体文字，我想得到一些设计反馈。为了更好的可读性，我想尽量避免使用它。

@阿费西娅：现在我们有了#15683在全球范围内追踪这一点的过程中，这一公关是否已准备好为您服务？

@阿费西娅
复制链接
贡献者

👍 从我这里。

@马雷赫拉贝
复制链接
贡献者 作者

谢谢，@阿费西娅。你也能更新一下评论吗？Github仍然阻止我进行合并，因为您之前要求进行更改。

合并被阻止
一旦请求的更改得到解决，就可以自动执行合并。

@马雷赫拉贝 马雷赫拉贝合并提交b193b09号进入之内 主人 2019年5月16日
@马雷赫拉贝语 马雷赫拉贝删除了 修复/标记字段帮助文本 分支 2019年5月16日21:01
免费注册 在GitHub上加入此对话.已经有账户了吗？登录以发表评论
审阅者

@大戟 大戟 jorgefilipecosta留下了评论

@凯勒 凯勒 kjell批准了这些更改

@发烧 无尾目 非ercia批准了这些更改

@阿杜斯 阿杜斯 等待aduth的审查请求

@阿吉博拉 阿吉博拉 等待ajitbohra要求的审查

@克里斯万帕顿 克里斯万帕顿 等待chrisvanpatten的审查请求

@吉奥罗牌手表 吉奥罗牌手表 等待gziolo的审查请求

@吵闹声 吵闹声 等待Noissocks要求的审查

@塔尔丹 塔尔丹 等待塔尔丹的审查请求

@你知道吗里亚德 你知道吗里亚德 正在等待youknowriad要求的审查

@喀斯特化的 喀斯特化的 等待karmatosed要求的审查

受让人

@马雷赫拉贝 马雷赫拉贝

标签
[焦点]可访问性（a11y） 影响可访问性并需要相应审查的更改（例如标记更改）。 需要设计反馈 需要一般设计反馈。 [类型]增强功能 改进的建议。
项目
还没有
里程碑
5.8（古腾堡）
开发

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

有用的内容被不必要地隐藏
4名参与者
@马雷赫拉贝 @凯勒 @阿费西娅 @大戟