Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

t/138: Changed the labels of the insert column buttons for better UX and translatability #142

Merged
merged 3 commits into from
Nov 2, 2018

Conversation

oleq
Copy link
Member

@oleq oleq commented Oct 30, 2018

Suggested merge commit message (convention)

Other: Changed the labels of the insert column buttons for better UX and translatability. Closes ckeditor/ckeditor5#3245.

BREAKING CHANGE: The insertTableColumnBefore command has been renamed to insertTableColumnLeft.
BREAKING CHANGE: The insertTableColumnAfter command has been renamed to insertTableColumnRight.
BREAKING CHANGE: The valid values of the order option of the InsertColumnCommand constructor have changed.


Additional information

Requires executing Transifex commands after merge.

…o insertTableColumnLeft and insertTableColumnRight.
…column left' and 'Insert column after'->'Insert column right'.
@oleq oleq requested a review from ma2ciek October 30, 2018 16:10
@coveralls
Copy link

coveralls commented Oct 30, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 72d60db on t/138 into 6e46992 on master.

commandName: 'insertTableColumnBefore',
label: t( 'Insert column before' )
commandName: 'insertTableColumnLeft',
label: t( 'Insert column left' )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm thinking about Insert column to the left, longer, but more accurate.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a tricky topic. I've seen "Insert left" and "Insert column to the left" in different MS Word versions. GDocs uses simple "Insert column left". @AnnaTomanek?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd go for a simpler version.

@Reinmar
Copy link
Member

Reinmar commented Oct 31, 2018

Please ping me when you'll merge it. I need to push that to Transifex so we get those strings translated before the release.

@ma2ciek
Copy link
Contributor

ma2ciek commented Oct 31, 2018

We've agreed on this with @oleq that he will try to push these translations and do not break much of existing ones 😄

@ma2ciek ma2ciek merged commit fd2221f into master Nov 2, 2018
@ma2ciek ma2ciek deleted the t/138 branch November 2, 2018 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make add/remove col labels clearer?
4 participants