Skip to content

[4.0] Add space before the ellipsis#20135

Merged
laoneo merged 3 commits intojoomla:4.0-devfrom
Quy:patch-13
Jun 21, 2018
Merged

[4.0] Add space before the ellipsis#20135
laoneo merged 3 commits intojoomla:4.0-devfrom
Quy:patch-13

Conversation

@Quy
Copy link
Copy Markdown
Contributor

@Quy Quy commented Apr 10, 2018

Summary of Changes

To be consistent with the following language strings, add the space before the ellipsis.

PLG_QUICKICON_EXTENSIONUPDATE_CHECKING="Checking extensions ..."
PLG_QUICKICON_EXTENSIONUPDATE_ERROR="Unknown extensions ..."
PLG_QUICKICON_JOOMLAUPDATE_CHECKING="Checking Joomla..."
PLG_QUICKICON_JOOMLAUPDATE_ERROR="Unknown Joomla..."

Testing Instructions

Code review

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.0-dev labels Apr 10, 2018
@brianteeman
Copy link
Copy Markdown
Contributor

i need t check but i believe that for screenreaders to be able to read the word then there must be a space between the word and the elipsis

@ghost
Copy link
Copy Markdown

ghost commented Apr 11, 2018

In Print-Publishing there is also a Space between Letter and Ellipsis. The Space is small (1/8) and non-breakable.

@brianteeman
Copy link
Copy Markdown
Contributor

and the elipsis should be …

@brianteeman
Copy link
Copy Markdown
Contributor

Not had time to check with a screen reader as I am at drupalcon regarding using three dots or the html entity

However our style guide states that there should be a space before the dots

@Quy Quy changed the title [4.0] Remove space before the ellipsis [4.0] Add space before the ellipsis Apr 11, 2018
@infograf768
Copy link
Copy Markdown
Member

In Print-Publishing there is also a Space between Letter and Ellipsis. The Space is small (1/8) and non-breakable

Depends on the language I guess. In French no space at all for ellipsis, but 1/4 unbreakable before : or ! or ;, etc.

@brianteeman
Copy link
Copy Markdown
Contributor

Thats why we have language files :) and why the language string style guide is ONLY for en-GB

@infograf768
Copy link
Copy Markdown
Member

comment undesired. I was just exchanging with @franz-wohlkoenig about print rules in different languages, not about web en-gb rules.

@ghost
Copy link
Copy Markdown

ghost commented Apr 13, 2018

@infograf768 thanks for Info, French Style was new to me.

@Quy
Copy link
Copy Markdown
Contributor Author

Quy commented May 9, 2018

and the elipsis should be …

@brianteeman Please check which is the preferred syntax. Thanks.

@brianteeman
Copy link
Copy Markdown
Contributor

@Quy as i already wrote the stlyeguide says space dot dot dot - which is what you have and why I have approved this pr

@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on e70e93e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20135.

@Quy
Copy link
Copy Markdown
Contributor Author

Quy commented Jun 21, 2018

One more test please for this simple PR.

@laoneo
Copy link
Copy Markdown
Member

laoneo commented Jun 21, 2018

I have tested this item ✅ successfully on e70e93e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20135.

@laoneo laoneo merged commit 685f916 into joomla:4.0-dev Jun 21, 2018
@zero-24 zero-24 added this to the Joomla 4.0 milestone Jun 21, 2018
@Quy Quy deleted the patch-13 branch June 21, 2018 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants