Discussion:
D15332: Use the more common string "Edit" in place of "Change"
Nathaniel Graham
2018-09-07 14:26:02 UTC
Permalink
ngraham created this revision.
ngraham added reviewers: Dolphin, VDG.
Herald added a project: Baloo.
Herald added a subscriber: Baloo.
ngraham requested review of this revision.

REVISION SUMMARY
"Edit" is a more common string than "Change", so let's use that instead.

BUG: 398309
FIXED-IN: 18.12.0

TEST PLAN
[image goes here]

REPOSITORY
R824 Baloo Widgets

BRANCH
edit-instead-of-change (branched from master)

REVISION DETAIL
https://phabricator.kde.org/D15332

AFFECTED FILES
src/kcommentwidget.cpp
src/tagwidget.cpp

To: ngraham, #dolphin, #vdg
Cc: #baloo, ashaposhnikov, astippich, spoorun, abrahams
Nathaniel Graham
2018-09-07 14:28:10 UTC
Permalink
ngraham edited the test plan for this revision.

REPOSITORY
R824 Baloo Widgets

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg
Cc: #baloo, ashaposhnikov, astippich, spoorun, abrahams
Kai Uwe Broulik
2018-09-07 14:34:58 UTC
Permalink
broulik added a comment.


The window title for the tag editor should also be changed from "Change Tags" to "Edit Tags"

REPOSITORY
R824 Baloo Widgets

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg
Cc: broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Andres Betts
2018-09-07 14:41:32 UTC
Permalink
abetts added a comment.


+1

REPOSITORY
R824 Baloo Widgets

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg
Cc: abetts, broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Stefan Brüns
2018-09-07 18:37:35 UTC
Permalink
bruns added a comment.


As @broulik said, otherwise +1

REPOSITORY
R824 Baloo Widgets

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg
Cc: bruns, abetts, broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Nathaniel Graham
2018-09-07 22:27:45 UTC
Permalink
ngraham updated this revision to Diff 41184.
ngraham added a comment.


Also change dialog titles

REPOSITORY
R824 Baloo Widgets

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D15332?vs=41156&id=41184

BRANCH
edit-instead-of-change (branched from master)

REVISION DETAIL
https://phabricator.kde.org/D15332

AFFECTED FILES
src/kcommentwidget.cpp
src/kedittagsdialog.cpp
src/tagwidget.cpp

To: ngraham, #dolphin, #vdg
Cc: bruns, abetts, broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Kai Uwe Broulik
2018-09-08 09:47:57 UTC
Permalink
broulik accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
R824 Baloo Widgets

BRANCH
edit-instead-of-change (branched from master)

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg, broulik
Cc: bruns, abetts, broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Nathaniel Graham
2018-09-09 04:23:04 UTC
Permalink
ngraham closed this revision.

REPOSITORY
R824 Baloo Widgets

REVISION DETAIL
https://phabricator.kde.org/D15332

To: ngraham, #dolphin, #vdg, broulik
Cc: bruns, abetts, broulik, #baloo, ashaposhnikov, astippich, spoorun, abrahams
Loading...