-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/桌面模式下图片预览支持滚轮缩放与鼠标拖放 #72
Conversation
感谢pr,不过我更倾向于使用一个新的js库替换原本的zoom.js |
这个似乎看着更靠谱,那我直接替换😁 |
我在后来找到一个社区活跃度相对比较高的js库: timmywil/panzoom |
timmywil/panzoom#662 timmywil/panzoom#671 这两个问题似乎无法解决 |
@@ -484,29 +484,21 @@ html { | |||
opacity: 0.5 !important; | |||
} | |||
|
|||
.preview-image-dialog__image__wraper { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
使用新的js库,文件夹名也应该改变
不好意思,请问有什么进展吗?#72 (comment) |
在项目中这两个问题并没有影响正常功能, |
timmywil/panzoom#662 Screenrecorder-2024-11-05-22-37-22-366.1.mp4timmywil/panzoom#671 20241105_225840.mp4移动端也是没有发现问题 |
确实如此 zoom-helper.js
|
可行的,只不过wasm移动端没能测了,项目启动使用ip访问始终报错,你有空可以再试试,别的端粗略测试都没啥问题👍 |
set => SetVisible(value); | ||
} | ||
[Inject] | ||
private IPlatformIntegration PlatformIntegration { get; set; } = default!; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
没有被使用
@@ -55,7 +55,8 @@ | |||
|
|||
<link href="_content/Masa.Blazor/css/masa-blazor.min.css" rel="stylesheet"> | |||
|
|||
<link href="_content/SwashbucklerDiary.Rcl/css/materialdesign/v7.2.96/css/materialdesignicons.min.css" rel="stylesheet"> | |||
<link href="_content/SwashbucklerDiary.Rcl/css/materialdesign/v7.2.96/css/materialdesignicons.min.css" | |||
rel="stylesheet"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
多余的换行
<MIcon> | ||
mdi-restore | ||
</MIcon> | ||
</MButton> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
图片预览一般都没有这个按钮,已经有双击重置的情况下,感觉没什么必要
🙌OK |
Thanks @Memoyu |
大佬,斗胆pr一下,望不要嫌弃!🤪
看着桌面端下没法缩放和拖放预览图片,故从ant blazor那抄了一个过来;🤖