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
F19531945: D7336.diff
Tue, Jan 20, 7:59 AM
F19528237: D7336.diff
Mon, Jan 19, 7:54 AM
F19470216: D7336.id16545.diff
Mon, Jan 5, 11:34 AM
F19069301: D7336.id.diff
Nov 30 2025, 1:41 PM
F19063198: D7336.diff
Nov 29 2025, 6:09 PM
F18853804: D7336.id16543.diff
Oct 31 2025, 7:37 PM
F18850013: D7336.id.diff
Oct 30 2025, 1:50 PM
F18846650: D7336.diff
Oct 29 2025, 8:18 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