? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
16 Mar 2017

Pull Request for Issue #14660 #14635

Summary of Changes

tag teaser image was using the full image alt tag instead of the teaser alt tag
tag full image was not using the alt tag

avatar brianteeman brianteeman - open - 16 Mar 2017
avatar brianteeman brianteeman - change - 16 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Mar 2017
Category com_tags Front End
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 16 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Mar 2017

I have tested this item successfully on 482c97f

thanks @brianteeman


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

avatar zero-24 zero-24 - change - 16 Mar 2017
Labels Added: ?
avatar zero-24
zero-24 - comment - 16 Mar 2017

@brianteeman i have just fixed a space that was to much in that file with 4f9fba8 ;)

avatar brianteeman
brianteeman - comment - 28 Mar 2017

@tonypartridge can you test this one if you get a moment - I tested your tags PR ;)

avatar tonypartridge
tonypartridge - comment - 28 Mar 2017

Hey Brian,

I have already tested this because I wrote a very similar fix with @wilsonge closed in favour of:
https://github.com/joomla/joomla-cms/pull/13906/files

However, yours resolves two issues so therefore trumps #13906 IMHO as it is more in-depth.

I'll just double test now. then @wilsonge can merge this and close #13906 ?

avatar tonypartridge tonypartridge - test_item - 28 Mar 2017 - Tested successfully
avatar tonypartridge
tonypartridge - comment - 28 Mar 2017

I have tested this item successfully on 4f9fba8


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

avatar brianteeman
brianteeman - comment - 28 Mar 2017
avatar wilsonge wilsonge - change - 28 Mar 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-28 12:18:36
Closed_By wilsonge
avatar wilsonge wilsonge - close - 28 Mar 2017
avatar wilsonge wilsonge - merge - 28 Mar 2017
avatar wilsonge
wilsonge - comment - 28 Mar 2017

Merged with two good tests (ignoring tobias' codestyle fix)

avatar tonypartridge
tonypartridge - comment - 28 Mar 2017

@wilsonge my hero.

avatar zero-24
zero-24 - comment - 28 Mar 2017

@wilsonge why do you want to ignore me / my fix?

avatar wilsonge
wilsonge - comment - 28 Mar 2017

only for the sake of two good tests (because one was before and one was after your codestyle fix) :) your fix was good!

avatar zero-24
zero-24 - comment - 28 Mar 2017

Ah ok. I just was confused ;)

Add a Comment

Login with GitHub to post a comment