search

windingwind / zotero-types

TypeScript definitions for Zotero.
https://www.npmjs.com/package/zotero-types
MIT License
56 stars 9 forks source link

用ChromeWindow类代替所有Window？ #38

Closed volatile-static closed 6 months ago

volatile-static commented 8 months ago

现在这个类型定义是从115跑出来的

windingwind commented 8 months ago

合并之前记得清理一下commit记录

volatile-static commented 8 months ago

合并之前记得清理一下commit记录

到时候squash就好了

现在不是在等115，问题是定义里面全是unknown，使用时还要显式断言为any，但直接写any又不太好。而且之前说很久没维护的那个库不好，而这样需要我们手动维护，感觉也很难。另一个问题是这个PR里定义的XULDocument与HTMLDocument不兼容，很多地方需要借助unknown转换两次，很纠结这样是否有必要。