Unit/System Tests PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
2 Jul 2025

Pull Request for Issue #45665 .

Summary of Changes

fixed the truncateComplex method

Testing Instructions

set introtext to show
set Introtext Limit (characters)
create an article with more characters than Introtext Limit
publish the Articles module

Actual result BEFORE applying this Pull Request

Integer at diam ac nibh viverra vestibulum.

...

Expected result AFTER applying this Pull Request

Integer at diam ac nibh viverra vestibulum...

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar alikon alikon - open - 2 Jul 2025
avatar alikon alikon - change - 2 Jul 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jul 2025
Category Libraries
avatar alikon alikon - change - 2 Jul 2025
Labels Added: PR-5.3-dev
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jul 2025
Category Libraries Libraries Unit Tests
avatar alikon alikon - change - 2 Jul 2025
Labels Added: Unit/System Tests
avatar alikon alikon - change - 3 Jul 2025
The description was changed
avatar alikon alikon - edited - 3 Jul 2025
avatar alikon
alikon - comment - 3 Jul 2025

the drone failure is unrelated to this pr (are about to the stats plugin), the Github Action CI are successfull

avatar alikon alikon - change - 3 Jul 2025
The description was changed
avatar alikon alikon - edited - 3 Jul 2025
avatar alikon alikon - change - 3 Jul 2025
Title
Orphaned Ellipsis
[5] Orphaned Ellipsis in Articles Module
avatar alikon alikon - edited - 3 Jul 2025
avatar chmst
chmst - comment - 5 Jul 2025

Tested this and had a good result
grafik

But: My screenreader seems to ignore the "...".
While I can see that this is a truncated text, the screeneader (NVDA on win11) reads a incomplete phrase.

Maybe someone with better screenreader skills can test that?
If my observation is correct, I suggest a sr-only information like "The following text is truncated" or whatever is convenient in this case.

avatar brianteeman
brianteeman - comment - 5 Jul 2025

This depends on the screen reader.

In particular, most screen readers have a number of punctuation settings such as "some", "most", and "all". So it depends which category the screen reader puts the ellipses character into, which punctuation setting the user has selected, and what action the user is taking.

It also depends if it is a fake ellipsis as used here (three dots) or a real ellipsis … or …

Why Don’t Screen Readers Always Read What’s on the Screen? Part 1: Punctuation and Typographic Symbols

avatar brianteeman
brianteeman - comment - 5 Jul 2025

Maybe someone with better screenreader skills can test that?
If my observation is correct, I suggest a sr-only information like "The following text is truncated" or whatever is convenient in this case.

The read more button/link should act as the indicator that there is more text to follow and you dont need the sr-only information as that would be a double notification

avatar chmst
chmst - comment - 5 Jul 2025

It is the introtext, not the read more button.

avatar brianteeman
brianteeman - comment - 5 Jul 2025

It is the introtext, not the read more button.

Yes I know. The read more is the indication that there is more text.

avatar dautrich dautrich - test_item - 5 Jul 2025 - Tested successfully
avatar dautrich
dautrich - comment - 5 Jul 2025

I have tested this item ✅ successfully on 021a5aa


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

avatar exlemor exlemor - test_item - 5 Jul 2025 - Tested successfully
avatar exlemor
exlemor - comment - 5 Jul 2025

I have tested this item ✅ successfully on 021a5aa

I have succesfully tested this! (Thanks to Alikon, Christiane, Brian!


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

Add a Comment

Login with GitHub to post a comment