User tests: Successful: Unsuccessful:
Labels |
Added:
?
|
Category | ⇒ | Templates (admin) UI/UX |
I do not see b/c problem in replace SqueezeBox.close();
to jQuery("#menusModuleModal").modal("hide”);
...
or you got some problem with it?
@Fedik The problem will be as follow:
Someone got a button from a 3rd party dev.
This button does some fancy staff and on save it closes the modal.
Ooops problem because calling SqueezeBox.close();
won’t do anything.
I STRONGLY believe that we should (only on this PR) have a proxy for the old close function
Unless if we are SURE that NOBODY ever calls this function [honestly I don’t use those buttons at all so I am not the one to certify that]
@infograf768 I didn’t push the less/css changes in every PR for the shake of simplicity. I will update the description for a proper way of testing
Labels |
Added:
?
|
@test
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
@test, Modal box works fine in my test but in IE8 some CSS problem. Please take a look at attached screen shot.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
@luredweb and @trangredweb can you please confirm that you are not able to reproduce the issue on IE8. Because, @ rajeshstarlite experienced the issue with IE8. Please confirm, so we may ready to commit. Right now it's not possible to move RTC.
Thanks all.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
i can't test it
_Error
The patch could not be applied because it conflicts with a previously applied patch: administrator/templates/isis/css/template-rtl.css_
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
@micker try to apply patch manually using https://github.com/joomla/joomla-cms/pull/4664.diff or you may try to revert your other patch which you have applied.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
yes, @gunjanpatel, I tested IE8 it's some CSS problem in IE8 as @rajeshstarlite
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
@micker you need to always revert a patch before applying another one - this makes sure that you are always testing just one patch
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
Status | Pending | ⇒ | Ready to Commit |
Thanks. Moving to RTC as we have more than 3 successfull tests.
Sorry for typo in above comment. We have 3 successfull test so moving RTC. :)
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4664.
Status | Ready to Commit | ⇒ | Needs Review |
@brianteeman and the rest of the PLT:
WARNING
This PR might have negative impact in the front end templates, if no bootstrap css is loaded!
I am closing this. PLEASE DO NOT COMMIT.
Status | Needs Review | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-17 17:26:48 |
This one AS IT IS BREAKS B/C.
All needed is a proxy from
SqueezeBox.close();
to
jQuery("#menusModuleModal").modal("hide”);
Anyone?