? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
9 Nov 2018

Steps to reproduce the issue

#22979

install old Joomla 4 extension - example file: http://screenshot.myjoom.la/40198d99d777
go to Joomla 4 admin

Expected result

screenshot 2018-11-09 at 15 17 26

Actual result

screenshot 2018-11-09 at 15 08 02

avatar PhilETaylor PhilETaylor - open - 9 Nov 2018
avatar PhilETaylor PhilETaylor - change - 9 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Nov 2018
Category JavaScript Repository
avatar PhilETaylor PhilETaylor - change - 9 Nov 2018
Title
Closes #22979 - correct css class for backgrounds
[4.0] correct css class for backgrounds
avatar PhilETaylor PhilETaylor - edited - 9 Nov 2018
avatar wilsonge wilsonge - change - 9 Nov 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-11-09 16:21:43
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 9 Nov 2018
avatar wilsonge wilsonge - merge - 9 Nov 2018
avatar wilsonge
wilsonge - comment - 9 Nov 2018

Thanks :)

avatar PhilETaylor
PhilETaylor - comment - 9 Nov 2018

I cannot work out how to change the success to a danger/warning though :( any ideas?

avatar PhilETaylor
PhilETaylor - comment - 9 Nov 2018

Hmmm @infograf768 has a pr that removes that somewhere - so maybe that too is broken and needs fixing now :-( if you ass that kind of error: info at the moment you get two alert boxes!!!

——
Sent from my iPhone - sorry - if needed I’ll send longer email from my desk later.
——

On 9 Nov 2018, at 16:37, George Wilson notifications@github.com wrote:

https://github.com/PhilETaylor/joomla-cms/blob/5a50c5b52a8451c92f4c76f7c5e5c53dae64a210/build/media_src/system/js/core.es6.js#L396-L399


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Add a Comment

Login with GitHub to post a comment