Page MenuHomePhabricator

Improve description text in the "Create Diff" form
ClosedPublic

Authored by leoluk on Feb 5 2019, 12:24 PM.
Tags
None
Referenced Files
F13287646: D20092.diff
Tue, Jun 4, 9:04 AM
F13275193: D20092.diff
Fri, May 31, 4:30 AM
F13263216: D20092.id47978.diff
Mon, May 27, 7:57 AM
F13261278: D20092.diff
Mon, May 27, 12:52 AM
F13245224: D20092.diff
Thu, May 23, 5:52 AM
F13237825: D20092.id.diff
Tue, May 21, 3:15 PM
F13236687: D20092.diff
Tue, May 21, 10:27 AM
F13235826: D20092.diff
Tue, May 21, 6:45 AM
Subscribers

Details

Summary

The textarea is, in fact, above the description!

Test Plan

Description text changed.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

How do you feel about putting this text box back above the form instead, i.e. partially reverting D18529?

It's just a minor thing I was briefly confused about when I used the "Create Diff" for the first time.

IMO, the message is only useful for new users who do not know about Arcanist, and of no use to anyone who purposefully creates a diff, so it's fine to keep it at the bottom.

I added you to Community and Blessed Committers, so you should be able to land this yourself. See the project description for Blessed Committers if you run into trouble, or let me know if that isn't helpful.

This revision is now accepted and ready to land.Feb 5 2019, 1:41 PM
This revision was automatically updated to reflect the committed changes.

Thanks, worked perfectly!

Just one quick question - could you rename this user to leoluk to match GitHub and the Discourse forums? There are no literal mentions anywhere.

Would've created a new account but since that's no longer a thing... :-)

Sure: I renamed existing user leoluk to leoluk-moved-aside, and then renamed lou to leoluk.

Thanks, greatly appreciated.