? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
14 Apr 2016

Pull Request for New Issue.

Summary of Changes

When you have multilanguage status module active the modals are not correctly displayed because the admin status bar as a higher z-index.

image

That z-index is putted there by mod_multilangstatus.

This PR removes that forced z-index.

Testing Instructions

  1. Use latest staging with admin multilanguage status module enabled
  2. Try to do a batch (in article for instance)
  3. Check the admin status bar is not darkened like the rest of the page
  4. Apply the patch
  5. Do step 2 again, you will see now the admin status bar it's darkened like the rest of the page
avatar andrepereiradasilva andrepereiradasilva - open - 14 Apr 2016
avatar andrepereiradasilva andrepereiradasilva - change - 14 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 14 Apr 2016
Title
Remove the incorrect z-index in mod_multilangue
Remove the incorrect z-index in mod_multilangstatus
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

ups, now all other modals are ok, but not the multilangue (is darkened). have to check.

don't test yet.

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

will make another PR when solved.

avatar andrepereiradasilva andrepereiradasilva - change - 14 Apr 2016
Title
Remove the incorrect z-index in mod_multilangue
Remove the incorrect z-index in mod_multilangstatus
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-04-14 15:39:51
Closed_By andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 14 Apr 2016
avatar andrepereiradasilva andrepereiradasilva - close - 14 Apr 2016
avatar MATsxm
MATsxm - comment - 14 Apr 2016

Oh great... this is the why about the bad "tooltip" for "message" :+1:

avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 14 Apr 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

yes i'm working on it

Add a Comment

Login with GitHub to post a comment