Skip to content

feat: add '包含墨迹' toggle and ink-overlay support for area screenshots#436

Open
Tayasui-rainnya wants to merge 8 commits intoInkCanvasForClass:betafrom
Tayasui-rainnya:beta
Open

feat: add '包含墨迹' toggle and ink-overlay support for area screenshots#436
Tayasui-rainnya wants to merge 8 commits intoInkCanvasForClass:betafrom
Tayasui-rainnya:beta

Conversation

@Tayasui-rainnya
Copy link
Copy Markdown
Contributor

我嘞个豆啊,浪费额度修那个截屏偏移的bug

这玩意是之前版本带的

目前“包含墨迹”雀食会有一点点偏移,但可以忽略不计了(我1080p,windows缩放100%)

如果说有人测出来墨迹的偏移很大,再来修吧

@CJKmkp CJKmkp self-assigned this Apr 9, 2026
Copy link
Copy Markdown
Member

@CJKmkp CJKmkp left a comment

Choose a reason for hiding this comment

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

没什么问题

@CJKmkp
Copy link
Copy Markdown
Member

CJKmkp commented Apr 10, 2026

建议迁移net6分支

@CJKmkp CJKmkp requested a review from doudou0720 April 10, 2026 10:05
@doudou0720
Copy link
Copy Markdown
Member

建议迁移net6分支

只有发起pr的人可以改分支

Copy link
Copy Markdown
Member

@doudou0720 doudou0720 left a comment

Choose a reason for hiding this comment

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

没测实际的,我妈一周现在只让我用2h电脑

@CJKmkp 你自己周末测测罢

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

请同步修改Docstring

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

请同步修改Docstring

这样可以了嘛?
(不知道是不是太细了)

@doudou0720 doudou0720 requested a review from CJKmkp April 10, 2026 14:31
Updated XML documentation for ShowScreenshotSelector method to provide clearer details on parameters and return values.
@Tayasui-rainnya
Copy link
Copy Markdown
Contributor Author

建议迁移net6分支

只有发起pr的人可以改分支

awa……我压根没fork那个net6的…………
可以直接提交到net6的分支嘛?

@doudou0720
Copy link
Copy Markdown
Member

@Tayasui-rainnya 可以的,直接改目标分支,
还有你的缩进是不是炸了

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