Skip to content

Conversation

@brianteeman
Copy link
Contributor

Following the style guide and accessibility recommendations this PR changes the use of ... (dot dot dot) to the html entity for an ellipsis.

For example some screenreaders will announce the three dots eg read more dot dot dot but with an ellipsis they pause and do not say dot dot dot

This PR is only for the language strings. A further PR will follow to address truncation

PARTIAL Pull Request for Issue #38630

Following the style guide and accessibility recommendations this PR changes the use of ... (dot dot dot) to the html entity for an ellipsis.

For example some screenreaders will announce the three dots eg read more dot dot dot but with an ellipsis they pause and do not say dot dot dot

This PR is only for the language strings. A further PR will follow to address truncation
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.2-dev labels Aug 30, 2022
Copy link
Contributor

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

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

Maybe this should be separated. Already hyphenated and lowercase elsewhere.

brianteeman and others added 2 commits August 30, 2022 09:28
@brianteeman
Copy link
Contributor Author

Maybe this should be separated. Already hyphenated and lowercase elsewhere.

Sorry I dont understand what you are saying - maybe I need more coffee

Copy link
Contributor

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

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

Thanks @brianteeman

@tecpromotion
Copy link
Contributor

I have tested this item ✅ successfully on 56ac93c


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

@N6REJ
Copy link
Contributor

N6REJ commented Aug 30, 2022

@brianteeman docs will need to be updated as it specifically states "three periods"

@brianteeman
Copy link
Contributor Author

@Quy
Copy link
Contributor

Quy commented Aug 30, 2022

I have tested this item ✅ successfully on 56ac93c


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

@Quy
Copy link
Contributor

Quy commented Aug 30, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 30, 2022
@N6REJ
Copy link
Contributor

N6REJ commented Aug 31, 2022

@brianteeman https://developer.joomla.org/en-gb-user-interface-text-guidelines/punctuation.html

An ellipsis is a set of three periods ( ... ) indicating an omission. The group of periods should have a single space on either side. Unless it is at the end of a string where there is no space or period after.
Example

Read more ...
Create some text ... add some pictures.

Notice it says "three periods"

@brianteeman
Copy link
Contributor Author

@N6REJ thanks - the docs have been updated just waiting for a merge

@roland-d roland-d merged commit 6031e24 into joomla:4.2-dev Aug 31, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 31, 2022
@roland-d
Copy link
Contributor

Thank you

@roland-d roland-d added this to the Joomla! 4.2.2 milestone Aug 31, 2022
@brianteeman
Copy link
Contributor Author

thanks

@Razzo1987
Copy link
Contributor

Razzo1987 commented Sep 6, 2022

Also in

COM_MEDIA_SEARCH="Search..."

and
PLG_TASK_CHECK_FILES_LOG_RESIZE_FAIL="Failed to resize image due to an error in plugin logic..."

@brianteeman brianteeman mentioned this pull request Sep 6, 2022
@brianteeman
Copy link
Contributor Author

@Razzo1987 thanks - see #38710

@HLeithner
Copy link
Member

HLeithner commented Sep 7, 2022

What's the reason why you use the html encoding & hellip ; instead of the utf8 character … ?

@brianteeman
Copy link
Contributor Author

Clarity.

@HLeithner
Copy link
Member

can we please change this because this would not work in cli script or non-browser context, afaik is that we try to avoid html encoding in language files and use utf8 directly (it's done for easy writing)

@brianteeman
Copy link
Contributor Author

No I will not change it and we have plenty of html in our strings.

&hellip is easier to type and visually an editor will not mistakenly type ... when they mean …

As already explained we must use an ellipsis for accessibility and changing to use the utf8 … from the ... will not help as translators etc will not SEE that they are different and will continue to type ... especially as its much easier to type ... than it is to look up the utf8 character. It's certainly easier for me to type &hellip than it is to open a character map and locate the utf8 map and then locate the ellipsis and insert it into a document

@brianteeman
Copy link
Contributor Author

image

which of those is the real ellipsis

@HLeithner
Copy link
Member

That one which also works outside of a webbrowser.

@HLeithner
Copy link
Member

OK so the Unicode consortium are all idiots and we should go back to ascii and HTML encoding. Makes sense to me.

@HLeithner
Copy link
Member

forget the "idiots", I already argument like other people in this tracker and that makes it sick. People are doing anyway what they want to do so I should not take this personally and look at my health.

@brianteeman
Copy link
Contributor Author

brianteeman commented Sep 7, 2022

I know many of the unicode consortium, spoken at their conference twice, nice people. However, unfortunately, much of their work is not accessible at all or more accurately is misused and is therefore not accessible and we have no control/influence on how the different screen reader applications work. - they're all different to each other most of the time.

@brianteeman
Copy link
Contributor Author

No one was forced to merge this

@N6REJ
Copy link
Contributor

N6REJ commented Sep 8, 2022

is it possible to add the … to tiny as I'm darn sure never going to remember that and I use ... almost hourly

@brianteeman
Copy link
Contributor Author

What does that have to do with changing language strings?

brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request Dec 30, 2022
**I was wrong and should have listened more to what @HLeithner was saying.** He was obviously correct that you can't use &hellip, in a language string in the cli.

I still think the lack of readability of using utf characters in a search string is an issue so I have reverted the use to ...

I have also reviewed all the other changes that I made in that PR and instead of reverting I am removing the … as it really served no purpose

As far as I can see now the remaining uses from that PR all relate to Read more where I still believe it is best to use &hellip but I would be happy to revert those as well.

Sorry for causing this issue.

I am not sure if we are in a freeze for strings in 4.2 so if required I can rebase to 4.3
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.

8 participants