Skip to content

修复一些样式问题,修改路由名称,修改一些书写不规范的代码#32

Open
pangpangma wants to merge 21 commits into
mainfrom
dev-front-2023
Open

修复一些样式问题,修改路由名称,修改一些书写不规范的代码#32
pangpangma wants to merge 21 commits into
mainfrom
dev-front-2023

Conversation

@pangpangma

@pangpangma pangpangma commented Mar 2, 2023

Copy link
Copy Markdown

修改路由名称跟后端定义保持一致
去掉所有的console
定义props默认值不规范导致控制台warning提示
修改了一些样式问题

@pangpangma pangpangma changed the title 修改路由名称 修复一些样式问题,修改路由名称,修改一些书写不规范的代码 Mar 2, 2023
@pangpangma

Copy link
Copy Markdown
Author

优化极小宽度下的样式展示问题

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

提交的代码不要留下这种完全没有意义的垃圾页面
image

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

没有实现的占位,也不要用完全没有意义的数值18,应该使用0
image

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image

  • 数字5和15需要左对齐,如下所示
    image

  • 这些内容还不可以点击,如果你计划重写这些页面,那最低要求也是所有原来有的页面都有了,才提交给我合并,不然你给我一个做了一半的东西,后台同步并行开发的东西,就没法调试了。

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image
这个版本号,不是随便写的,是从后台获取的

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image

  • “创建新任务” 遮挡了内容
  • 文字和按钮OK都太小

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image
+点击红圈位置,发生了胡乱跳转

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image

  • 字体太小
  • 任务的数目也和原来的不一致,这个对比看一下很明显,不应该对此视而不见的(原来有一个逻辑,如果状态是成功,那么过了几天以后,就不再显示在这里了,避免太多太杂乱) 如果重写页面,把原来有的功能都弄没了,那这个重写就完全没有价值了

@AceBear AceBear assigned pangpangma and unassigned AceBear Mar 6, 2023
@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image
此页面,刷新,这个API请求了2次,这不是预期的情况

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image
OK使用大写字母
另外字体偏小

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

image

image

宽度在1060px和950px字体大小不一致

@AceBear

AceBear commented Mar 6, 2023

Copy link
Copy Markdown
Member

字体不一致,除非是特别设计的,否则字体要保持一致

image
image

@pangpangma pangpangma assigned AceBear and unassigned pangpangma Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants