Skip to content
This repository was archived by the owner on Jan 26, 2023. It is now read-only.

Add icon for Unlink and Shortcode#269

Merged
jasmussen merged 2 commits intomasterfrom
add/unlink-and-shortcode
Mar 1, 2018
Merged

Add icon for Unlink and Shortcode#269
jasmussen merged 2 commits intomasterfrom
add/unlink-and-shortcode

Conversation

@jasmussen
Copy link
Copy Markdown
Collaborator

Unlink is the reverse of link. What it does on click is to break a link, unlink it, stop it from being a link. Requested by @karmatosed for an iteration on the link interface.

screen shot 2018-02-09 at 11 27 03

screen shot 2018-02-09 at 11 27 09

The other icon is for shortcodes. Fixes #261. It's for the shortcode block.

screen shot 2018-02-09 at 11 33 28

screen shot 2018-02-09 at 11 33 24

Unlink is the reverse of link. What it does on click is to _break a link_, unlink it, stop it from being a link. Requested by @karmatosed for an iteration on the link interface.

The other icon is for shortcodes. Fixes #261. It's for the shortcode block.
@field2
Copy link
Copy Markdown
Contributor

field2 commented Feb 9, 2018

Can we "flatten" the top and bottom edges of the slash in the shortcodes icon?

@field2
Copy link
Copy Markdown
Contributor

field2 commented Feb 9, 2018

Also, is this unlink meant to replace what we already have? See https://developer.wordpress.org/resource/dashicons/#editor-unlink
I can't find the request by @karmatosed ... is it in trac?

@jasmussen
Copy link
Copy Markdown
Collaborator Author

Yes, I'll flatten the top and bottom edges of the slash, and oh my — I didn't realize that there was already an unlink icon — looks like Tammie missed that too. I believe the request was on the Gutenberg Github. I'm afk next week, but maybe I'll tweak this PR today real quick to address your feedback and to remove the unlink icon.

@jasmussen
Copy link
Copy Markdown
Collaborator Author

screen shot 2018-02-11 at 07 05 31

How's that?

Also, removed the unlink icon.

@field2
Copy link
Copy Markdown
Contributor

field2 commented Feb 28, 2018

Looks good, but will this conflict with the PR I just did?

@jasmussen
Copy link
Copy Markdown
Collaborator Author

Yes but if you give me a thumbs up I will rebase, fix, and merge 🙂

@jasmussen
Copy link
Copy Markdown
Collaborator Author

Per discussion in https://wordpress.slack.com/archives/C03EESJAW/p1519836029000445 and based on your "looks good", I'm goign to merge this, rebase #274, and merge that after.

@jasmussen jasmussen merged commit 1e79e25 into master Mar 1, 2018
@jasmussen jasmussen deleted the add/unlink-and-shortcode branch March 1, 2018 08:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants