Page MenuHomePhabricator
Feed Advanced Search

Jan 26 2015

chad added a project to T7042: JX.Scrollbar z-index puts it over dialogs: PHUI.
Jan 26 2015, 4:11 PM · PHUI

Jan 24 2015

chad edited projects for T7027: "Leaving high security" page looks a bit off on mobile, added: PHUI; removed Design.
Jan 24 2015, 11:52 PM · PHUI

Jan 19 2015

chad closed T6963: Phriction action menu doesn't display on mobile as Resolved by committing Restricted Diffusion Commit.
Jan 19 2015, 8:42 PM · Phriction, PHUI

Jan 17 2015

chad updated the task description for T5752: Navigation menus should work correctly on mobile by default.
Jan 17 2015, 11:04 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad renamed T5752: Navigation menus should work correctly on mobile by default from Navigation menus and action items should work correctly on mobile by default to Navigation menus should work correctly on mobile by default.
Jan 17 2015, 11:04 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a revision to T6963: Phriction action menu doesn't display on mobile: Restricted Differential Revision.
Jan 17 2015, 4:54 PM · Phriction, PHUI

Jan 14 2015

chad closed T6926: Reorder Columns looks bad as Resolved by committing Restricted Diffusion Commit.
Jan 14 2015, 12:29 AM · PHUI, Design
chad added a revision to T6926: Reorder Columns looks bad: Restricted Differential Revision.
Jan 14 2015, 12:28 AM · PHUI, Design
chad closed T6964: Title is cut off by action menu as Resolved by committing Restricted Diffusion Commit.
Jan 14 2015, 12:11 AM · PHUI, Mobile
chad added a revision to T6964: Title is cut off by action menu: Restricted Differential Revision.
Jan 14 2015, 12:07 AM · PHUI, Mobile

Jan 13 2015

chad edited projects for T6964: Title is cut off by action menu, added: PHUI; removed Design.
Jan 13 2015, 9:02 PM · PHUI, Mobile
chad created T6963: Phriction action menu doesn't display on mobile.
Jan 13 2015, 7:32 PM · Phriction, PHUI
chad moved T6926: Reorder Columns looks bad from Backlog to Short Term on the Design board.
Jan 13 2015, 12:43 AM · PHUI, Design

Jan 12 2015

chad added a revision to T5752: Navigation menus should work correctly on mobile by default: Restricted Differential Revision.
Jan 12 2015, 4:19 AM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Oh silly, I see we're passing PropertyList into ObjectBoxView anyways. Alright, should be able to figure this out after kiddo bath time.

Jan 12 2015, 2:25 AM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Still not sure how to do this, initial plan was to just push the button up into the header, but I forgot that headers already have their own buttons (not often, but somethings like on the current project homepage). It's still likely the best place for it, just not sure how to make this auto-magical.

Jan 12 2015, 2:10 AM · ApplicationEditor, Infrastructure, PHUI, Mobile

Jan 11 2015

epriestley added a comment to T5752: Navigation menus should work correctly on mobile by default.

Yeah, I think it's just:

Jan 11 2015, 7:17 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Is this just a hidden button on ActionListView that gets some display: block and negative margin when on mobile? I could probably build that.

Jan 11 2015, 7:15 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Yeah I think #2 is better.

Jan 11 2015, 6:23 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley added a comment to T5752: Navigation menus should work correctly on mobile by default.

Alternatively could have a little menu on PHUIObjectBoxView appear on each section, though I can't imagine how to make that happen automatically.

Jan 11 2015, 6:19 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley added a comment to T5752: Navigation menus should work correctly on mobile by default.

Specifically, you want things like "Edit Task" in the application menu?

Jan 11 2015, 6:16 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Alternatively could have a little menu on PHUIObjectBoxView appear on each section, though I can't imagine how to make that happen automatically.

Jan 11 2015, 6:08 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

I'd want to move these out of crumbs and into ApplicationMenu by default, that is any action link added to a page, also gets built into the mobile menu. Further since some pages have multiple sections of action links, we'd need maybe some label or dividers automatically built as well.

Jan 11 2015, 6:02 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad claimed T5752: Navigation menus should work correctly on mobile by default.
Jan 11 2015, 5:56 PM · ApplicationEditor, Infrastructure, PHUI, Mobile

Jan 9 2015

chad triaged T6926: Reorder Columns looks bad as Normal priority.
Jan 9 2015, 11:31 PM · PHUI, Design
chad created T6926: Reorder Columns looks bad.
Jan 9 2015, 11:19 PM · PHUI, Design
chad renamed T6913: Make all "Quick Add" links be dialogs? from Make all "Quick Add" links be dialogs to Make all "Quick Add" links be dialogs?.
Jan 9 2015, 4:57 AM · PHUI
chad removed a parent task for T6913: Make all "Quick Add" links be dialogs?: T5116: Amazon SES mailer does not allow specifying endpoint.
Jan 9 2015, 4:56 AM · PHUI
• elite.liao added a parent task for T6913: Make all "Quick Add" links be dialogs?: T5116: Amazon SES mailer does not allow specifying endpoint.
Jan 9 2015, 3:40 AM · PHUI

Jan 8 2015

chad added a comment to T6913: Make all "Quick Add" links be dialogs?.

I mean more from a psychological point of view it feels faster. The UX now is page load (new task) page load (submitted task), so 2 page loads to submit a new task. Where with a dialog (even is same end to end time) the user only experiences one page load change (submitted task).

Jan 8 2015, 11:54 PM · PHUI
epriestley added a comment to T6913: Make all "Quick Add" links be dialogs?.

When loaded as raw pages, the "New Conpherence" page is >2x faster for me than the "New Task" page:

Jan 8 2015, 11:48 PM · PHUI
chad added a comment to T6913: Make all "Quick Add" links be dialogs?.

I guess Conpherence only exists as a dialog.

Jan 8 2015, 11:47 PM · PHUI
chad added a comment to T6913: Make all "Quick Add" links be dialogs?.

Should I say it feels quicker to be a dialog? I have no actual data on the speed, just that I was very happy with the UEX of the Conpherence Dialog.

Jan 8 2015, 11:45 PM · PHUI
epriestley added a comment to T6913: Make all "Quick Add" links be dialogs?.

Basically, the proposed solution increases the complexity of the codebase (all the create pages need to have dialog rendering paths) and only improves the situation if the page frame has a significant performance issue. If it does, it sweeps it under the rug in a non-general way that applies to only one relatively rare workflow.

Jan 8 2015, 11:43 PM · PHUI
epriestley added a comment to T6913: Make all "Quick Add" links be dialogs?.

The cost of the page frame should be so small that it never motivates a page -> dialog change. If it isn't, this task should be "remedy performance problems of page frame", or if we can't we should merge it into T2086.

Jan 8 2015, 11:42 PM · PHUI
chad added a comment to T6913: Make all "Quick Add" links be dialogs?.

(do we fake it?)

Jan 8 2015, 11:34 PM · PHUI
chad added a comment to T6913: Make all "Quick Add" links be dialogs?.

Is it? I'd assume a dialog at least isn't building an entire new page?

Jan 8 2015, 11:33 PM · PHUI
epriestley added a comment to T6913: Make all "Quick Add" links be dialogs?.

Isn't it exactly the same speed?

Jan 8 2015, 11:32 PM · PHUI
chad claimed T6913: Make all "Quick Add" links be dialogs?.
Jan 8 2015, 11:21 PM · PHUI

Dec 22 2014

chad merged task T6798: Pull-down menu on comments simply fails when not logged in into T6594: Logged-out users can not access "View Raw" in comment dropdown menus.
Dec 22 2014, 4:50 PM · PHUI
chad added a comment to T6798: Pull-down menu on comments simply fails when not logged in.

There is already a task for this somewhere... if I can find it.

Dec 22 2014, 4:48 PM · PHUI
chad raised the priority of T6798: Pull-down menu on comments simply fails when not logged in from Low to Needs Triage.
Dec 22 2014, 4:46 PM · PHUI

Dec 10 2014

chad added a comment to T6700: Opening new tabs from search dropdown results.

We should make middle clicks work as expected though, regardless.

Dec 10 2014, 9:41 PM · PHUI, Search
chad added a comment to T6700: Opening new tabs from search dropdown results.

We already have planned several UI improvements to Typeaheads, see T1906 and T6579.

Dec 10 2014, 9:40 PM · PHUI, Search
tgr added a comment to T6700: Opening new tabs from search dropdown results.

A more specific use case: when I am adding a new task and am unsure whether it fits into a certain project, typing that project name in the search box and middle-clicking would be a convenient way of checking that project's description.

Dec 10 2014, 9:29 PM · PHUI, Search
chad added a revision to T6723: Titles of objects in lists should wrap: Restricted Differential Revision.
Dec 10 2014, 5:25 PM · PHUI, Design
chad added a revision to T6723: Titles of objects in lists should wrap: Restricted Differential Revision.
Dec 10 2014, 4:40 PM · PHUI, Design
chad closed T6723: Titles of objects in lists should wrap as Invalid.

Basically, we clip (actually we use ellipsis if you're at HEAD) overly long titles. If you make a dashboard and put content heavy items in a the narrow column on a very narrow screen, this is what can happen. We have alternate displays for tablet and mobile widths.

Dec 10 2014, 3:23 PM · PHUI, Design
chad edited projects for T6723: Titles of objects in lists should wrap, added: Design, PHUI; removed Phabricator.
Dec 10 2014, 3:04 PM · PHUI, Design

Dec 9 2014

chad triaged T6700: Opening new tabs from search dropdown results as Low priority.
Dec 9 2014, 3:20 PM · PHUI, Search

Dec 3 2014

chad added a comment to T6680: Tokenizer input field loses focus after tab completion.

tab and enter both seem correct / commonly used in this UEX, at least from my limited testing of a few mail clients, gmail, facebook, etc.

Dec 3 2014, 11:31 PM · PHUI
sophiebits added a comment to T6680: Tokenizer input field loses focus after tab completion.

I don't think expecting people to hit enter is entirely reasonable – I'm personally scared that it'll submit the form, which is what enter ordinarily does in a form (except in a textarea).

Dec 3 2014, 9:15 PM · PHUI
csilvers added a comment to T6680: Tokenizer input field loses focus after tab completion.

Oh! I didn't think of hitting enter (I'm used to tab-completing instead).

Dec 3 2014, 9:13 PM · PHUI
epriestley added a comment to T6680: Tokenizer input field loses focus after tab completion.

For me, focus goes to the next input, which seems correct. That is:

Dec 3 2014, 12:03 PM · PHUI
chad renamed T6680: Tokenizer input field loses focus after tab completion from Text input field loses focus after tab completion to Tokenizer input field loses focus after tab completion.
Dec 3 2014, 3:54 AM · PHUI

Nov 26 2014

chad closed T6644: Actions menu <button> rendering issue in Firefox as Resolved.

Closed by commit rPfa77fdd168d4.

Nov 26 2014, 2:09 AM · PHUI
chad added a revision to T6644: Actions menu <button> rendering issue in Firefox: Restricted Differential Revision.
Nov 26 2014, 12:08 AM · PHUI

Nov 25 2014

chad claimed T6644: Actions menu <button> rendering issue in Firefox.
Nov 25 2014, 11:50 PM · PHUI

Nov 24 2014

chad renamed T6616: Typeahead uses locale-specific comparison from Typeahead uses locale-specific comparison, i.e. projects never showing up and being unselectable to Typeahead uses locale-specific comparison.
Nov 24 2014, 9:16 PM · Typeahead, PHUI
epriestley added a comment to T6616: Typeahead uses locale-specific comparison.

For mixed-language data, surely collating in the user's locale is preferable to collating in the default locale for the install? An example of this type of data is user "Real Names", which show up when trying to typeahead users, but contain letters from different languages with some frequency. This specific type of data is present to some degree on most installs, but there's no single valid language for it, since user human names are written in a variety of languages.

Nov 24 2014, 7:40 PM · Typeahead, PHUI
matmarex added a comment to T6616: Typeahead uses locale-specific comparison.

I want the strings to collate as the user expects strings in their language to collate, even if the strings are not actually written in their language. This will produce the best result if the strings are written in their language, and a reasonable (consistent with expectations) result if the strings are written in some other language or a mixture of languages.

Nov 24 2014, 7:30 PM · Typeahead, PHUI
epriestley added a comment to T6616: Typeahead uses locale-specific comparison.

I don't know if that assumption (that browser locale settings are reliable) actually is reasonable. T1888 documents a case where toLocaleDateString() did something a user found surprising/undesirable.

Nov 24 2014, 7:22 PM · Typeahead, PHUI
epriestley added a comment to T6616: Typeahead uses locale-specific comparison.

If you really want as-good-as-possibly locale-insensitive collation

Nov 24 2014, 7:15 PM · Typeahead, PHUI
chad renamed T6616: Typeahead uses locale-specific comparison from The dropdown for project names uses locale-specific comparison when sorting the projects, which can lead to weird results (projects never showing up and being unselectable) to Typeahead uses locale-specific comparison, i.e. projects never showing up and being unselectable.
Nov 24 2014, 7:04 PM · Typeahead, PHUI

Nov 18 2014

epriestley added a subtask for T6579: Display hovercards on Tokens in Typeaheads: T3670: Add subprojects.
Nov 18 2014, 4:32 PM · Hovercards, PHUI
epriestley added a comment to T6579: Display hovercards on Tokens in Typeaheads.

Per T5403, I agree that pursuing richer results is probably something we should do first, since it would be better to give you the information you need to make this choice in the menu. I'm going to mark T3670 as blocking this since we'll need to reexamine some of the design there anyway, and I think that will also tend to make this choice easier by providing more context.

Nov 18 2014, 4:32 PM · Hovercards, PHUI
chad renamed T6579: Display hovercards on Tokens in Typeaheads from Display hovercards on Tokens in Typeaheads? to Display hovercards on Tokens in Typeaheads.
Nov 18 2014, 4:16 PM · Hovercards, PHUI
chad renamed T6579: Display hovercards on Tokens in Typeaheads from Tooltips for projects to Display hovercards on Tokens in Typeaheads?.
Nov 18 2014, 4:13 PM · Hovercards, PHUI

Nov 6 2014

btrahan closed T6465: Suggestions are not coming while creating herald rule as Wontfix.
In T6465#82339, @chad wrote:

Our datasource typeaheads are all able to be called without javascript. For example, the repository typeahead (like in your Herald image) is here:

https://secure.phabricator.com/typeahead/class/DiffusionRepositoryDatasource/

Or as JSON:

https://secure.phabricator.com/typeahead/class/DiffusionRepositoryDatasource/?__ajax__=true

Nov 6 2014, 8:25 PM · PHUI, Herald

Nov 5 2014

chad updated subscribers of T6465: Suggestions are not coming while creating herald rule.

I don't know if @epriestley has other thoughts, but sounds suspiciously like your webserver is sending the incorrect JS files, or old JS files (either by caching or other mechanic).

Nov 5 2014, 5:20 PM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

We don't even use getPreventDefault that I can find. https://github.com/phacility/phabricator/search?utf8=%E2%9C%93&q=getPreventDefault&type=Code

Nov 5 2014, 6:44 AM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Nop, versions are

Nov 5 2014, 6:37 AM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

Are you using Beta browsers?

Nov 5 2014, 6:23 AM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

We have two errors while inspecting the element. Checking on this

Nov 5 2014, 6:22 AM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Ok, we shall check this.

Nov 5 2014, 6:06 AM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

Our datasource typeaheads are all able to be called without javascript. For example, the repository typeahead (like in your Herald image) is here:

Nov 5 2014, 5:54 AM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

sadpanda

Nov 5 2014, 5:07 AM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Ok, Chad. We shall continue to figure it out. Thanks for your support.

Nov 5 2014, 5:06 AM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

Unfortunately, we cannot reproduce this and have no other reports right now. Happy to leave this open, but we don't have any additional thoughts. See https://secure.phabricator.com/book/phabricator/article/feedback/#unreproducible-problems for more information.

Nov 5 2014, 4:42 AM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Yes. Typeahead is coming with Maniphest and other tools. It was working prior to the up-gradation of phabricator installation on October 24 around 3 a.m EST.

Nov 5 2014, 4:37 AM · PHUI, Herald
• unais updated subscribers of T6465: Suggestions are not coming while creating herald rule.
Nov 5 2014, 4:29 AM · PHUI, Herald

Nov 4 2014

chad added a comment to T6465: Suggestions are not coming while creating herald rule.

and Herald is the only Typeahead you are having issues with?

Nov 4 2014, 5:44 PM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Thank You,
*UNAIS A*
Cloud Engineer

Nov 4 2014, 5:38 PM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Hi Chad,

Nov 4 2014, 5:35 PM · PHUI, Herald
chad added a comment to T6465: Suggestions are not coming while creating herald rule.

I don't have any other ideas. I'd try the following:

Nov 4 2014, 5:25 PM · PHUI, Herald
• unais added a comment to T6465: Suggestions are not coming while creating herald rule.

Thanks for looking into this, Chad. I think this is not a browser issue. I am able to see the suggestions on a local installation using the same browser

Nov 4 2014, 4:52 PM · PHUI, Herald
chad added projects to T6465: Suggestions are not coming while creating herald rule: Herald, PHUI.
Nov 4 2014, 4:37 PM · PHUI, Herald