Page MenuHomePhabricator
Differential D7336

Add Link button to Remarkup assist
ClosedPublic
Actions

Authored by vrana on Oct 17 2013, 2:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 9, 6:57 PM
Unknown Object (File)
Mon, Dec 9, 2:06 PM
Unknown Object (File)
Sun, Dec 8, 11:09 PM
Unknown Object (File)
Thu, Dec 5, 6:19 AM
Unknown Object (File)
Tue, Dec 3, 10:14 PM
Unknown Object (File)
Tue, Dec 3, 5:52 AM
Unknown Object (File)
Mon, Dec 2, 4:16 PM
Unknown Object (File)
Sat, Nov 30, 9:36 PM
View All Files
Subscribers
aran
chad
epriestley

Details

Reviewers
epriestley
chad
Commits
Restricted Diffusion Commit
rP82e57ab8a76d: Add Link button to Remarkup assist
Summary

Believe it or not, I forgot how to create a link in Remarkup.

Test Plan

Clicked on it with selected URL, selected text and without a selection.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

vrana planned changes to this revision.Oct 17 2013, 2:10 AM
vrana added a subscriber: chad.

@chad, can you please sketch up some icon for it?

vrana updated this revision to Unknown Object (????).Oct 18 2013, 1:41 AM

Name the class, move the icon

vrana updated this revision to Unknown Object (????).Oct 18 2013, 1:45 AM

Phtize

vrana added reviewers: chad, epriestley.Oct 18 2013, 1:46 AM
epriestley accepted this revision.Oct 18 2013, 1:48 AM
vrana closed this revision.Oct 18 2013, 1:59 AM

Closed by commit rP82e57ab8a76d (authored by @vrana).

chad added a comment.Oct 18 2013, 2:39 AM

In general the remarkup bar is:

Fonts | Formatting | Helpers/Misc

vrana added a comment.Oct 18 2013, 3:44 PM

It seemed to me more like:

Inline | Block | Tools

Feel free to move the button but this seems like a good position to me.

Revision Contents
Changeset List

PathSize
resources/
sprite/
manifest/
2 lines
src/
176 lines
infrastructure/
javelin/
2 lines
view/
form/
control/
3 lines
webroot/
rsrc/
css/
2 lines
js/
core/
8 lines

Diff 16545

View Options

resources/sprite/manifest/icons.json

Loading...
View Options

src/__celerity_resource_map__.php

Loading...
View Options

src/infrastructure/javelin/Javelin.php

Loading...
View Options

src/view/form/control/PhabricatorRemarkupControl.php

Loading...
View Options

webroot/rsrc/css/sprite-icons.css

Loading...
View Options

webroot/rsrc/js/core/behavior-phabricator-remarkup-assist.js

Loading...
Phabricator · Built by Phacility