? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
10 Apr 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

avatar Quy Quy - open - 10 Apr 2018
avatar Quy Quy - change - 10 Apr 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Apr 2018
Category Administration Language & Strings
avatar Quy Quy - change - 10 Apr 2018
The description was changed
avatar Quy Quy - edited - 10 Apr 2018
avatar brianteeman
brianteeman - comment - 10 Apr 2018

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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Apr 2018

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

avatar brianteeman
brianteeman - comment - 11 Apr 2018

and the elipsis should be …

avatar brianteeman
brianteeman - comment - 11 Apr 2018

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

avatar Quy Quy - change - 11 Apr 2018
Labels Added: ? ?
avatar Quy Quy - change - 11 Apr 2018
Title
[4.0] Remove space before the ellipsis
[4.0] Add space before the ellipsis
avatar Quy Quy - edited - 11 Apr 2018
avatar brianteeman brianteeman - change - 12 Apr 2018
The description was changed
avatar brianteeman brianteeman - edited - 12 Apr 2018
avatar infograf768
infograf768 - comment - 13 Apr 2018

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.

avatar brianteeman
brianteeman - comment - 13 Apr 2018

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

avatar infograf768
infograf768 - comment - 13 Apr 2018

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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Apr 2018

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

avatar Quy
Quy - comment - 9 May 2018

and the elipsis should be …

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

avatar brianteeman
brianteeman - comment - 9 May 2018

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

avatar brianteeman
brianteeman - comment - 9 May 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.

avatar brianteeman brianteeman - test_item - 9 May 2018 - Tested successfully
avatar Quy
Quy - comment - 21 Jun 2018

One more test please for this simple PR.

avatar laoneo laoneo - test_item - 21 Jun 2018 - Tested successfully
avatar laoneo
laoneo - comment - 21 Jun 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.

avatar laoneo
laoneo - comment - 21 Jun 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.

avatar laoneo laoneo - change - 21 Jun 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-06-21 04:28:48
Closed_By laoneo
avatar laoneo laoneo - close - 21 Jun 2018
avatar laoneo laoneo - merge - 21 Jun 2018

Add a Comment

Login with GitHub to post a comment