跳到内容
新发行

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

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

已经在GitHub上了?登录到您的帐户

从_colors.scss中取消许多颜色 #23454

已合并
将9个提交合并到 2020年6月29日
已合并

从_colors.scss中取消许多颜色 #23454

将9个提交合并到 2020年6月29日

对话

贾斯穆森
复制链接
贡献者

此PR从颜色变量样式表中删除了许多颜色。

为什么?因为当我们减少可用颜色的数量时,它会增加焦点、清晰度和简单性,并有助于收紧编辑器的视觉表达。你真的不应该需要10种深浅的灰色,甚至可以选择更多。

这也有利于最近合并的专色主题CSS变量,也有利于核心项目围绕管理颜色模式所做的任何未来努力。

测试时,应该很少有有意义的视觉变化,可能只有以下几个变化:

屏幕截图2020-06-25,11 26 53

此占位符文本的对比度稍高。

dropzone现在是不透明的,并且主题色很好:

屏幕截图2020-06-25,11 36 00

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

值得一提的是,我想继这一次之后,推出更多的PR,淘汰剩下的颜色,再加上另一个PR,增加一些可供选择的颜色。

我抄送了你,@伊姆托马斯主教,因为在做这件事的时候,我没有接触任何本土的东西。我也很高兴能接触到这一点,但我想先听听你的想法。

@github-操作
复制链接

github-操作 机器人程序 评论2020年6月25日

尺寸变化:+44 B(0%)

总大小:1.13 MB

文件名 大小 更改
构建/块编辑器/index.js 109千字节 +58亿(0%)
build/block-editor/style-rtl.css 10.7千字节 -3亿(0%)
构建/块编辑器/样式.css 10.7千字节 -3亿(0%)
构建/块库/editor-rtl.css 7.58千字节 -20亿(0%)
build/block-library/editor.css 7.58千字节 -20亿(0%)
build/block-library/style-rtl.css 8.04千字节 +5亿(0%)
build/block-library/style.css 8.05千字节 +5亿(0%)
构建/components/style-rtl.css 15.9千字节 +6亿(0%)
build/components/style.css 15.9千字节 +5亿(0%)
构建/编辑器/样式-rtl.css 3.84千字节 -3亿(0%)
构建/编辑器/样式.css 3.85千字节 -2亿(0%)
build/nux/style-rtl.css 671亿 +80亿(1%)
build/nux/style.css 668亿 +80亿(1%)
ℹ️查看未更改
文件名 大小 更改
构建/a11y/index.js 1.14千字节 0亿
构建/注释/index.js 3.62千字节 0亿
构建/api-fetch/index.js 3.4千字节 0亿
构建/auto/index.js 2.82千字节 0亿
构建/blob/index.js 620亿 0亿
build/block-directory/index.js 7.39千字节 0亿
构建/块目录/样式-rtl.css 941亿 0亿
构建/块目录/样式.css 942亿 0亿
build/block-library/index.js 130千字节 0亿
构建/块库/theme-rtl.css 730亿 0亿
构建/块-库/theme.css 732亿 0亿
build/block-serialization-default-parser/index.js 1.88千字节 0亿
build/block-serialization-specifi-parser/index.js 3.1千字节 0亿
build/blocks/index.js 48.2千字节 0亿
build/components/index.js构建/组件/索引 198千字节 0亿
构建/合成/索引.js 9.65千字节 0亿
构建/core-data/index.js 11.4千字节 0亿
构建/数据控制/索引.js 1.29千字节 0亿
build/data/index.js 8.44千字节 0亿
构建/日期/索引.js 5.47千字节 0亿
build/已弃用/index.js 772磅 0亿
构建/dom-ready/index.js 569亿 0亿
构建/dom/index.js 3.19千字节 0亿
构建/编辑导航/index.js 9.87千字节 0亿
构建/编辑导航/样式-rtl.css 1.02千字节 0亿
构建/编辑导航/样式.css 1.02千字节 0亿
build/edit-post/index.js 303千字节 0亿
构建/编辑-发布/样式-rtl.css 5.51千字节 0亿
构建/编辑-发布/样式.css 5.5千字节 0亿
build/edit-site/index.js 16.7千字节 0亿
构建/编辑站点/style-rtl.css 3.03千字节 0亿
构建/编辑site/style.css 3.03千字节 0亿
build/edit-widgets/index.js 9.32千字节 0亿
build/edit-widgets/style-rtl.css 2.42千字节 0亿
build/edit-widgets/style.css 2.42千字节 0亿
build/editor/editor-styles-rtl.css 537亿 0亿
构建/编辑器/编辑器样式.css 539亿 0亿
build/editor/index.js 44.8千字节 0亿
构建/元素/索引js 4.65千字节 0亿
构建/转义-html/index.js 733亿 0亿
构建/格式化库/index.js 7.72千字节 0亿
build/format-library/style-rtl.css 547亿 0亿
构建/格式库/样式.css 548亿 0亿
build/hooks/index.js 2.13千字节 0亿
build/html-entities/index.js 622亿 0亿
构建/i18n/index.js 3.56千字节 0亿
build/is-shallow-equal/index.js 711亿 0亿
构建/键盘快捷方式/index.js 2.51千字节 0亿
build/keycodes/index.js版本 1.94千字节 0亿
构建/列出可重用块/index.js 3.12千字节 0亿
构建/list-rusable-blocks/style-rtl.css 450亿 0亿
build/list-rusable-blocks/style.css 451亿 0亿
build/media-utils/index.js 5.29千字节 0亿
build/notices/index.js 1.79千字节 0亿
构建/nux/index.js 3.4千字节 0亿
build/plugins/index.js 2.56千字节 0亿
build/primitives/index.js 1.5千字节 0亿
构建/priority-queue/index.js 788亿 0亿
build/redux-routine/index.js 2.85千字节 0亿
构建/rich-text/index.js 14千字节 0亿
build/server-side-render/index.js 2.68千字节 0亿
build/shortcode/index.js 1.7千字节 0亿
构建/token-list/index.js 1.28千字节 0亿
构建/url/index.js 4.06千字节 0亿
build/viewport/index.js 1.85千巴 0亿
build/warning/index.js 1.14千字节 0亿
构建/wordcount/index.js 1.17千字节 0亿

压缩尺寸作用

@@-12,7+12,8@@$dot-scale:3;//脉冲动画应按大小放大多少

&::之前{
动画:nux-pulse 1.6s无限立方贝塞尔(0.17,0.67,0.92,0.62);
背景:rgba($blue-medium-800,0.9);
背景:rgba(#00739c,0.9);
复制链接
贡献者

选择隐藏此评论的原因

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

我想知道这是否应该使用主题颜色(cssvars)

复制链接
贡献者 作者

选择隐藏此评论的原因

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

应该是这样。但我不知道我们在哪里再使用这些,所以我无法测试这个,看起来rgba颜色不透明度是动画的,这对于CSS变量来说是不容易做到的,对吗?所以我暂时不去管它。

复制链接
贡献者

选择隐藏此评论的原因

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

它是一个已弃用的组件,因此不太重要。

复制链接
贡献者

@你知道riad 你知道riad 留下了评论

选择隐藏此评论的原因

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

酷公关

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

我很想看到这一个。我认为它很安全,因为它只改变了一些颜色,而且在几乎所有情况下,它只会带来简单性和更好的对比度。但如果有一个模糊的地方我错过了几个节拍,我会喜欢测试。

@ItsJonQ公司
复制链接

真棒@贾斯穆森! 我要旋转一下🤞

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

谢谢您!

@@ -46,7 +46,7 @@
}

&:悬停{
颜色:蓝色400美元;
颜色:var(--wp-admin-theme-color);
}
复制链接
贡献者 作者

选择隐藏此评论的原因

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

嗯,这个可能需要轻一点,或者干脆去掉。(自我提醒)

@@-12,7+12,8@@$dot-scale:3;//脉冲动画应按大小放大多少

&::之前{
动画:nux脉冲1.6s无限立方贝塞尔(0.17,0.67,0.92,0.62);
背景:rgba($blue-medium-800,0.9);
背景:rgba(#00739c,0.9);
不透明度:0.9;
复制链接
贡献者 作者

选择隐藏此评论的原因

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

这条线本不应该被承诺。我会把它取下来。

复制链接

@ItsJonQ公司 ItsJonQ公司 留下了评论

选择隐藏此评论的原因

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

@贾斯穆森我在几个浏览器中对它进行了大量测试。我觉得一切都很好!构建没有任何问题。

我唯一的想法是(也许)第三方消费者@wordpress/基本样式。我们可能需要在changelog中添加迁移信息。

除此之外🚀 来自我!

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

太棒了,非常感谢你的测试,Q。

我唯一的想法是(也许)@wordpress/base-styles的第三方消费者。我们可能不得不在changelog中使用迁移信息进行小改动。

我当然可以整理一份开发说明。但是你能稍微澄清一下吗?为了让它成为升级拦截器,你不需要做一个完整的分叉吗?也就是说,因为这些是SCSS变量,所以它们并没有真正以有意义的方式出现。

@你知道riad
复制链接
贡献者

我可以肯定地整理一份开发说明

不,这不需要在包的CHANGELOG.md上添加开发注释,更多的是一个“破坏性更改”部分,因为这是对npm用户的破坏性更改,而不是对WordPress用户的。

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

因为这对npm用户来说是一个突破性的改变,而不是WordPress用户

我一定会帮你写这篇文章,但为了我自己的理解,与我们每天所做的许多SCSS更改相比,是什么让NPM用户做出了这样的更改?

@你知道riad
复制链接
贡献者

@贾斯穆森我们可能在无意中进行了类似的破坏性更改。删除Sass变量对npm用户来说是一个突破性的更改,因为如果使用这些变量,那么使用该包的人无法确保自己的样式在更新后继续工作。

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

哦,当然,谢谢你。

我想退休更多,改名。但我打算在后续行动中这样做,以避免公关规模过大。但也许我应该一次完成这一切?

@你知道riad
复制链接
贡献者

可以对npm包进行多次中断更改。(与WP不同)。因此,只需像往常一样单独进行PR,如果包的版本没有发布,您可以继续向CHANGELOG.md中的中断更改添加列表项

@凯勒
复制链接
贡献者

🌟 这个公关让我很高兴——我们真的不需要所有这些额外的变量。这里的额外限制将帮助我们在古腾堡设计新作品时更加一致,所以我完全支持。

我只是简单地转了一下,没有发现任何不正确的地方。👍

@伊姆托马斯主教

我爱你,@伊姆托马斯主教,因为在做这件事的时候,我没有接触任何本土的东西。我也很高兴接触到这一点,但我想先听听你的想法。

谢谢你的ping,@贾斯穆森!

简短版本:假设这里没有任何东西破坏移动设备,您可以随意忽略特定于移动设备的内容。

长版:我们将在不久的将来改进我们的颜色使用方法,以利用平台颜色在RN中,以跨平台“中立”系统为后盾,在两个平台上都感觉很自然。

//ping(平)@最大值在不久的将来处理颜色工作时,让我们记住这一点。

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

@你知道riad测试正在通过,我很想合并,你能看一下我的变更日志更新,看看我做得对吗?

@你知道riad
复制链接
贡献者

你做得对,因为另一个变更也被标记为“未发布”,但实际上另一个更改已经发布了,所以我想知道为什么变更日志没有更新。当我们发布最新的npm包cc时,我们可能错过了向master的backport@吉奥罗牌手表

不过,请随意合并此PR。

@贾斯穆森 贾斯穆森合并提交6e80f4英寸进入之内 主人 2020年6月29日
@贾斯穆森 贾斯穆森删除了 试穿/退役颜色 分支 2020年6月29日14:50
@github-操作 github-操作 机器人程序将此添加到古腾堡8.5里程碑2020年6月29日
@吉奥罗牌手表
复制链接
成员

吉奥罗牌手表 评论2020年6月30日

下次我们发布npm时,它应该会触发主要版本碰撞。我们有一种非常幼稚的检测版本更改的方法,它与标题的重要性不匹配。请参阅带修复的提交:

a689羊毛

免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
[类型]代码质量 与代码质量相关的问题或PR [类型]增强功能 改进建议。
项目
还没有
开发

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

还没有

6名参与者