跳到内容
新问题

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

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

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

字体库:完全删除上下文 #58428

正常开放
跟踪人 #60528
t-hamano公司已打开此问题2024年1月30日·2条评论
正常开放
跟踪人 #60528

字体库:完全删除上下文 #58428

t-hamano公司已打开此问题2024年1月30日·2条评论
受让人
标签
[功能]字体库 [类型]代码质量 与代码质量相关的问题或PR

评论

@t-hamano公司
复制链接
贡献者

上下文最初的主要目的是解决中提到的冗余道具传递问题这个文档然而,与字体相关的组件树并没有那么深,而且有许多上下文曾经使用过,但从未使用过。

这些上下文应该能够通过传递道具或使用特殊方法完全删除。

不使用提供程序可以更容易地将模式对话框和排版面板完全分离,并将打开和关闭模式对话框的控制委托给数据存储,如中所尝试的那样#58350.

最终,通过命令调色板触发Font Library模式对话框将更加容易,如#54880.

唯一需要关注的是这个评论,字体库模式对话框的初始状态也是上下文管理的。为了解决这个问题,我们需要添加一个参数openModal(开放模式)在中接口定义初始状态的包。

@t-hamano公司
复制链接
贡献者 作者

附言:我们不需要用一个公关一次性解决这个问题,但我认为使用多个公关一步一步地删除上下文会更好。

@格特戴夫
复制链接
贡献者

如果某些值可以作为上下文共享,另一种选择是将上下文分解为特定用例的较小上下文,并在尽可能靠近需要的地方使用它们。或者只是把一些上下文中的值。不管怎样,避免一个庞大的背景是一个好计划。

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

没有分支或拉请求

2名参与者