Skip to content
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

支持边框和credit #1

Open
Nambers opened this issue Mar 13, 2024 · 7 comments
Open

支持边框和credit #1

Nambers opened this issue Mar 13, 2024 · 7 comments
Labels
enhancement New feature or request

Comments

@Nambers
Copy link

Nambers commented Mar 13, 2024

能不能像之前一样支持添加白边框
以及能不能加个作者的字段(
以及实景图缩小后用白色(可选) 填充空缺,也就是设置白色为背景颜色

用起来挺不错就是操作的时候会有一些卡顿

@ihkk ihkk added the enhancement New feature or request label Mar 13, 2024
@Nambers
Copy link
Author

Nambers commented Mar 14, 2024

好快的加

@ihkk
Copy link
Collaborator

ihkk commented Mar 14, 2024

白边框:已加✅,但是文字定位上不太完美,待后期更新
背景色:已加✅,之前其实考虑过这个问题,但是感觉很少会出现实景图比画框小的情况,通常都是放大调整视角,出于性能考虑就没做。但是刚才试了一下好像影响不大(都很卡orz),就补上了

作者字段:您是指原图巡礼信息的提供者,还是拍摄/对比图制作者(用户)的credit?是否有排版方案可以分享,因为目前底下四角已经放满了,暂时没有特别好的想法

性能:canvas处理图片好像难以避免地卡,未来重构

@Nambers
Copy link
Author

Nambers commented Mar 14, 2024

作者字段:您是指原图巡礼信息的提供者,还是拍摄/对比图制作者(用户)的credit?是否有排版方案可以分享,因为目前底下四角已经放满了,暂时没有特别好的想法

奥我指的就是用户的credit, 不过放哪里确实是个好问题。
可能可以加成三行? 最后一行放名字 + 一段话 (作为可选, 比如可以添加当时的台词之类的), 不填就默认两行这种

@Nambers
Copy link
Author

Nambers commented Mar 14, 2024

还遇到了一个问题: 从有带作品截图的巡礼地点切换到没有作品截图的巡礼地点的时候上传按钮还是禁用的

@Nambers
Copy link
Author

Nambers commented Mar 14, 2024

白边框:已加✅,但是文字定位上不太完美,待后期更新 背景色:已加✅,之前其实考虑过这个问题,但是感觉很少会出现实景图比画框小的情况,通常都是放大调整视角,出于性能考虑就没做。但是刚才试了一下好像影响不大(都很卡orz),就补上了

奥对, 移动实景照片的y轴往下可以遮住边框(( 这个是预期的吗

@ihkk
Copy link
Collaborator

ihkk commented Mar 14, 2024

白边框:已加✅,但是文字定位上不太完美,待后期更新 背景色:已加✅,之前其实考虑过这个问题,但是感觉很少会出现实景图比画框小的情况,通常都是放大调整视角,出于性能考虑就没做。但是刚才试了一下好像影响不大(都很卡orz),就补上了

奥对, 移动实景照片的y轴往下可以遮住边框(( 这个是预期的吗

不是预期的。如果实景图向上移动遮住中间边框是之前的bug,请刷新网页缓存再试一下?向下移动应该不会遮住,

@ihkk
Copy link
Collaborator

ihkk commented Mar 14, 2024

还遇到了一个问题: 从有带作品截图的巡礼地点切换到没有作品截图的巡礼地点的时候上传按钮还是禁用的

代码逻辑上的问题,目前建议是直接刷新)后期会作调整,比如改成刷新键

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants