Skip to content

极小概率在主页卡片卡出“内容见 ModSetup” 的 Hint #6222

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.

Sign up for GitHub

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

Jump to bottom
Open
1 task done
CreeperIsASpy opened this issue Apr 28, 2025 · 19 comments · May be fixed by #6227
Open
1 task done

极小概率在主页卡片卡出“内容见 ModSetup” 的 Hint #6222

CreeperIsASpy opened this issue Apr 28, 2025 · 19 comments · May be fixed by #6227
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue

Comments

@CreeperIsASpy
Copy link

CreeperIsASpy commented Apr 28, 2025 •
edited
Loading

检查项

描述

Image

这玩意凌云说反馈过，极小概率会卡出来这个 Hint。
具体原因是因为这一段内容的定义在 PageSetupUI 但是实现在 ModSetup。 该死的高耦合

重现步骤

到主页的地方多卡几次。
约 1/1x10^2147483647 的几率会卡出来

日志与附件

……
@CreeperIsASpy CreeperIsASpy added · Bug 新提交 需要社区初步确认其有效的新提交 labels Apr 28, 2025
@MoYuan-CN
Copy link
Collaborator

MoYuan-CN commented Apr 28, 2025 •
edited
Loading

实现机制很奇怪…… 极小概率会卡出来…… 也不影响使用……
我挂个等待确认看看龙猫修不修吧……

凌云说反馈

尼玛祸从天上来 我啥时候说过要你来反馈了……

@MoYuan-CN MoYuan-CN added 等待确认 已经过社区确认，等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Apr 28, 2025
@wyc-26
Copy link
Contributor

wyc-26 commented Apr 28, 2025

Image

@shimoranla
Copy link
Collaborator

这个概率......

完全就是在卡 Bug 吧（

@Open-KFC

This comment has been minimized.

Sign in to view
@shimoranla
Copy link
Collaborator

直接在 xaml 里加个Visibility=“Collapsed”修了得了，也符合 PCL 的代码风格

这个东西现在就是 Collapsed 啊.....

@wyc-26
Copy link
Contributor

wyc-26 commented Apr 28, 2025 •
edited
Loading

直接在 xaml 里加个Visibility=“Collapsed”修了得了，也符合 PCL 的代码风格

Public Sub UiCustomType(Value As Integer)
    If FrmSetupUI Is Nothing Then Exit Sub
    Select Case Value
        Case 0 '无
            FrmSetupUI.PanCustomPreset.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomLocal.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomNet.Visibility = Visibility.Collapsed
            FrmSetupUI.HintCustom.Visibility = Visibility.Collapsed
            FrmSetupUI.HintCustomWarn.Visibility = Visibility.Collapsed
        Case 1 '本地
            FrmSetupUI.PanCustomPreset.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomLocal.Visibility = Visibility.Visible
            FrmSetupUI.PanCustomNet.Visibility = Visibility.Collapsed
            FrmSetupUI.HintCustom.Visibility = Visibility.Visible
            FrmSetupUI.HintCustomWarn.Visibility = If(Setup.Get("HintCustomWarn"), Visibility.Collapsed, Visibility.Visible)
            FrmSetupUI.HintCustom.Text = $"从 PCL 文件夹下的 Custom.xaml 读取主页内容。{vbCrLf}你可以手动编辑该文件，向主页添加文本、图片、常用网站、快捷启动等功能。"
            FrmSetupUI.HintCustom.EventType = ""
            FrmSetupUI.HintCustom.EventData = ""
        Case 2 '联网
            FrmSetupUI.PanCustomPreset.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomLocal.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomNet.Visibility = Visibility.Visible
            FrmSetupUI.HintCustom.Visibility = Visibility.Visible
            FrmSetupUI.HintCustomWarn.Visibility = If(Setup.Get("HintCustomWarn"), Visibility.Collapsed, Visibility.Visible)
            FrmSetupUI.HintCustom.Text = $"从指定网址联网获取主页内容。服主也可以用于动态更新服务器公告。{vbCrLf}如果你制作了稳定运行的联网主页，可以点击这条提示投稿，若合格即可加入预设！"
            FrmSetupUI.HintCustom.EventType = "打开网页"
            FrmSetupUI.HintCustom.EventData = "https://github.com/Hex-Dragon/PCL2/discussions/2528"
        Case 3 '预设
            FrmSetupUI.PanCustomPreset.Visibility = Visibility.Visible
            FrmSetupUI.PanCustomLocal.Visibility = Visibility.Collapsed
            FrmSetupUI.PanCustomNet.Visibility = Visibility.Collapsed
            FrmSetupUI.HintCustom.Visibility = Visibility.Collapsed
            FrmSetupUI.HintCustomWarn.Visibility = Visibility.Collapsed
    End Select
    FrmSetupUI.CardCustom.TriggerForceResize()
End Sub

@Open-KFC
Copy link
Collaborator

我是说这里

PCL2/Plain Craft Launcher 2/Pages/PageSetup/PageSetupUI.xaml

Line 211 in e51a5cb

<local:MyHint Margin="0,15,0,1" Text="内容见 ModSetup" IsWarn="False" x:Name="HintCustom" />

@CreeperIsASpy
Copy link
Author

实现机制很奇怪…… 极小概率会卡出来…… 也不影响使用…… 我挂个等待确认看看龙猫修不修吧……

凌云说反馈

尼玛祸从天上来 我啥时候说过要你来反馈了……

笑死我原来想说的是“凌云说反馈过” 结果 sb 输入法给我干了……

@CreeperIsASpy
Copy link
Author

我是说这里

PCL2/Plain Craft Launcher 2/Pages/PageSetup/PageSetupUI.xaml

Line 211 in e51a5cb

<local:MyHint Margin="0,15,0,1" Text="内容见 ModSetup" IsWarn="False" x:Name="HintCustom" />

wyc 摘的那点是这个的具体实现，反正我是这么理解的。

@wyc-26
Copy link
Contributor

wyc-26 commented Apr 28, 2025 •
edited
Loading

你这个的 Issue 主要是复现的概率过低，而且可能是微软的问题，我们解决不了，就和 MC 忽然闪退一样（
代码上应该是没啥漏洞的……

@CreeperIsASpy
Copy link
Author

你这个的 Issue 主要是复现的概率过低，而且可能是微软的问题，我们解决不了（

和微软有啥关系，这两个本来就不应该分开。这段具体实现怎么说也应该放在 PageSetupUI 里面的。

@shimoranla
Copy link
Collaborator

请各位冷静讨论

你这个的 Issue 主要是复现的概率过低，而且可能是微软的问题，我们解决不了（

和微软有啥关系，这两个本来就不应该分开。这段具体实现怎么说也应该放在 PageSetupUI 里面的。

你可以翻翻 ModLaunch，龙猫把登录和启动都糊在了一起（

@LTCatt
Copy link
Member

LTCatt commented Apr 28, 2025

PageSetupUI 的 92 行已经调用了这个设置的 Load 了，我不太清楚为啥会出现这情况（
有稳定复现方法么……

@LTCatt LTCatt added 需要社区复现 等待反馈者以外的其他人重现 and removed 等待确认 已经过社区确认，等待开发者确认 labels Apr 28, 2025
@CreeperIsASpy
Copy link
Author

反正我目前测试出来的概率大概是：
PCL2-CE（vs 编译的） > PCL2 开源版本 ~= PCL2-CE（发版的）> PCL2 快照版 > PCL2 正式版

@CreeperIsASpy
Copy link
Author

等等！稳定的复现方法！

打开 PCL2 （任何版本都可以稳定复现）。
在主页面不要动，然后找到任何一个自定义主页文件（只要是 .xaml 的都可以）拖到窗口内。
接下来，不管主页是否有效，加载成功后翻到设置页 -> 个性化。
内容见 ModSetup出现了！

cc @LTCatt

@Open-KFC
Copy link
Collaborator

Open-KFC commented Apr 29, 2025 •
edited
Loading

看样子是 Set 之后被标记为已处理然后后面的 Load 直接跳过了。

@shimoranla shimoranla added 等待确认 已经过社区确认，等待开发者确认 and removed 需要社区复现 等待反馈者以外的其他人重现 labels Apr 29, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度：中 and removed 等待确认 已经过社区确认，等待开发者确认 labels Apr 29, 2025
CreeperIsASpy added a commit to CreeperIsASpy/PCL2 that referenced this issue Apr 29, 2025
@CreeperIsASpy
fix Hex-Dragon#6222
88be968
@CreeperIsASpy CreeperIsASpy linked a pull request Apr 29, 2025 that will close this issue
Open
@LTCatt LTCatt removed the 处理中 开发者正在调查或处理该项 label Apr 29, 2025
@LTCatt LTCatt added 可合并 已有 Pull Request 处理了该 Issue and removed 🟨 中 优先度：中 labels Apr 29, 2025
shimoranla added a commit to CreeperIsASpy/PCL2 that referenced this issue May 3, 2025
@shimoranla
Revert(88be968): fix Hex-Dragon#6222
825334c
@dfggtth
Copy link

dfggtth commented May 5, 2025

EBEB-B00A-D09C-2990

@shimoranla
Copy link
Collaborator

shimoranla commented May 5, 2025 •
edited
Loading

EBEB-B00A-D09C-2990

请不要在其他人的 Issue 发你的解锁码，这个 Issue 不属于你

不要为了隐藏主题影响其他 Issue 的正常跟踪和活动，感谢配合

@Hex-Dragon Hex-Dragon temporarily blocked dfggtth May 5, 2025
@LTCatt
Copy link
Member

LTCatt commented May 5, 2025

由于 @dfggtth 多次在无关 issue 下索要解锁码，影响社区正常讨论，现决定禁言 7 日。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: 极小概率在主页卡片卡出“内容见 ModSetup” 的 Hint CreeperIsASpy/PCL2
7 participants
@LTCatt @MoYuan-CN @wyc-26 @CreeperIsASpy @Open-KFC @shimoranla @dfggtth