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

Remove incorrect dashicon and unnecessary label from the Shortcode block #6001

Closed
2 tasks
bobbingwide opened this issue Apr 5, 2018 · 3 comments
Closed
2 tasks
Labels
[Feature] Shortcodes Related to shortcode functionality [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Enhancement A suggestion for improvement.

Comments

@bobbingwide
Copy link
Contributor

Issue Overview

This is just one of my many requirements for the Shortcode block.
It is the most easily implementable.

The shortcode block is currently displayed with a label consisting of the code-editor dashicon SVG and a text label of "Shortcode", followed by a textarea field into which the user types the shortcode / shortcodes to be expanded.

  • The dashicon is the incorrect icon - the block itself uses shortcode
  • The label is unnecessary - taking up a lot of space

I suggest that whole label section is removed.

Steps to Reproduce (for bugs)

  1. Insert a Shortcode block.
  2. Notice that the dashicon in the label shows </> where the Block's icon is [/]
  3. Question the need for the text label "Shortcode"

Expected Behavior

More room to type the shortcode or shortcodes.

Current Behavior

Incorrect icon, unnecessary label.

Possible Solution

Remove the label.

Screenshots / Video

image

Related Issues and/or PRs

Further requirements on the shortcode block will be raised.
I have already raised #5123 and #4456.

Todos

  • Tests
  • Documentation
@danielbachhuber
Copy link
Member

PR is #6033

@danielbachhuber danielbachhuber added [Type] Enhancement A suggestion for improvement. [Feature] Shortcodes Related to shortcode functionality labels Apr 10, 2018
@afercia afercia added the [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). label Apr 10, 2018
@afercia
Copy link
Contributor

afercia commented Apr 10, 2018

I'd just like to remind that any issue/PR that touches accessibility-related things, should be marked with the related accessibility label.
For other accessibility considerations, please refer to the related PR.

@danielbachhuber
Copy link
Member

We're not going to remove the label per #6033 (comment)

The most simple thing in this case is to keep the label. Removing it is definitely an accessibility regression compared to the previous implementation.

The block icon is now correct on the master branch:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Shortcodes Related to shortcode functionality [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants