? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
4 Oct 2017

PR for #15269

I cant see any reason for the active class to be green and I cant find any other use of it so I simply removed the class from the template

Unless anyone has any better idea

avatar joomla-cms-bot joomla-cms-bot - change - 4 Oct 2017
Category Administration Templates (admin) Front End Templates (site)
avatar brianteeman brianteeman - open - 4 Oct 2017
avatar brianteeman brianteeman - change - 4 Oct 2017
Status New Pending
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 4 Oct 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 4 Oct 2017

I have tested this item successfully on 30731ed

Theres also a Alignment-Issue ("Begin date" and "End date" are not vertical aligned) :
screenshot-2017-10-4 banners tracks - 3 - administration


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18231.
avatar C-Lodder
C-Lodder - comment - 4 Oct 2017

@brianteeman - can you also use the code I provided in this PR to: #15269 (comment)

cause currently you're only addressing half of it.

avatar brianteeman
brianteeman - comment - 4 Oct 2017

@C-Lodder what is not addressed?

screenshotr16-23-27

avatar C-Lodder
C-Lodder - comment - 4 Oct 2017

The margin:

a2d10e96-1f89-11e7-85c7-134b6fc5e497

You're also editing the BS CSS file in this PR which shouldn't be done

avatar brianteeman
brianteeman - comment - 4 Oct 2017

Ah I see the margin now. was hard to see on my screen as it was on a second row.

The only file i edited was media/jui/less/forms.less

avatar C-Lodder
C-Lodder - comment - 4 Oct 2017

In which case this should be done at template level. If you edit media/jui/less/forms.less, you to modify the bootstrap.css which should never happen

avatar brianteeman
brianteeman - comment - 4 Oct 2017

re the margin
your code was not correct as it is set here https://github.com/joomla/joomla-cms/blob/staging/media/jui/css/jquery.searchtools.css#L63

avatar C-Lodder
C-Lodder - comment - 4 Oct 2017

Hmm worked fine for me

avatar brianteeman brianteeman - change - 4 Oct 2017
Labels Added: ?
avatar Quy Quy - test_item - 4 Oct 2017 - Tested successfully
avatar Quy
Quy - comment - 4 Oct 2017

I have tested this item successfully on 09f1fdc


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 5 Oct 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Oct 2017

I have tested this item successfully on 09f1fdc


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Oct 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Oct 2017
Easy No Yes
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Oct 2017

RTC after two successful tests.

avatar mbabker
mbabker - comment - 5 Oct 2017

Before I merge, I have to ask since this is changing the Bootstrap stuff. Is there anywhere else this combination of classes is used producing the green display?

avatar brianteeman
brianteeman - comment - 5 Oct 2017

I couldnt find any

avatar mbabker mbabker - change - 6 Oct 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-10-06 22:07:01
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 6 Oct 2017
avatar mbabker mbabker - merge - 6 Oct 2017
avatar brianteeman
brianteeman - comment - 6 Oct 2017

Thanks

Add a Comment

Login with GitHub to post a comment