Feeling Lucky
?
[#13261] - Modals style cleanup
- Fixed in Code Base
- 18 Dec 2016
- Medium
- Build: staging
- # 13261
- Diff
- ciar4n:modals
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
- Moves modal template styling to blocks/_modal.less (template.less getting a bit out of control)
- Removes padding outside of scroll
- Styling added to 'close' and some other minor tweaks
- Unnecessary CSS removed
Testing Instructions
Apply patch and check non MCE modals
Documentation Changes Required
No
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Templates (admin) |
anibalsanchez
- comment
- 18 Dec 2016
I have tested this item
Test OK
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13261.
| Status | Pending | ⇒ | Ready to Commit |
jeckodevelopment
- comment
- 18 Dec 2016
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13261.
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-18 20:19:02 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
cpfeifer
-
I have tested this item✅ successfully on 18bb715
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13261.