Remove unused sprite image

Summary: minor, unused

Test Plan: grep

Reviewers: epriestley, btrahan

Reviewed By: btrahan

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D10936
Transactions - fix bug in differential where older inlines weren't appearing

Summary: Fixes T6699. We need to "loadInlineComments" consistently, though for an unexpected reason - this mutates the $changesets to include all $changesets that have an associated inline comment, which is necessary to make them render properly.

Test Plan: Took a diff with inline comments and updated it, noting the inline comments disappeared. applied this patch and the inlines reappeared.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T6699

Differential Revision: https://secure.phabricator.com/D10935
Clean up spacing, sizing on mobile app menus

Summary: Lots of minor spacing/alignment tweaks on mobile menus

Test Plan: set browser to 320 width, inspect for issues and alignments

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D10937
Add addtional hints you haven't submitted comments on a Diff

Summary: Ref T3669. Probably. Adds a yellow warning at the top of the Diff View and makes the comment draft icon yellow on lists of revisions.

Test Plan:
Test a diff with many warnings, see warning. Test a diff with draft comments, see warning. Test new icon in list view.

{F230133}

{F230134}

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T3669

Differential Revision: https://secure.phabricator.com/D10789
Move cancel/retry/free task queue actions to bin/worker

Summary:
Fixes T6702. Ref T3554. Currently, tasks can be cancelled, retried and freed from the web UI by any logged in user.

This isn't appreciably dangerous (I can't come up with a way that a user could do anything security-affecting), but I think I probably intended this to be admin-only, but these actions should move to the CLI anyway.

Move them to the CLI. Lay some groundwork for some future `bin/worker cancel --class SomeTaskClass`, but don't implement that yet.

Test Plan: Used `cancel`, `retry` and `free` from the CLI. Hit all the error/success states.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T3554, T6702

Differential Revision: https://secure.phabricator.com/D10939
Revert "Make 'Unsubmitted Comments' icon fully configurable"

This reverts commit 0c3ff4c5b07570e32b5af74638e6d299fd8e523f.
Merge branch 'master' of github.com:phacility/phabricator

* 'master' of github.com:phacility/phabricator:
  Move cancel/retry/free task queue actions to bin/worker
  Add addtional hints you haven't submitted comments on a Diff
  Clean up spacing, sizing on mobile app menus
  Transactions - fix bug in differential where older inlines weren't appearing
  Remove unused sprite image
Change icon color match changes in a0907819cdb86f03024d8a5bf98268d79f5e23ab
Merge branch 'T6660-audit-unsubmitted-comments-icon'

* T6660-audit-unsubmitted-comments-icon:
  Change icon color match changes in a0907819cdb86f03024d8a5bf98268d79f5e23ab