User tests: Successful: Unsuccessful:
It seems that this makes reference to the image_fulltext_alt but the rest of the code seems to be targeting the intro text. Shoot me down if I'm wrong, I only noticed this because I was trying to merge a conflict.
I would also question (and I really don't know the pros/cons) as to whether this could use a JLayout instead. The pro in my eyes would be code maintainability and standardisation. There may be an argument against this I really am not skilled enough to know.
Pull Request for Issue # .
It referenced the intro text instead of the full text. Fill in both items differently so you can test.
Set up an article with different intro text and full text (I suggest the text "introtext" and "fulltext").
Make sure intro image is set up with an image. Make sure intro image is set to show.
Go to a tag view ensuring the article is showing. View the source of the image and ensure there is an alt text.
article full image alt text shows
n/a
Status | New | ⇒ | Pending |
Category | ⇒ | com_tags Front End |
Looks correct to me :)
No. I don't think so. Tags does it's own thing because UCM
Test on Article with different intro- and fulltext and different alt-Text on Intro- and full-Article-Image.
With and -out PR correct alt-Text of intro-Image is shown.
I have tested this item
@franz-wohlkoenig can you confirm a test on this? Make sure no information in full image is filled out. Thanks!
I have tested this item
Test on Article show no Difference, but on Tags works.
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-19 10:53:35 |
Closed_By | ⇒ | wilsonge | |
Labels |
Added:
?
|
@wilsonge can you check that I'm not being foolish? this should be a quick change if I'm right.