Skip to content
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

关于项目的一些建议 #6

Open
Relax-87 opened this issue May 8, 2022 · 0 comments
Open

关于项目的一些建议 #6

Relax-87 opened this issue May 8, 2022 · 0 comments

Comments

@Relax-87
Copy link

Relax-87 commented May 8, 2022

项目集成对于新手来说需要很长一段时间才能撸懂 我作为新手中的新手 对此研究很久 因此我对您的项目进行了部分修改

下面是一些我个人看点和建议

  • 1、i18n 国际化代码片段有些问题
  • 1.1、命名方面locales一眼不太明白含义虽然有readme但是,lang比较贴合大部分新手老手一眼能知道这是语言部分
  • 1.2、localeChanger下的index.vue中
原 7行处
{{ $t('localeChangerTitle') }} <span style="font-weight: bold;">{{ $i18n.locale }}</span>
改
{{ $t('langChangerTitle') }} <span style="font-weight: bold;">{{ $t(`langs.` + $i18n.locale) }}</span>
并在zh.json文件中 添加
如下
  "langs":{
    "zh-CN":"中文-简体",
    "zh-TW":"漢語-台灣",
    "zh-HK":"漢語-香港",
    "en":"English"
  }

如此修改 可以直接在i18n组件中显示正确语言

2、I18n的组件位置,个人建议可以放在 App.vue 的div id=app-action下第一个位置有助于任意页面任意时间直接操作语言切换

3、微标logo建议在div id="app-bar"下放一个

4、关于eslint 的规范 standard,个人感觉不太友好 如果可能请换prettier
对于 standard与prettier 区别在哪
请参考这篇文章https://blog.csdn.net/qq_21567385/article/details/109136668

5、关闭窗体 建议可配置 close() 与 remote.app.exit() 不是每个程序都要 用托盘付关闭程序

我仅仅是没事爱写点小工具 所以上述您可以关注也可以无视!只是一些不专业的建议!

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

No branches or pull requests

1 participant