Differential D12970

Fixed grammar issue
ClosedPublic
Actions

Authored by ssaxena on May 18 2018, 6:49 PM.

Details

Reviewers
ngraham
valeriymalov
Commits
R530:2627742156f6: Fixed grammar issue
Summary

Changed "Allows to ..." to "Allows one to" in the 3 files I could find it.

BUG: 393471

Diff Detail

Repository
R530 Wacom tablet support
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ssaxena requested review of this revision.May 18 2018, 6:49 PM
ssaxena created this revision.
valeriymalov accepted this revision.May 19 2018, 10:49 AM

English is not my first langauge, so I'll trust you on this one :v
Looks good to land, or I can land it for you

This revision is now accepted and ready to land.May 19 2018, 10:49 AM
ngraham accepted this revision.May 19 2018, 2:46 PM

Yup, the new string is grammatically correct.

ssaxena added a comment.May 19 2018, 4:02 PM
In D12970#264986, @valeriymalov wrote:

English is not my first langauge, so I'll trust you on this one :v
Looks good to land, or I can land it for you

I do not have a KDE developer account yet (hope to apply soon!), so could you please land it for me?

Thanks!

Closed by commit R530:2627742156f6: Fixed grammar issue (authored by ssaxena, committed by valeriymalov). · Explain WhyMay 20 2018, 9:27 AM
This revision was automatically updated to reflect the committed changes.
valeriymalov added a comment.May 20 2018, 9:29 AM

@ssaxena for future patches, please configure your git to use full name in the author field

Revision Contents
Changeset List

Diff 34504

View Options

src/kcmodule/tabletpagewidget.ui

Loading...
View Options

src/kcmodule/touchpagewidget.ui

Loading...
View Options

src/kded/tabletdaemon.h

Loading...