hughkli / Lookin

Free macOS app for iOS view debugging.
GNU General Public License v3.0
1.09k stars 178 forks source link

Feature: 无线连接 #19

Closed mlch911 closed 3 months ago

mlch911 commented 1 year ago

10

mlch911 commented 1 year ago

能看下吗?

hughkli commented 1 year ago

不好意思工作太忙了,刚看到。

你这也太强了吧,代码有点多,我这周末看一下哈,太强了。

mlch911 commented 1 year ago

不好意思工作太忙了,刚看到。

你这也太强了吧,代码有点多,我这周末看一下哈,太强了。

其实是"参考"了 Echo 的部分代码。 目前功能基本正常,但无线传输时,小概率会有崩溃,还没修复。 另外现在的逻辑不太好,如果局域网内有很多设备开了这个无线功能,可能得做个选择弹窗才行

hughkli commented 1 year ago

是的,这个 feature 的体验想做好的话应该还是要花不少时间的,也许我可以在设置里给个说明,指明这是个 Beta 功能,降低一下用户预期、但是确保能用。

mlch911 commented 1 year ago

还有就是 info.plist,需要加两个字段

<key>NSLocalNetworkUsageDescription</key>
<string>本地网络使用说明</string>
<key>NSBonjourServices</key>
<array>
    <string>_Lookin._tcp</string>
</array>
hughkli commented 1 year ago

我刚刚开了一个 feature/wireless 分支,你可以把你的这个 pull request 的 target branch 从 Develop 改成 feature/wireless 吗? 原因在这里有说明:https://github.com/QMUI/LookinServer/pull/130#issuecomment-1712896416

mlch911 commented 1 year ago

现在的代码会自动连接所有发现的设备,我在改为手动点击连接