0xasahi / neogroup

Donation: https://ko-fi.com/neogroup
https://neogrp.club/
MIT License
12 stars 5 forks source link

feat: neogrp topic use new UI #3

Closed anig1scur closed 1 year ago

anig1scur commented 1 year ago

主要是 topic UI 的改动 (需要我提供一些什么方便 review 吗 ? )感觉最好是先 pull 看看用线上数据库跑会不会有什么问题

@0xasahi plz when you're OK 👀


我想做一个色调是灰/紫的主题 ! 好像有点不太官方

image

image

然后下面这是设想的 user profile 和 home 页布局

image image
0xasahi commented 1 year ago

另外,settings.py 这个文件不能删除阿?

0xasahi commented 1 year ago

前端这个改法动作好大....可能最大的难点在于,我不太会前端那一套工具链....后续的维护我感觉会很吃力.....不知道有没有可能仍然是借助django templates那一套继续做....

0xasahi commented 1 year ago

@anig1scur 就是,有没有可能主要是在 HTML + CSS 上做美化升级,先不急着引入 React 那一套工具链

anig1scur commented 1 year ago

另外,settings.py 这个文件不能删除阿?

诶 没注意。。。。 不知道啥时候删掉的

anig1scur commented 1 year ago

前端这个改法动作好大....可能最大的难点在于,我不太会前端那一套工具链....后续的维护我感觉会很吃力.....不知道有没有可能仍然是借助django templates那一套继续做....

超级简单! 我可以写一个前端 dev readme ,写起来会比 templates 那一套易读、方便很多

anig1scur commented 1 year ago

@anig1scur 就是,有没有可能主要是在 HTML + CSS 上做美化升级,先不急着引入 React 那一套工具链

最开始是打算这样的, 但是在改的过程中发现写模板好痛苦 😭 ... 有想要的交互什么的做起来也麻烦

0xasahi commented 1 year ago

超级简单! 我可以写一个前端 dev readme ,写起来会比 templates 那一套易读、方便很多

确实需要有个文档,就写到 README 里吧,感谢

0xasahi commented 1 year ago

这两天我本地配置一下环境试试~

0xasahi commented 1 year ago

对了,还有个问题是,现在的发 topic 的页面,其实是可以发图片的,复制一个图片,粘贴到文本框里,可以试试

anig1scur commented 1 year ago

对了,还有个问题是,现在的发 topic 的页面,其实是可以发图片的,复制一个图片,粘贴到文本框里,可以试试

嗯嗯 我还没有修改新建 topic 这个页面的 :

https://anig1scur-zany-space-engine-5xw6q9vr6jg376g6-8000.preview.app.github.dev/group/1/new_topic


还是说你是指回复现在可以发图片 ? (试了下不能也)

0xasahi commented 1 year ago

还是说你是指回复现在可以发图片 ? (试了下不能也)

回复不行,只有 new_topic 那个页面可以发图片

0xasahi commented 1 year ago

这两天我本地配置一下环境试试~

本地环境顺利起来了,我本地这两天再 review 一下~

0xasahi commented 1 year ago

环境没什么问题了,但准备往线上部署的时候,想到一个问题,现在这种写法,会导致SEO无效吧? @anig1scur

anig1scur commented 1 year ago

噢! 容我研究一下 ssr ,这周我会修改小组主页和 home 页

anig1scur commented 1 year ago

我在这个 commit 把 SSR 差不多弄好了

image

可访问 https://anig1scur-zany-space-engine-5xw6q9vr6jg376g6-8000.preview.app.github.dev/group/topic/1/?comment_id=2 禁用 js 试试看


ssr 需要额外起一个 node server ,大概是这样

cd static_source
node hypernova-bootstrap.js
0xasahi commented 1 year ago

超赞👍,这个ssr有可能fallback回django吗?必须是node吗

anig1scur commented 1 year ago

(我感觉... 你好像一个研究生导师在一步步引导我 improve 这个 commit 😂 )

目前的 ssr 方案是 https://github.com/marteinn/django-react-polls-with-hypernova-examples 这里的 example 2 ,是没法 fallback 到 django 直接走 client side rendering 的。 ssr service 挂了的话就没法渲染,必须是 node

example 1 的话也有 pro & con s ,是一个更具弹性的方案


之前有想过用 next.js 来做把前后端完全分离,但这个动作更大了

0xasahi commented 1 year ago

@anig1scur OK,走node也行,那这个serve启动区分dev模式和prod模式吗(确实没怎么用过node,择日我准备部署现在这个PR试试了

anig1scur commented 1 year ago

那我找时间开始写首页和小组首页了 有什么要求建议都可以提 lol

anig1scur commented 1 year ago

为啥把我的 style branch 删掉了 😂

0xasahi commented 1 year ago

为啥把我的 style branch 删掉了

这个删除的是远程branch,没事的,理论上来说一个branch被merge之后,就会删除掉,不影响你下次还用style branch,重新push就好~