Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for #9832 + content embedded in a container div #9991

Merged
merged 1 commit into from
Apr 19, 2016

Conversation

cyrezdev
Copy link
Contributor

Pull Request for Issue #9832 .

Summary of Changes

When Bootstrap modal loads an iframe, BS tooltips with placement top are truncated at the top border of the iframe. (no auto placement in BS2)
This PR sets the tooltip placement to bottom.
In the same time, content is embedded in a container div (using already existing container-popup class for modal.php)

Testing Instructions

Go to: Components > Messaging > click on Settings button to open this modal.

Before Patch:
capture d ecran 2016-04-19 a 17 11 43

After Patch:
capture d ecran 2016-04-19 a 17 38 26

When Bootstrap modal loads an iframe, BS tooltip with placement top are truncated at the top border of the iframe. (no auto placement in BS2)
This PR sets the tooltip placement to bottom.
In the same time, content is embedded in a container div (using already existing container-popup class for modal.php)
@cyrezdev cyrezdev changed the title Fix for #9832 + embed modal content as container Fix for #9832 + content embedded in a container div Apr 19, 2016
@andrepereiradasilva
Copy link
Contributor

I have tested this item ✅ successfully on a9c4e02

works as described. thanks


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

@richard67
Copy link
Member

I have tested this item ✅ successfully on a9c4e02


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

@cyrezdev
Copy link
Contributor Author

Thanks @andrepereiradasilva and @richard67 for testing 👍
Another one in this series started: Versions modal #9995 ;-)

@brianteeman
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 19, 2016
@brianteeman brianteeman added this to the Joomla 3.5.2 milestone Apr 19, 2016
@rdeutz rdeutz merged commit b70dff5 into joomla:staging Apr 19, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 19, 2016
@rdeutz rdeutz modified the milestones: Joomla 3.5.2, Joomla! 3.6.0 May 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants