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

Add icon for Unlink and Shortcode #269

Merged
merged 2 commits into from
Mar 1, 2018
Merged

Conversation

jasmussen
Copy link
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
Contributor

field2 commented Feb 9, 2018

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

@field2
Copy link
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
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
Collaborator Author

screen shot 2018-02-11 at 07 05 31

How's that?

Also, removed the unlink icon.

@field2
Copy link
Contributor

field2 commented Feb 28, 2018

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

@jasmussen
Copy link
Collaborator Author

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

@jasmussen
Copy link
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