#21197 closed defect (bug) (fixed)
There are two diffrent full screen editors
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | 3.4.1 |
Component: | TinyMCE | Keywords: | |
Focuses: | Cc: |
Description
In the WP editor there is a full screen button. It has shortcut ALT+SHIFT+G. When the button is pressed it shows a different full screen editor then when the shortcut is used.
I tried this on multiple 3.4.1 installations, one of them had no plugins to interfere.
I'm not sure if this is a bug, maybe it is intended.
If so can someone explain why?
Attachments (1)
Change History (27)
#2
@
13 years ago
I think I've found a related bug:
- Press Alt + Shift + G.
- In the opened window, press "Toggle fullscreen mode" button. Nothing happens.
- Press Alt + Shift + G once again. The DFW editor shows up.
- "Exit fullscreen" link doesn't work. There's an error message in JS console:
Error: edd is undefined Source File: wp-includes/js/tinymce/plugins/wpfullscreen/editor_plugin_src.js?ver=3541-21229 Line: 31
#4
@
13 years ago
Currently there are several problems with the shortcuts in the editors. One is that Firefox uses Alt + Shift + letter
for the accesskey
HTML attribute and it's no longer possible to overwrite/prevent that default. Another is that in the latest version TinyMCE has different/new API for handling key strokes (still working out few bugs there).
Thinking it would be best to redo all custom shortcuts we add in TinyMCE avoiding all browser limitations even if they would be different for different browsers (the shortcuts are already different for MacOS and Windows).
#5
@
13 years ago
Related: ticket:17992:10
#6
@
13 years ago
- Owner set to azaozz
- Resolution set to fixed
- Status changed from new to closed
In [21266]:
#7
@
13 years ago
- Component changed from Editor to TinyMCE
- Milestone changed from Awaiting Review to 3.5
#8
@
13 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
comment:2 is still an issue.
Tested in Firefox 13, Chrome 20, IE 8, Opera 12. In Opera, a blank window with a scrollbar appears in step 3 instead of the DFW editor.
#10
@
13 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
In some languages Alt + letter is used regularly to type characters like ç (Alt+C), ł (Alt+l), ż (Alt+z), etc.
In that terms I'm not sure what shortcuts can be used for the editor buttons. Alt + Shift + letter works in Chrome, IE and Opera but doesn't work in Firefox (it triggers the "accesskey" attributes regardless of whether the element is visible or not).
The only key combination left is Ctrl + Alt + letter (Command + Option + letter on Mac), however that may run into keystroke conflicts too.
#11
@
13 years ago
For now thinking we should revert to Alt + Shift + letter for Chrome and use Ctrl + Alt + letter (Command + Option + letter) in Firefox for the custom editor shortcuts.
In the long term perhaps we can use the "Keyboard Shortcuts" checkbox in the user profile to turn on/off the shortcuts not only for moderating comments but also for the editor and any other place we may use them as well.
Edit: opened #21414 for that.
#15
follow-up:
↓ 16
@
13 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
TinyMCE help still lists Alt + Shift + G shortcut for non-DFW fullscreen mode:
http://core.trac.wordpress.org/browser/trunk/wp-includes/js/tinymce/wp-mce-help.php?rev=21545#L216
However, unless 'wpfullscreen'
plugin is manually disabled via tiny_mce_before_init
filter, the shortcut is not registered (since [21272]).
Perhaps the shortcut should still work if 'wpfullscreen'
is loaded but not active at the moment. Or should just be removed from help.
#16
in reply to:
↑ 15
@
13 years ago
Replying to SergeyBiryukov:
Think best would be to remove the TinyMCE's fullscreen from the default config. Opened #21976 for that.
#17
@
13 years ago
Something missing from the discussion so far is the "fullscreen" button in the HTML tab. That button opens DFW. So if a button is being removed from TinyMCE it might make sense to remove it in both places. Or were you fixing the existing button(s)?
#19
follow-up:
↓ 20
@
13 years ago
What's left to do here? I see #21976 for removing the TinyMCE fullscreen plugin. Anything else?
#20
in reply to:
↑ 19
@
13 years ago
Replying to miqrogroove:
Something missing from the discussion so far is the "fullscreen" button in the HTML tab. That button opens DFW. So if a button is being removed from TinyMCE it might make sense to remove it in both places. Or were you fixing the existing button(s)?
The button on the Visual tab also opens DFW. What's being removed here is TinyMCE's fullscreen plugin (which has "Alt + Shift + G" shortcut), not the DFW button.
Replying to helenyhou:
What's left to do here?
21197.patch replaces TinyMCE's fullscreen shortcut in the help window with the one for DFW.
#21
@
13 years ago
Is it possible to hook up Alt + Shift + G for DFW? That would make a lot more sense to me than removing it. That was how I found DFW in the first place, along with its many related keyboard shortcuts.
Confirmed. Looks like alt+shift+g brings up the TinyMCE fullscreen editor instead of the WordPress fullscreen (distraction-free) editor - not sure what the right command is to bind to that key combo instead, if there is one, or else I'd try patching it :)