Page MenuHomePhabricator
Feed Advanced Search

Jun 12 2017

chad closed T12825: Allow setting of an image per repository as Resolved by committing rP83a89166ee33: Add profile images to Repositories.
Jun 12 2017, 2:51 PM · Design, Diffusion
chad closed T12825: Allow setting of an image per repository, a subtask of T12824: Redesign Diffusion, as Resolved.
Jun 12 2017, 2:51 PM · Design, Diffusion
epriestley added a comment to T12823: Customization of Projects / Diffusion homepages.

The requests I've seen seem like users want this done automatically/by default, not that they want better tools to do it themselves.

Jun 12 2017, 2:41 PM · Design
chad added a revision to T12825: Allow setting of an image per repository: D18116: Add profile images to Repositories.
Jun 12 2017, 3:31 AM · Design, Diffusion

Jun 11 2017

chad added a comment to T12825: Allow setting of an image per repository.

Let's do both!

Jun 11 2017, 9:48 PM · Design, Diffusion
avivey added a comment to T12825: Allow setting of an image per repository.

Maybe every object should allow a cover image then?

Jun 11 2017, 9:40 PM · Design, Diffusion
chad added 1 mock(s) for T12824: Redesign Diffusion: M1477: Diffusion - June 2017.
Jun 11 2017, 8:29 PM · Design, Diffusion
chad added a revision to T12824: Redesign Diffusion: D18115: Add a DiffusionTagListView.
Jun 11 2017, 7:18 PM · Design, Diffusion
chad added a revision to T12824: Redesign Diffusion: D18113: Add DiffusionBranchListView for browsing branches.
Jun 11 2017, 7:18 PM · Design, Diffusion
chad created T12825: Allow setting of an image per repository.
Jun 11 2017, 7:18 PM · Design, Diffusion
chad added a subtask for T12824: Redesign Diffusion: T8407: Control diffusion's syntax highlighting.
Jun 11 2017, 7:16 PM · Design, Diffusion
avivey added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

also, T8407 maybe fits here? I think it's mostly a UI issue now.

Jun 11 2017, 7:16 PM · Diffusion, Design
chad added a parent task for T12804: Diffusion pre-redesign UI feedback (June 2017): T12824: Redesign Diffusion.
Jun 11 2017, 7:15 PM · Diffusion, Design
chad added subtasks for T12824: Redesign Diffusion: T12823: Customization of Projects / Diffusion homepages, T12804: Diffusion pre-redesign UI feedback (June 2017), T12752: Pager in Diffusion doesn't tell you what page you're on, T12749: Performance test and tune DiffusionHistoryListView.
Jun 11 2017, 7:15 PM · Design, Diffusion
chad added a parent task for T12823: Customization of Projects / Diffusion homepages: T12824: Redesign Diffusion.
Jun 11 2017, 7:15 PM · Design
chad added a parent task for T12752: Pager in Diffusion doesn't tell you what page you're on: T12824: Redesign Diffusion.
Jun 11 2017, 7:15 PM · Design, Diffusion
chad added projects to T12824: Redesign Diffusion: Diffusion, Design.
Jun 11 2017, 7:14 PM · Design, Diffusion
chad renamed T12823: Customization of Projects / Diffusion homepages from Customization of Projects ./ Diffusion homepages to Customization of Projects / Diffusion homepages.
Jun 11 2017, 5:42 PM · Design
chad created T12823: Customization of Projects / Diffusion homepages.
Jun 11 2017, 5:42 PM · Design

Jun 9 2017

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

good stuff, thanks!

Jun 9 2017, 2:00 AM · Diffusion, Design

Jun 8 2017

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

I resorted to hg log -r "descendants('XYZ123')"

Jun 8 2017, 10:45 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Maybe this was the workflow?

Jun 8 2017, 10:44 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Oh jump to the history via pagination and scroll to it? I don't know if that's technically possible. It's interesting though.

Jun 8 2017, 10:44 PM · Diffusion, Design
cspeckmim added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Hmm the history view on this install is significantly different than the one on mine~

Jun 8 2017, 10:41 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

The button. That says Browse. 💃

Jun 8 2017, 10:35 PM · Diffusion, Design
cspeckmim added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I just came across this today, though I can't recall a time in recent history of it coming up previously. I was looking at a commit in history and wanted to browse through history at that point - if there was a way to jump to the commit history at the point from a commit, would've helped. In this scenario I knew a commit was made shortly after the one I was tracing and trying to locate - that doesn't happen often though.

Jun 8 2017, 10:34 PM · Diffusion, Design
jcox added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

image.png (206×220 px, 65 KB)

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

@jcox ???

Jun 8 2017, 2:55 PM · Diffusion, Design
bjshively updated subscribers of T12804: Diffusion pre-redesign UI feedback (June 2017).

Screen Shot 2017-06-08 at 6.09.21 AM.png (255×1 px, 61 KB)

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

The pagination view for very large source trees/folders (in our case located at ourdomain.com/diffusion/name/browse/master/averylargefolder/) is confusing. Specifically, the pagination buttons are placed below related Open Revisions. This can often be confusing or misleading, as it is not immediately obvious that you're not seeing the full set of files.

It might be clearer if these buttons were immediately below the content being paginated (i.e. sandwiched between the averylargefolder list view and the Open Revisions.)

Jun 8 2017, 1:09 PM · Diffusion, Design
bjshively added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

The pagination view for very large source trees/folders (in our case located at ourdomain.com/diffusion/name/browse/master/averylargefolder/) is confusing. Specifically, the pagination buttons are placed below related Open Revisions. This can often be confusing or misleading, as it is not immediately obvious that you're not seeing the full set of files.

Jun 8 2017, 1:07 PM · Diffusion, Design
chad added 1 mock(s) for T12804: Diffusion pre-redesign UI feedback (June 2017): M1477: Diffusion - June 2017.
Jun 8 2017, 3:38 AM · Diffusion, Design
chad claimed T12688: Inline Comment Design Opportunities for Improvement.
Jun 8 2017, 1:27 AM · Design, Differential, Inline Comments
avivey added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think the Twitter server has some uncommon load issues, partly because a git at twitter has some unusual load issues (I know Twitter had to patch git locally at some point to make it work better). You should probably reach out to the internal Code Review team and complain again.

Jun 8 2017, 12:30 AM · Diffusion, Design

Jun 7 2017

lvital added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).
In T12804#226416, @chad wrote:

I don't believe moving buttons and changing layout will affect page load speed. If you have real performance issues, we should address those in another task.

Jun 7 2017, 10:42 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Though we could remove branches/tags on the initial home page of the repository, I'm only seeing about 500ms a load.

Jun 7 2017, 10:37 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I don't believe moving buttons and changing layout will affect page load speed. If you have real performance issues, we should address those in another task.

Jun 7 2017, 10:34 PM · Diffusion, Design
amckinley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I'd like to see little forward and back arrows at the top of the page when viewing actual commit objects. This use case might be unusual, but I've needed to find "the last commit before DXXX" and "the first commit after DYYY" a few times. GitHub doesn't appear to offer this either, but it feels a little easier than just providing links to the parent commit.

Jun 7 2017, 10:32 PM · Diffusion, Design
lvital added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think the majority of issues I (and many others at Twitter) run into is due to performance / slow page loads. If there is anything in the UI that could be minimized / simplified to speed up page loads, that would be really helpful. However, I imagine that performance is mainly a back-end issue (and probably our instance's fault in some ways).

Jun 7 2017, 10:31 PM · Diffusion, Design
bbrdaric added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think branches could be more accessible so I don't have to scroll to the bottom of the page, especially if my file tree is large.

Jun 7 2017, 10:15 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I think "search across multiple repos because we have 2,000 extensions" or whatever is a great use case, that just didn't seem to be the motivation of any of the requests above -- I think it wouldn't make sense to put cross-repo search on the main page of a particular repo, or particularly on subdirectory browse views.

Jun 7 2017, 9:34 PM · Diffusion, Design
20after4 added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

You can use, for example, -C3 with git grep to show 3 lines of Context around each match.

About 95% of the time when I'm searching for content it's because I want to edit a subset of the callsites. I do this with git grep some_function | maybe_more_grep | give, which opens all matches in my editor. This would take me about 100 years from the web UI for a nontrivial number of results. Do you have some other use case for searching for content?

Jun 7 2017, 9:24 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

My rough plan is to get rid of all the curtain stuff and move to button bars so the content can be full width. Owners stuff could get a new UI under the file content.

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

I just mean the curtain, yeah (It can be a largish chunk of the screen on a laptop)

Jun 7 2017, 9:18 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

It's limited if your company doesn't provide you with a 31" Dell monitor.

Jun 7 2017, 9:18 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Maybe this is a better screenshot:

Jun 7 2017, 9:17 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Slightly less important is that the width in the Browse File page is limited

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

When grepping in a repository, the results should (optionally) be ordered by their blame date.

Jun 7 2017, 9:15 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

bwahaha

Jun 7 2017, 9:14 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

u w0t m8

Jun 7 2017, 9:14 PM · Diffusion, Design
20after4 added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

When I view a repository, it would be incredibly helpful if the files I most recently touched were at the top of the UI.

Jun 7 2017, 9:10 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

When I view a repository, it would be incredibly helpful if the files I most recently touched were at the top of the UI.

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

(Some of this discussion is pertinent to T9287: Working with many repositories)

Jun 7 2017, 8:45 PM · Diffusion, Design
chad closed T12808: DarkConsole tables are now dark-on-grey as Resolved by committing rP345d80b40d9b: Clean up Dark Console broken table css.
Jun 7 2017, 8:42 PM · Design
mbm added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

What do you use file content search from the web UI for? (That is, in what situations is web UI search better than git grep?)

I execute content searches very frequently (hundreds per day?) but about 99.9% of them are git grep and maybe ~0.1% are web UI.

Jun 7 2017, 8:42 PM · Diffusion, Design
chad added a revision to T12808: DarkConsole tables are now dark-on-grey: D18099: Clean up Dark Console broken table css.
Jun 7 2017, 8:36 PM · Design
20after4 added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

What I would really love to see is global filename search. Currently you can search for a filename within one repository but you have to know which repo to look in first.

Jun 7 2017, 8:24 PM · Diffusion, Design
20after4 added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

We have thousands of repositories and it's hard to differentiate in a list which repository I need.

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

I would also say I use git grep locally probably 95% of the time, and web maybe 5%. It's not part of my usual routine.

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

I don't want to assume everyone is a knowledgable engineer who is sitting at a desktop with a cli and the code checked out. Diffusion to me is a web front end that is much more enjoyable, sometimes quicker, sometimes slower to browse and search for things in codebase that maybe aren't "I'm, coding right now" related?

Jun 7 2017, 8:19 PM · Diffusion, Design
ftdysa added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

What do you use file content search from the web UI for? (That is, in what situations is web UI search better than git grep?)

I execute content searches very frequently (hundreds per day?) but about 99.9% of them are git grep and maybe ~0.1% are web UI.

Jun 7 2017, 8:18 PM · Diffusion, Design
sascha-egerer added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

T8442 would help a lot for those "I have to many repositories"

Today, you could save searches in different Spaces. For example, here I've done this:

  • Search for "Spaces: Phacility".
  • Save the search as "Phacility Repositories".

Screen Shot 2017-06-07 at 11.08.35 AM.png (1×1 px, 193 KB)

Do you use this strategy today? How would T8442 improve on this? If you don't do this already, but dividing repositories by Space is useful, why don't you currently do it? Put another way, what problem exists with search today that T8442 solves but search changes could not solve?

Jun 7 2017, 8:14 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

You can use, for example, -C3 with git grep to show 3 lines of Context around each match.

Jun 7 2017, 8:08 PM · Diffusion, Design
chad added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I like GH search where I can see content before and after the term searched for. This has been helpful for migrations where I'm searching for something generic like addButton that is on multiple classes.

Jun 7 2017, 8:05 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

What do you use file content search from the web UI for? (That is, in what situations is web UI search better than git grep?)

Jun 7 2017, 8:04 PM · Diffusion, Design
ftdysa added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I'd be happy if you just added Grep File Contents search at the base repository view. It is kinda annoying that I have to click into a sub-folder to grep (sometimes I don't remember which folder it is in).

Jun 7 2017, 8:00 PM · Diffusion, Design
mbm added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Maybe I'm misunderstanding what you mean by "omnibox"? How do you use the search input to search for anything other than file content?

Jun 7 2017, 7:50 PM · Diffusion, Design
chad added a comment to T12808: DarkConsole tables are now dark-on-grey.

💯

Jun 7 2017, 7:44 PM · Design
epriestley created T12808: DarkConsole tables are now dark-on-grey.
Jun 7 2017, 7:43 PM · Design
richardvanvelzen added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Thanks - I thought it was already filed but couldn't seem to find it.

Jun 7 2017, 7:24 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

(I think that's also filed in T10767.)

Jun 7 2017, 7:19 PM · Diffusion, Design
richardvanvelzen added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I'd love folder shortcuts (for directories that only contain a single directory). Clicking through a few levels in various Java projects would be slightly more pleasant.

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

Have a most recently accessed repo list in there somewhere. Right now the order is always listed by most recently updated repo shows at the top. Maybe a additional column in there that shows most frequented repos? or maybe a "favorites" list for repos. A user can have access to many repos but not use them frequently, so pushing them towards the bottom.

Jun 7 2017, 6:44 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

T8442 would help a lot for those "I have to many repositories"

Jun 7 2017, 6:10 PM · Diffusion, Design
epriestley added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

GitHub, which has an omnibox

Jun 7 2017, 6:07 PM · Diffusion, Design
mbm added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

I find it really annoying how there isn't a real repository search (grep files) before you go click on "Browse". Generally speaking, search in Diffusion is very confusing with three different search fields in comparison to GitHub, which has an omnibox.

Jun 7 2017, 5:23 PM · Diffusion, Design
sascha-egerer added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

T8442 would help a lot for those "I have to many repositories"

Jun 7 2017, 4:22 PM · Diffusion, Design
chad moved T7701: Surface Code/File search better in Diffusion from Backlog to UI Refresh on the Diffusion board.
Jun 7 2017, 3:01 AM · Restricted Project, Design, Diffusion
chad moved T12752: Pager in Diffusion doesn't tell you what page you're on from Backlog to UI Refresh on the Diffusion board.
Jun 7 2017, 2:55 AM · Design, Diffusion
chad moved T12804: Diffusion pre-redesign UI feedback (June 2017) from Backlog to UI Refresh on the Diffusion board.
Jun 7 2017, 2:55 AM · Diffusion, Design
chad claimed T12804: Diffusion pre-redesign UI feedback (June 2017).
Jun 7 2017, 2:53 AM · Diffusion, Design
chad created T12804: Diffusion pre-redesign UI feedback (June 2017).
Jun 7 2017, 2:49 AM · Diffusion, Design

Jun 4 2017

chad updated the image for Design from F1064783: profile to F4988944: profile.
Jun 4 2017, 8:38 PM

May 30 2017

epriestley merged T12774: No margin after inline comment reply button into T12688: Inline Comment Design Opportunities for Improvement.
May 30 2017, 12:32 PM · Design, Differential, Inline Comments

May 26 2017

chad renamed T12763: Have custom icons made? Object + State? from Have custom icons made for object + state to Have custom icons made? Object + State?.
May 26 2017, 3:38 PM · Design
chad updated the task description for T12763: Have custom icons made? Object + State?.
May 26 2017, 1:36 AM · Design
chad added a comment to T12763: Have custom icons made? Object + State?.

Or maybe even more specifically, I want to be able to tell a story on each commit in History. From start to finish every place code went or came from before it ended up in the repository.

May 26 2017, 12:39 AM · Design
chad added a project to T12763: Have custom icons made? Object + State?: Design.
May 26 2017, 12:23 AM · Design

May 25 2017

amckinley closed T6906: Archived / closed objects don't have "closed" look in typeahead results, in global search typeahead only as Resolved by committing rPf29b54944f9c: Make closed objects in global typeahead look closed.
May 25 2017, 8:44 PM · Typeahead, Contributor Onboarding, Design
amckinley added a revision to T6906: Archived / closed objects don't have "closed" look in typeahead results, in global search typeahead only: D18017: Make closed objects in global typeahead look closed.
May 25 2017, 12:28 AM · Typeahead, Contributor Onboarding, Design

May 24 2017

epriestley added a comment to T12752: Pager in Diffusion doesn't tell you what page you're on.

😢

May 24 2017, 8:40 PM · Design, Diffusion
chad updated subscribers of T12752: Pager in Diffusion doesn't tell you what page you're on.
May 24 2017, 8:38 PM · Design, Diffusion
epriestley added a comment to T12752: Pager in Diffusion doesn't tell you what page you're on.

hahaha

May 24 2017, 8:38 PM · Design, Diffusion
avivey added a comment to T12752: Pager in Diffusion doesn't tell you what page you're on.

Does it at least tell you how fast you're going?

May 24 2017, 8:37 PM · Design, Diffusion
chad created T12752: Pager in Diffusion doesn't tell you what page you're on.
May 24 2017, 5:38 PM · Design, Diffusion
chad created T12750: Somehow get rid of double "Description" header on Projects.
May 24 2017, 5:14 PM · Design, Projects

May 23 2017

chad closed T12670: Workboard action UI needs improvements as Resolved.

I think I shipped this.

May 23 2017, 9:17 PM · Customer Impact, Design, Workboards

May 22 2017

20after4 added a comment to T8646: Provide more context for search results, particularly wiki documents.

FWIW I partially implemented this in Wikimedia's fork, and I did so in a reusable way. I'd like to eventually upstream it but I'm not sure that my approach is desired upstream. I'll give it another shot though.

May 22 2017, 3:51 AM · Design, Search, Restricted Project

May 18 2017

epriestley moved T12688: Inline Comment Design Opportunities for Improvement from Glitches to Future on the Inline Comments board.
May 18 2017, 5:49 PM · Design, Differential, Inline Comments

May 17 2017

epriestley added a comment to T12688: Inline Comment Design Opportunities for Improvement.

In this screenshot:

May 17 2017, 2:37 PM · Design, Differential, Inline Comments