Page MenuHomePhabricator
Feed Advanced Search

Aug 22 2017

chad added a revision to T12824: Redesign Diffusion: D18452: Update blame view in Diffusion.
Aug 22 2017, 4:13 AM · Design, Diffusion

Aug 21 2017

chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

11" Macbook breakpoint is 1080 it looks, I think we still do tablet around 960? 1080 is enough space for anything we do. Not sure where @avivey 's laptop actually is. I do want to add more information to blame view though, so the space will be used.

Aug 21 2017, 8:56 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I was joking.

Aug 21 2017, 8:47 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Did you consider increasing the "tablet" breakpoint above the size of an 11" macbook instead?

Aug 21 2017, 8:45 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Just for @avivey and his 11" macbook: D18448

Aug 21 2017, 8:36 PM · Diffusion, Design
chad closed T12838: Make sure "state" is in new Branch view, a subtask of T12824: Redesign Diffusion, as Resolved.
Aug 21 2017, 4:09 PM · Design, Diffusion
chad closed T12838: Make sure "state" is in new Branch view as Resolved by committing rPd2a3f2da73a0: Add indication of hg branch open/closed in branch list.
Aug 21 2017, 4:09 PM · Design, Diffusion
chad added a revision to T12838: Make sure "state" is in new Branch view: D18447: Add indication of hg branch open/closed in branch list.
Aug 21 2017, 3:55 PM · Design, Diffusion

Aug 17 2017

chad added a revision to T12838: Make sure "state" is in new Branch view: D18443: Add more information to Branch status page in Manage Repository.
Aug 17 2017, 7:25 PM · Design, Diffusion
chad added a subtask for T12824: Redesign Diffusion: T12964: SSH Key adding workflows could use better UX.
Aug 17 2017, 2:25 AM · Design, Diffusion

Aug 16 2017

chad added a comment to T12838: Make sure "state" is in new Branch view.

neato!

Aug 16 2017, 10:52 PM · Design, Diffusion
epriestley added a comment to T12838: Make sure "state" is in new Branch view.

Yes. In Mercurial, branch heads may be "Open" or "Closed".

Aug 16 2017, 9:14 PM · Design, Diffusion
chad closed T12837: Make it clear you're browsing the history of a tag, a subtask of T12824: Redesign Diffusion, as Resolved.
Aug 16 2017, 7:16 PM · Design, Diffusion
chad closed T12837: Make it clear you're browsing the history of a tag as Resolved by committing rP19dae8872851: Add branch, tag info to Diffusion Headers.
Aug 16 2017, 7:16 PM · Design, Diffusion
chad added a revision to T12837: Make it clear you're browsing the history of a tag: D18434: Add branch, tag info to Diffusion Headers.
Aug 16 2017, 7:16 PM · Design, Diffusion
chad added a comment to T12838: Make sure "state" is in new Branch view.

That the branch is closed?

Aug 16 2017, 7:15 PM · Design, Diffusion
chad added a comment to T12838: Make sure "state" is in new Branch view.

What is state here? I don't really remember what this was about, or if it's needed.

Aug 16 2017, 7:15 PM · Design, Diffusion
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Search is now in the header, see rP

Aug 16 2017, 1:47 AM · Diffusion, Design

Aug 12 2017

epriestley added a subtask for T12824: Redesign Diffusion: T929: Allow two arbitrary revisions to be compared in Diffusion.
Aug 12 2017, 10:52 AM · Design, Diffusion
epriestley merged task T12831: Update compare page with more features into T929: Allow two arbitrary revisions to be compared in Diffusion.
Aug 12 2017, 10:52 AM · Design, Diffusion
epriestley added a comment to T12831: Update compare page with more features.

I'm just going to merge this into T929, I don't think it's a distinct task. I'll link that to T12824.

Aug 12 2017, 10:52 AM · Design, Diffusion

Aug 11 2017

kojik1010 added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

adding my vote to @ivo's suggestion.

Aug 11 2017, 6:59 PM · Diffusion, Design

Aug 10 2017

chad closed T12832: Add autoclose table to Diffusion Manage as Resolved by committing rPa7124f8f7a11: Add status table to Diffusion Branch manage page.
Aug 10 2017, 8:26 PM · Design, Diffusion
chad closed T12832: Add autoclose table to Diffusion Manage, a subtask of T12824: Redesign Diffusion, as Resolved.
Aug 10 2017, 8:26 PM · Design, Diffusion
chad added a revision to T12832: Add autoclose table to Diffusion Manage: D18386: Add status table to Diffusion Branch manage page.
Aug 10 2017, 5:16 PM · Design, Diffusion
chad closed T12833: Credential workflow for URIs has spacing issues as Resolved by committing rP3ba196152f9b: Clean up some dialog spacing.
Aug 10 2017, 2:38 PM · Design, Diffusion
chad closed T12833: Credential workflow for URIs has spacing issues, a subtask of T12824: Redesign Diffusion, as Resolved.
Aug 10 2017, 2:38 PM · Design, Diffusion
chad added a revision to T12833: Credential workflow for URIs has spacing issues: D18382: Clean up some dialog spacing.
Aug 10 2017, 3:04 AM · Design, Diffusion

Aug 6 2017

chad closed T12752: Pager in Diffusion doesn't tell you what page you're on as Resolved.
Aug 6 2017, 2:51 AM · Design, Diffusion
chad closed T12752: Pager in Diffusion doesn't tell you what page you're on, a subtask of T12824: Redesign Diffusion, as Resolved.
Aug 6 2017, 2:51 AM · Design, Diffusion
chad closed T12763: Have custom icons made? Object + State? as Wontfix.
Aug 6 2017, 2:51 AM · Design
chad closed T11651: Owners empty states are lookin' weird as Resolved.

seems ok now.

Aug 6 2017, 2:50 AM · Owners, Design
chad closed T12099: Implement caret as part of PHUIIconView as Resolved.

caret now appears when attaching dropdown menus to buttons.

Aug 6 2017, 2:47 AM · PHUI, Design
chad closed T10208: Document something about Design Feedback as Wontfix.
Aug 6 2017, 2:46 AM · Documentation, Design

Aug 1 2017

ivo added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

In the past there used to be a Browse button at the repository homepage in Diffusion. Clicking on it would show the file tree for the current branch and some extra buttons. You could then click Search to grep the file contents of the current branch.

Aug 1 2017, 6:45 PM · Diffusion, Design

Jul 29 2017

epriestley added a comment to T11456: Don't lose user in NUX flow because of Timezone issues.

Most browsers can only give us a timezone offset, like "UTC-7", not a locale name like "America/Los_Angeles". We already attempt to read this value (D15962, T3025#176547) but when I wrote that stuff it was only available in Chrome and not necessarily reliable. We do try to guess the default zone selection based on the information we get from the browser, but currently ask the user to confirm it in all cases.

Jul 29 2017, 12:40 PM · Design, Phacility, NUX
avivey added a comment to T11456: Don't lose user in NUX flow because of Timezone issues.

I was thinking that the first time they log in, we just take the browser timezone and set the user config to that.

Jul 29 2017, 3:46 AM · Design, Phacility, NUX

Jul 22 2017

avivey renamed T12923: Settings Forms, EditEngine instructions are visually confusing from Forms instructions are visually confusing to Settings Forms instructions are visually confusing.
Jul 22 2017, 12:56 AM · Design

Jul 17 2017

chad added a comment to T12923: Settings Forms, EditEngine instructions are visually confusing.

image.png (892×978 px, 823 KB)

Jul 17 2017, 9:10 PM · Design
avivey created T12923: Settings Forms, EditEngine instructions are visually confusing.
Jul 17 2017, 12:25 AM · Design

Jul 14 2017

jasonrumney added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Since the History is now dumbed down and separated from the Technical details on the Graph view, it might be more useful to filter out merge commits from the History view. Perhaps they can be replaced by a small "Merged to <branch>" link at the bottom of the box for the last non-merge commit that was part of the merge.

Jul 14 2017, 6:11 AM · Diffusion, Design

Jun 27 2017

epriestley added a comment to T12840: (maybe) bring back commit graphs in history.

We use (most of) the graph code for drawing the "Task Graph" and the "Revision Graph" so we need to keep it around in the upstream anyway.

Jun 27 2017, 9:43 PM · Design, Diffusion
20after4 added a comment to T12840: (maybe) bring back commit graphs in history.

doh! nice. I obviously need more coffee.

Jun 27 2017, 9:39 PM · Design, Diffusion
epriestley added a comment to T12840: (maybe) bring back commit graphs in history.

("Graph" button in upper right.)

Jun 27 2017, 9:39 PM · Design, Diffusion
epriestley added a comment to T12840: (maybe) bring back commit graphs in history.

https://secure.phabricator.com/source/phabricator/history/master/

Jun 27 2017, 9:38 PM · Design, Diffusion
20after4 added a comment to T12840: (maybe) bring back commit graphs in history.

I know just maintaining the hooks is more work but certainly less than what's involved in maintaining the graph code.

Jun 27 2017, 9:38 PM · Design, Diffusion
epriestley added a comment to T12840: (maybe) bring back commit graphs in history.

I think that's what D18131 did -- or do you mean something else?

Jun 27 2017, 9:38 PM · Design, Diffusion
20after4 added a comment to T12840: (maybe) bring back commit graphs in history.

Maybe provide some extension hook-points in diffusion so that the community can maintain the commit graph visualization as an extension?

Jun 27 2017, 9:37 PM · Design, Diffusion

Jun 22 2017

epriestley closed T12859: Some means of updating builtin images without a manual purge as Resolved.

Ah, alright. Yeah, behavior is:

Jun 22 2017, 6:24 PM · Design
chad added a comment to T12859: Some means of updating builtin images without a manual purge.

This is great for testing!

Jun 22 2017, 6:21 PM · Design
chad added a comment to T12859: Some means of updating builtin images without a manual purge.

I'm fine with 7 days, I didn't think we had any auto-update.

Jun 22 2017, 6:21 PM · Design
epriestley reopened T12859: Some means of updating builtin images without a manual purge as "Open".
Jun 22 2017, 6:20 PM · Design
epriestley added a comment to T12859: Some means of updating builtin images without a manual purge.

Oh, maybe I misunderstood. I thought you wanted an easier way to test changes in a development environment, but it sounds like you actually want end users to immediately get the new images instead of needing to wait 7 days for them?

Jun 22 2017, 6:20 PM · Design
chad added a comment to T12859: Some means of updating builtin images without a manual purge.

I was wanted something I could trigger like celerity, generate a map hash or something. I presume users still need to run this command?

Jun 22 2017, 6:18 PM · Design
epriestley closed T12859: Some means of updating builtin images without a manual purge as Resolved by committing rP224c4692ee0e: Add a cache purger for builtin files.
Jun 22 2017, 6:14 PM · Design
epriestley added a comment to T12859: Some means of updating builtin images without a manual purge.

A slightly more aggressive fix would be to make these have a 5 second TTL in development or something (or hash the file modification time into the builtin key? But that's a bit tricky...) so you'd never have to purge the cache, but I think this arises rarely enough that it isn't too valuable.

Jun 22 2017, 5:58 PM · Design
epriestley added a comment to T12859: Some means of updating builtin images without a manual purge.

Strictly speaking, you still have to do a "manual purge" (with bin/cache ...) after D18146, but I assume you just didn't want to have to fish around in the DB.

Jun 22 2017, 5:57 PM · Design
epriestley added a revision to T12859: Some means of updating builtin images without a manual purge: D18147: Add a cache purger for builtin files.
Jun 22 2017, 5:56 PM · Design
epriestley added a revision to T12859: Some means of updating builtin images without a manual purge: D18146: Modularize "bin/cache" purgers.
Jun 22 2017, 5:44 PM · Design
chad renamed T12859: Some means of updating builtin images without a manual purge from Some means of purging/updating builtin images to Some means of updating builtin images without a manual purge.
Jun 22 2017, 7:13 AM · Design

Jun 21 2017

chad added a comment to T12859: Some means of updating builtin images without a manual purge.

So simple

Jun 21 2017, 8:24 PM · Design
epriestley added a comment to T12859: Some means of updating builtin images without a manual purge.
$engine = new PhabricatorDestructionEngine();
$engine->destroyObject($file);
Jun 21 2017, 8:13 PM · Design
chad added a revision to T12859: Some means of updating builtin images without a manual purge: D18143: New icons for repositories.
Jun 21 2017, 8:07 PM · Design
chad added a comment to T12859: Some means of updating builtin images without a manual purge.
In T12859#227762, @chad wrote:

image.png (288×1 px, 47 KB)

Jun 21 2017, 7:54 PM · Design
epriestley added a comment to T12859: Some means of updating builtin images without a manual purge.

Also not sure how to "retire" ones from Projects I don't intend to replace (low value).

Jun 21 2017, 6:57 PM · Design
chad added a comment to T12859: Some means of updating builtin images without a manual purge.

image.png (288×1 px, 47 KB)

Jun 21 2017, 6:45 PM · Design
chad created T12859: Some means of updating builtin images without a manual purge.
Jun 21 2017, 12:36 PM · Design

Jun 20 2017

chipironcin added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Ouch, sorry for the noise. I usually get confused between Diffusion and Differential.
Keep up the good work.

Jun 20 2017, 1:11 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

This task is for Diffusion feedback, not Audit or Differential

Jun 20 2017, 12:18 PM · Diffusion, Design
chipironcin added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

My two cents here:

Jun 20 2017, 11:48 AM · Diffusion, Design

Jun 19 2017

chad closed T12840: (maybe) bring back commit graphs in history as Resolved by committing rPd0898116d8c2: Add a graph view page to Diffusion.
Jun 19 2017, 3:57 PM · Design, Diffusion
chad closed T12840: (maybe) bring back commit graphs in history, a subtask of T12824: Redesign Diffusion, as Resolved.
Jun 19 2017, 3:57 PM · Design, Diffusion

Jun 18 2017

chad added a revision to T12840: (maybe) bring back commit graphs in history: D18131: Add a graph view page to Diffusion.
Jun 18 2017, 7:58 AM · Design, Diffusion

Jun 15 2017

jasonrumney added a comment to T12840: (maybe) bring back commit graphs in history.

+1 for bringing this back.

Jun 15 2017, 9:13 AM · Design, Diffusion

Jun 13 2017

chad added a comment to T12840: (maybe) bring back commit graphs in history.

(Fine to keep open so I don't forget)

Jun 13 2017, 9:44 PM · Design, Diffusion
chad added a comment to T12840: (maybe) bring back commit graphs in history.

There will be a place to get to this.

Jun 13 2017, 9:44 PM · Design, Diffusion
avivey added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

(I've extracted the graph issue to T12840)

Jun 13 2017, 9:42 PM · Diffusion, Design
avivey created T12840: (maybe) bring back commit graphs in history.
Jun 13 2017, 9:41 PM · Design, Diffusion
chad created T12838: Make sure "state" is in new Branch view.
Jun 13 2017, 8:04 PM · Design, Diffusion
avivey added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Equally valuable as "what branches contain that commit" is "what commits are in the branches we care about". For us, the answer to this is the commit graph -- it's a succinct, easy to understand representation of our branches and their relationships.

I get that if you only have one primary branch, there's not much value in the commit graph. Similarly, if you're not linearising history, your commit graph will be so messy as to be meaningless. But if you do have the requirement to actively maintain several releases of some B2B software package and have a well defined branching strategy for your team to make this happen, then the commit graph is meaningful and unexpected merges will stick out.

Jun 13 2017, 6:57 PM · Diffusion, Design
chad created T12837: Make it clear you're browsing the history of a tag.
Jun 13 2017, 6:30 PM · Design, Diffusion
cspeckmim added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

For perspective we use commit hooks to ensure that commits made in patch releases must exist in upstream major release first before being allowed in the respective patch branch. We then utilize herald to notify a group about suspicious merges or commits.

Jun 13 2017, 11:40 AM · Diffusion, Design
nickhutchinson added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think we have it documented somewhere that "we have developers who refuse to learn git" is considered a bad reason to ask for a feature, and you should maybe consider solving this issue with some workflow changes.
Note also that in your link, epriestley says

In projects which do not linearize history with arc land + squash merges, I believe this feature is virtually useless because many commits are merges and visualizing the repository isn't very informative (the history has so many parent/child relationships that none of them convey much of anything).

which can translate to "We don't understand exactly what you're trying to see - what's the root issue?"
From your description, it sounds like a listing of "is commit in branch" for a given commit and all/some branches would be more useful for you.

Jun 13 2017, 10:05 AM · Diffusion, Design
avivey added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think we have it documented somewhere that "we have developers who refuse to learn git" is considered a bad reason to ask for a feature, and you should maybe consider solving this issue with some workflow changes.
Note also that in your link, epriestley says

In projects which do not linearize history with arc land + squash merges, I believe this feature is virtually useless because many commits are merges and visualizing the repository isn't very informative (the history has so many parent/child relationships that none of them convey much of anything).

Jun 13 2017, 7:43 AM · Diffusion, Design
nickhutchinson added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I'd like the commit graph to come back in some form -- looks like it was removed in rPc5bb69fd7d79. For the reasons @epriestley outlined here: rPf2fcafb40dde#38208, this view is really useful for visualising the relationship between Git branches and sanity-checking recent merge commits. These are tasks that would otherwise require something like gitk.

Jun 13 2017, 7:28 AM · Diffusion, Design
chad created T12833: Credential workflow for URIs has spacing issues.
Jun 13 2017, 3:15 AM · Design, Diffusion
chad created T12832: Add autoclose table to Diffusion Manage.
Jun 13 2017, 2:11 AM · Design, Diffusion

Jun 12 2017

avivey added a comment to T12831: Update compare page with more features.

T929 is probably the earliest request for this feature :)

Jun 12 2017, 9:49 PM · Design, Diffusion
chad created T12831: Update compare page with more features.
Jun 12 2017, 8:59 PM · Design, Diffusion
epriestley added a comment to T12688: Inline Comment Design Opportunities for Improvement.

Not exactly related, but these cancel buttons used to be grey and are now blue:

Jun 12 2017, 4:57 PM · Design, Differential, Inline Comments
chad added a comment to T12823: Customization of Projects / Diffusion homepages.

Well let me actually re-read everything then.

Jun 12 2017, 4:03 PM · Design
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

Sorry, I guess I'm being unclear: I don't believe that (1) is a reasonable starting point.

Jun 12 2017, 4:00 PM · Design
chad added a comment to T12823: Customization of Projects / Diffusion homepages.

The flag color system seems like overkill though, I wonder if we can simplify that.

Jun 12 2017, 3:56 PM · Design
chad added a comment to T12823: Customization of Projects / Diffusion homepages.

I'm agreeing that (1) is a reasonable starting point.

Jun 12 2017, 3:55 PM · Design
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

I think maybe we're talking about a couple different things? These are both relatively easy:

Jun 12 2017, 3:54 PM · Design
chad added a comment to T12823: Customization of Projects / Diffusion homepages.

Have flags as an automatic search filter then like projects?

Jun 12 2017, 3:12 PM · Design
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

have a flagged filter automatically everywhere

Jun 12 2017, 3:05 PM · Design
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

Maybe also, in general, we could define an "explore by tags" view, that works something like this:

Jun 12 2017, 3:03 PM · Design
chad added a comment to T12823: Customization of Projects / Diffusion homepages.

Flags is something I definitely thought about, since it's already everywhere. If it integrated with applicationsearch (have a flagged filter automatically everywhere) I think that's reasonably handy and easy to build UI upon.

Jun 12 2017, 3:02 PM · Design
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

Constructively, some adjacent changes which are maybe worth considering instead:

Jun 12 2017, 2:54 PM · Design