Skip to content

Conversation

@yuxi-ovo
Copy link

image

我在菜单栏选项里添加了一项复制图片的选项 复制的图片与导出功能一致

@yuxi-ovo
Copy link
Author

后续应该添加到页面右键菜单栏中 但现在还没开发所以我就先放到菜单栏里了 后续随时可以修改

@NaoCoding
Copy link
Member

嗨你好
我先 request review 給 @pubuzhixing8 和我自己,先感謝你的貢獻!

@yuxi-ovo
Copy link
Author

嗯好的 我在使用过程中习惯直接复制 导出会导致下载文件夹一大堆文件 用截图工具截取但有不居中对齐 所以才想添加这个功能 后续有任何需要改动的地方 随时联系我

Copy link
Member

@NaoCoding NaoCoding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

這個 PR 看起來沒問題,
但是我很好奇複製圖片有 JPG 和 PNG 差別的必要嗎。
我知道它和匯出圖片一樣會有透明不透明以及對應等等的屬性。
我對這部分沒有特別的意見,
再看看 @pubuzhixing8 怎麼看哈哈。

@yuxi-ovo
Copy link
Author

仅用于区别背景是否透明 浏览器复制jpeg格式的图片无效且会报错:DOMException: Failed to execute 'write' on 'Clipboard': Type image/jpeg not supported on write. 所以我复制逻辑那里写死了png格式 具体要怎么改看你们

@pubuzhixing8
Copy link
Contributor

@NaoCoding @yuxi-ovo 感谢贡献,后面我找时间确认下复制 jpeg 出错的问题,确认是否可以解决。

另外我觉得可以开始支持右键菜单了,我也觉得这个放右键菜单比较合适,我新建了一个 issue https://github.com/plait-board/drawnix/issues/325。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants