[#11079] - Enable fullscreen mode in TinyMCE
- Closed
- 13 Jul 2016
- Medium
- Build: staging
- # 11079
- Diff
- PhilETaylor:patch-2
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #10879 .
Summary of Changes
Enable TinyMCE Fullscreen mode
Testing Instructions
Apply patch - note new full screen menu option in tinymce editor - use it - test it
as proposed by @c-lodder in #10879 (cc: @brianteeman)
Milestone 3.6.1 please
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
Cheers. I'll look at this after I've done the menu fix
find somewhere appropriate for this css and this fixes that overlay bug as the fixed navbar z-index is 1030
div.mce-fullscreen {
z-index: 1031;
}
That's one reason I didn't implement it already Phil :)
On 12 Jul 2016 8:37 p.m., "Lodder" notifications@github.com wrote:
Cheers. I'll look at this after I've done the menu fix
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
ready for testing :)
@PhilETaylor - CSS should be applied to the LESS files ;)
ah crap... like I said
that probably needs a frontend JS/CSS guru
| Category | ⇒ | Plugins |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-07-13 00:56:33 |
| Closed_By | ⇒ | PhilETaylor |
This PR has a bug.. in so much as the top navbar that is fixed in Joomla hides the top menu bar of tinymce in full screen mode and therefore there is no way to come out of full screen mode in order to save the content with the joomla save button :-(
The PR therefore cannot be merged in its current state until this bug is fixed... but that probably needs a frontend JS/CSS guru and some funky z-index css rule or something :-( Beyond me this late at night :)