
从截图中可以看出,除了“Billing”之外,所有其他按钮都只包含一个 SVG 图标而没有文字,但“Billing”却有,这导致了视觉上的不匹配。
移除文字是否更好?我认为图标已经足够了。
相关的 PR 在这里。
1 个赞
kelv
(Kelvin T)
2
太好了,感谢你提出这个问题,@small-lovely-cat!我明白在你的 UI 上只有一个图标伴随着文字显示确实会显得有些突兀。
从我这边来看,这很可能是因为视口变得太窄了,然后我们应用了一些样式来阻止这些文本元素显示在页面上:
https://github.com/discourse/discourse/blob/06f42e0206af62b1f87501bfa799cd6d79d726a6/app/assets/stylesheets/common/base/new-user.scss#L37-L50
你在 PR 中更新的 Billing 组件缺少了国际化文本周围的 span 元素。我们应该添加这个元素,这应该足以处理这种情况。否则,我们不希望完全移除文本,否则在更宽的视口中,该图标会缺少文字,而所有其他图标都有伴随的文字。
4 个赞
好的,感谢您的快速回复~
已修复的 PR 版本在此,旧版本已关闭(github 未能让我合并它 >﹏<)。
3 个赞
kelv
(Kelvin T)
5
再次感谢 @small-lovely-cat,我已经批准并合并了 PR 
3 个赞