martyr-deepin / deepin-terminal-gtk

DDE terminal emulator application
GNU General Public License v3.0
264 stars 57 forks source link

feat: add open link menu item #145

Closed shiqimei closed 5 years ago

shiqimei commented 5 years ago

Closes https://github.com/linuxdeepin/developer-center/issues/1073

不太会本地化😅 大佬有空可以在 i18n目录下写个简要的 README.md 说明下本地化步骤吗?

Preview: deepin-screen-recorder_Desktop_20190413210322

BLumia commented 5 years ago

_("Open in the browser") 就可以了,更新翻译只需要更新 pot 文件(有个 create_pot.sh 供使用)然后传到 transifex 上去,这个我们会定期做同步。

不过像这样的特性会不会被采纳还得经过产品那边确定是不是可以接受这样的需求,所以还是等一下产品那边的意见吧 :)

shiqimei commented 5 years ago

嗯,好的。

hualet commented 5 years ago

有一个 Copy link了,再来一个 Open link 不会好一点么? 另外,这个提交应该需要rebase一下了 @lolimay

shiqimei commented 5 years ago

有一个 Copy link了,再来一个 Open link 不会好一点么? 另外,这个提交应该需要rebase一下了 @lolimay

是指把 Open in the browser 改成 Open link 吗?

BLumia commented 5 years ago

有一个 Copy link了,再来一个 Open link 不会好一点么?

linuxdeepin/developer-center#1073 需要产品那边确认一下,以及看看现在的文案有没有问题 :)

hualet commented 5 years ago

有一个 Copy link了,再来一个 Open link 不会好一点么? 另外,这个提交应该需要rebase一下了 @lolimay

是指把 Open in the browser 改成 Open link 吗?

wangjia12 commented 5 years ago

文案: 【打开链接】【Open link】 cc @zhuxiujuancindy

shiqimei commented 5 years ago

https://github.com/linuxdeepin/deepin-terminal/pull/145/commits/009929eabded86cf33405c87617a91e9fe48eef2 仅将 commit message 由 i18m: update open link 改成 i18n: update open link.

BLumia commented 5 years ago

😂 您得记得 rebase 一下啊(((((

shiqimei commented 5 years ago

您得记得 rebase 一下啊(((((

前辈称呼晚辈“您”,晚辈可真受不住啊 :joy:

请问是 https://github.com/linuxdeepin/deepin-terminal/commit/4bc393110eaf8e8aa3e9c863c2029dccd54bc46e 这样的吗?

snyh commented 5 years ago

可以类似 https://github.com/linuxdeepin/deepin-terminal/pull/149

直接把open link 功能合并到 open 里去,这样不用修改用户界面和文案。

提交!149时,没看到这个PR ...


另外Open in filemanager和这里的几个Open意思是不同的,应该加个分隔符更合适。 open in filemanager是打开当前ctty进程的cwd目录,和终端中出现的文字没有任何关系。

shiqimei commented 5 years ago

@BLumia 前辈怎么看?

hualet commented 5 years ago

@snyh 我倒是觉得应该把目前Open的功能合并到 Open in file manager里面,Open link 和 Copy link 应该算到一类是因为他俩都不需要选中。

snyh commented 5 years ago

@snyh 我倒是觉得应该把目前Open的功能合并到 Open in file manager里面,Open link 和 Copy link 应该算到一类是因为他俩都不需要选中。

ps: Open in file manager这个文案很有歧义,且Open行为也不完善,只是粗暴的用cwd+path来检测,导致行为不正常

hualet commented 5 years ago

那要不就:

这样就比较明确了。 @snyh 老大你觉得呢?

BLumia commented 5 years ago

Open link 是新增项。不与任何一项别的合并比较好。

说实话我还真没注意到原有那里有个 Open.. 改成叫 Open selected item 我觉得更好些。

在文件管理器中显示那个文案我觉得没问题(注:Konsole 里叫 Open File Manager)。

@snyh @hualet 觉得呢?

另外,就这个 PR 而言, @lolimay 可以考虑只做新增 Open link 的功能(这样可以先 review 先合并,也保证一个 PR 只专注一件事),等具体其他行为讨论确定下来之后我们再在别的新提交里做。

shiqimei commented 5 years ago

Open link 是新增项。不与任何一项别的合并比较好。

说实话我还真没注意到原有那里有个 Open.. 改成叫 Open selected item 我觉得更好些。

在文件管理器中显示那个文案我觉得没问题(注:Konsole 里叫 Open File Manager)。

@snyh @hualet 觉得呢?

另外,就这个 PR 而言, @lolimay 可以考虑只做新增 Open link 的功能(这样可以先 review 先合并,也保证一个 PR 只专注一件事),等具体其他行为讨论确定下来之后我们再在别的新提交里做。

嗯,好的。

BLumia commented 5 years ago

看来手动合并有时候不会自动关闭 PR 的样子。

已 merged ( https://github.com/linuxdeepin/deepin-terminal/commit/0e3233e4d3556f99303162f66b7eb1c9fe305792 https://github.com/linuxdeepin/deepin-terminal/commit/6a880e1580d15f3e5e05d6410a7c890f50f8f91d ),整合原有 Open 功能的需求在其他 PR (#149)做就好 :)