-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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: 添加 Leaflet 项目作为地图打点,可以在web上预览 #899
Conversation
@MarkShawn2020 有什么想法, 可以执行 |
|
@MarkShawn2020 你的 npm 版本是多少?我这把跑了一下没问题。 要求
能发一下给我看看你的 |
|
@MarkShawn2020 我运行了一下, 没有问题, 你试试单独用 vscode 打开这个项目 然后访问 http://127.0.0.1:9080/marker-icon.png 这个地址,应该可以找到对应的 icon |
我怀疑是在根目录执行的命令,导致没有找到对应的 public 目录 |
知道图片地址就行,我来调试调试 |
@MarkShawn2020 感觉弄一个 github pages 会好一点, 然后采用 github action 进行部署 |
找到原因了,是因为前端申请的资源是 Imagemarker-icon-2x.png,为啥有个2x啊? |
This comment was marked as resolved.
This comment was marked as resolved.
@MarkShawn2020 等等 我打个补丁. 我这个代码写的有点问题,没考虑到 retina显示屏 |
ok,验证通过了,你继续做你想做的吧(可以看一下我们的前端todo),我先给你合并一下 |
@MarkShawn2020 我需要一个小区级别的数据格式. 这个能帮忙准备一下吗? |
这个需要借助我今早发布的 backend-py-sdk,用来爬开发商与小区数据,目前优先级不是很高,而且量大,脏处理比较多,你会的话可以试一下,我还要先做其他 features |
你的project已合入主分支,另外,你的技术栈跟我接近,或许可以考虑与我之前的react项目融合,一起开发 |
我看了一下你的项目,是基于 CreateReactApp 的后面调整webpack 比较麻烦, 而且打包的时候按需加载也比较麻烦, 如果加 React Router DOM 做路由还要自己配置, 所以用 fastpack 的脚手架会好点, 编译速度和打包速度,以及首页加载速度都比 Create React App 要快 |
可以,那就基于你的项目开发,看情况我们合作 |
webpack的调整一般可以用eject,暂时还没那个需求,其实如果有建站需求的话,可以考虑用一些ssr框架,性能会更好 |
eject 太麻烦, 没事 fastpack 我熟悉一点, 而且 ui 框架 fast-ui 一整套搭建起来很方便的. 主要是后面想做 pwa, ssr 很消耗服务器资源的, pwa 比 ssr 快很多,并且不怎么消耗服务器资源 |
嗯好的~ |
效果信息如下 地图数据来源 openstreetmap
link
后续加上搜索的功能,但是
/data/generated/cities-for-visualization.json
这里面生成的数据信息好像没有小区名称