-
Notifications
You must be signed in to change notification settings - Fork 37
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: add translation of dde-shell plugin #141
Conversation
shell/appletloader.cpp
Outdated
@@ -86,6 +100,9 @@ DAppletLoader::DAppletLoader(class DApplet *applet, QObject *parent) | |||
{ | |||
D_D(DAppletLoader); | |||
d->m_applet = applet; | |||
connect(this, &DAppletLoader::failed, this, [d](const QString &pluginId) { | |||
d->removeTranslation(pluginId); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个对多实例的话,可能失败一个把所有的都卸载了,还不能这样卸载,
不卸载,或者在最后init完做个卸载的检查,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, yixinshark The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
be68f68
to
819417c
Compare
TAG Bot New tag: 0.0.7 |
Add translation of dde-shell plugin Log: add translation of dde-shell plugin Influence: translation
Add translation files Log: add translation files Influence: translation
Add translation of dde-shell plugin
Log: add translation of dde-shell plugin
Influence: translation