#31161 closed enhancement (fixed)
TinyMCE Help button is irrelevant on devices without keyboards
Reported by: | siobhan | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | TinyMCE | Keywords: | make-flow has-patch |
Focuses: | Cc: |
Description
When working on a device without a keyboard, the TinyMCE help text is irrelevant as it provides information on keyboard shortcuts.
I suggest either:
a) providing contextual help text
b) removing the button entirely <--- my preference since I'm not sure what help we need to provide there
Attachments (3)
Change History (13)
#4
@
10 years ago
- Keywords has-patch added; needs-patch removed
Not sure if this is a very elegant patch but it works
#5
@
10 years ago
- Milestone changed from Awaiting Review to 4.2
Not sure if this is a very elegant patch but it works
Better to patch the editor class for this. :)
Let's see if this is good to go in.
#6
@
10 years ago
Wouldn't it be better to apply the filter at the end of the button array construction like the updated patch?
#7
@
10 years ago
v2.patch wouldn't apply for me so I tested with 31161.patch. Help was hidden on iPhone 6+ and iPad Air but showed on Macnchrome, as expected.
$ patch -p0 < ~/Downloads/31161v2.patch patching file src/wp-includes/class-wp-editor.php patch: **** malformed patch at line 18: * Filter the third-row list of TinyMCE buttons (Visual tab).
This ticket was mentioned in Slack in #core by boren. View the logs.
10 years ago
#9
@
10 years ago
- Owner set to ocean90
- Resolution set to fixed
- Status changed from new to closed
In 31718:
+1 Thanks for all the feedback! :)