Page MenuHomePhabricator
Feed Advanced Search

Mar 19 2016

chad closed T7290: Mobile CSS doesn't always load properly on the home page as Wontfix.

Nothing seems actionable here, closing.

Mar 19 2016, 1:02 AM · Javelin, Mobile

Mar 17 2016

epriestley added a comment to T6027: Way to edit which column a task is in from the task itself.

Is there any chance that this feature will be implemented sometime soon?

Mar 17 2016, 10:16 PM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest
rshuler added a comment to T6027: Way to edit which column a task is in from the task itself.

We recently switched from GitHub / HuBoard to Phabricator, and overall I'm really happy, but the one thing that I (and the other engineers on my team) sorely miss is the ability to change columns from within the task. It is really awkward to comment on the task but not be able to change columns without swiching back to the workboard.

Mar 17 2016, 9:57 PM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Mar 15 2016

chad added a comment to T10385: Diffusion code view doesn't hide blame pane.

I spent like 20 minutes on this in the redesign and couldn't figure it out.

Mar 15 2016, 4:18 PM · Diffusion (v3), Mobile

Feb 19 2016

chad added a comment to T10400: Dropdown menus can overlap.

What is your expectation of what should happen here? This doesn't seem like a Workboard issue, just an issue with menus in general. Should we just automatically close any open ones if you change your mind and open a different one?

Feb 19 2016, 10:31 PM · PHUI, Bug Report
joshuaspence created T10400: Dropdown menus can overlap.
Feb 19 2016, 10:23 PM · PHUI, Bug Report
epriestley added a project to T10385: Diffusion code view doesn't hide blame pane: Diffusion (v3).
Feb 19 2016, 1:09 PM · Diffusion (v3), Mobile
chad added a comment to T10385: Diffusion code view doesn't hide blame pane.

tumblr_nnhhzusKSV1uu9z7xo1_400.gif (225×400 px, 485 KB)

Feb 19 2016, 5:42 AM · Diffusion (v3), Mobile
avivey added a comment to T10385: Diffusion code view doesn't hide blame pane.

It also doesn't go away on desktop, I just never tried before (Latest Chrome, android and windows).

Feb 19 2016, 5:14 AM · Diffusion (v3), Mobile
avivey renamed T10385: Diffusion code view doesn't hide blame pane from Diffusion code view doesn't hide blame pane on mobile to Diffusion code view doesn't hide blame pane.
Feb 19 2016, 5:13 AM · Diffusion (v3), Mobile
avivey created T10385: Diffusion code view doesn't hide blame pane.
Feb 19 2016, 4:06 AM · Diffusion (v3), Mobile

Jan 29 2016

aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Works for me on the iPad Pro. Thank you!

Jan 29 2016, 7:02 PM · Safari, Mobile, Bug Report
chad added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Seems OK for me on a physical iPhone 4s.

Jan 29 2016, 4:02 PM · Safari, Mobile, Bug Report
epriestley added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

This is live on this server (secure.phabricator.com) -- let us know if you're still seeing issues. Seems OK for me on a physical iPhone 4s.

Jan 29 2016, 2:59 PM · Safari, Mobile, Bug Report
epriestley closed T10229: In iOS Safari, buttons must be tapped twice to fire. as Resolved by committing rPbf43d4cf2aa6: Don't mutate DOM on touch-originated cursor events in Differential.
Jan 29 2016, 2:55 PM · Safari, Mobile, Bug Report
epriestley claimed T10229: In iOS Safari, buttons must be tapped twice to fire..
Jan 29 2016, 2:47 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Yep and to be honest this huge iPad is fairly desktop-y. The first time I saw it I kind of had to look twice to make sure it wasn't some run away item from Alice in Wonderland that had snuck into my reality.

Jan 29 2016, 2:20 PM · Safari, Mobile, Bug Report
epriestley added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

D15136 distinguishes between "touch-device" mouseovers and "mouse-device" mouseovers rather than relying on device detection, since the screen resolution of the iPad Pro is apparently a billion pixels by a trillion pixels and device detection is clearly increasingly hopeless.

Jan 29 2016, 1:51 PM · Safari, Mobile, Bug Report
epriestley added a revision to T10229: In iOS Safari, buttons must be tapped twice to fire.: D15136: Don't mutate DOM on touch-originated cursor events in Differential.
Jan 29 2016, 1:49 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

I gave this solution a quick whirl and it works. It preserves mouse over behaviour in regular browsers, and removes the need for double tapping in iOS (while still getting the selection highlights). Let me know if you'd like a patch or if you prefer your device detection approach.

Jan 29 2016, 1:31 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Yeah. On a pure touch device, mouseover is not the most useful event to listen for when it comes to highlighting rows.

Jan 29 2016, 1:15 PM · Safari, Mobile, Bug Report
epriestley added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Nice work! I arrived at the same conclusion shortly after you did.

Jan 29 2016, 1:09 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

I have found the bug. For iOS Safari to send a click event, the handlers of mouseover and mousemove must not change the document content. (Presumably to allow a touch user to reveal things that require mousing over, without actually clicking things.)

Jan 29 2016, 1:02 PM · Safari, Mobile, Bug Report
epriestley added a revision to T10229: In iOS Safari, buttons must be tapped twice to fire.: D15135: Improve reliability of detecting small devices when loading Differential changesets.
Jan 29 2016, 1:00 PM · Safari, Mobile, Bug Report
epriestley reopened T10229: In iOS Safari, buttons must be tapped twice to fire. as "Open".

I can reproduce this on my iPhone 4s which correctly selects 1-up mode, so I agree that this is probably not a duplicate of T9969.

Jan 29 2016, 12:57 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Okay I don't think it's related to the 1up vs 2up renderer. Reading the source code I found out that the "1up" renderer seems to be triggered for certain screen widths. So I used an iOS Split View to reduce the width of my browser until it was fairly narrow, and as result I got a unified diff instead, which I believe is the "1up" view.

Jan 29 2016, 12:04 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

Not sure what the "1-up renderer" is but if it's 1 vs 2 columns, that's not the problem really. The formatting of the diff is fine (it's a large, desktop size screen).

Jan 29 2016, 10:02 AM · Safari, Mobile, Bug Report

Jan 28 2016

chad merged task T10229: In iOS Safari, buttons must be tapped twice to fire. into T9969: Differential 1-up render doesn't trigger on iOS.
Jan 28 2016, 5:14 AM · Safari, Mobile, Bug Report

Jan 27 2016

chad updated subscribers of T10229: In iOS Safari, buttons must be tapped twice to fire..

Maaaybe an issue with how we detect and decide to use the 1-up renderer? cc @epriestley

Jan 27 2016, 6:46 PM · Safari, Mobile, Bug Report
chad added a project to T10229: In iOS Safari, buttons must be tapped twice to fire.: Safari.
Jan 27 2016, 6:43 PM · Safari, Mobile, Bug Report
chad added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..

I think this is a duplicate of T9969. Basically between Safari 9.1 and 9.2 somewhere, it now reports a "diff" page to us as a desktop page, where if we normally detect being on a phone or tablet, we do not trigger tooltips, hovers, etc.

Jan 27 2016, 6:41 PM · Safari, Mobile, Bug Report
aljungberg added a comment to T10229: In iOS Safari, buttons must be tapped twice to fire..
In T10229#155876, @chad wrote:

I can't say I have tried every button, but the same behaviour affects even buttons without tooltips.

Do you mind testing other buttons, by other buttons I mean any form anywhere on Phabricator that is not on the "diff" page.

Jan 27 2016, 6:40 PM · Safari, Mobile, Bug Report
chad added a project to T10229: In iOS Safari, buttons must be tapped twice to fire.: Mobile.
Jan 27 2016, 6:37 PM · Safari, Mobile, Bug Report

Jan 19 2016

chad updated the image for Mobile from F1064800: profile to F1064802: profile.
Jan 19 2016, 10:03 PM
chad updated the image for Mobile from F139423: profile-project.png to F1064800: profile.
Jan 19 2016, 10:03 PM

Dec 31 2015

DocTaxon added a comment to T6027: Way to edit which column a task is in from the task itself.
In T6027#135704, @isfs wrote:

Would it perhaps be simpler if instead of developing a "Change columns" UI, actions can be added to move columns, something more along the lines of what I mocked up for T9280, i.e. a simple action with two parameters, rather than a monster action with parameters for all existing boards?:

mock.png (400×800 px, 27 KB)

If you want to move the task on multiple boards (which I imagine is relatively, if not absolutely rare) you would just add multiple actions. The order of the actions is clear, even if not mutable.

Dec 31 2015, 12:57 AM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Dec 22 2015

epriestley moved T6027: Way to edit which column a task is in from the task itself from Backlog to Task Relationships on the Maniphest board.
Dec 22 2015, 1:00 AM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Dec 10 2015

epriestley closed T9132: Build an ApplicationEditor abstraction, a subtask of T5031: Draft Comment Persists After Submission, as Resolved.
Dec 10 2015, 1:43 PM · Restricted Project, Support Impact, Mobile

Dec 9 2015

epriestley added a comment to T5031: Draft Comment Persists After Submission.

Broadly, if comments have the new "stacked action" UI (an "Add Action..." dropdown, allowing you to take multiple actions at once), the expectation is that they support the new drafts too. This includes Maniphest at HEAD.

Dec 9 2015, 1:05 PM · Restricted Project, Support Impact, Mobile

Dec 5 2015

chad closed T9910: All Columns are as long as the longest column on the board. Due to this the mobile view has a lot of spaces after each column. as Invalid.

Can't reproduce anything here, with Workboards/Mobile. If this was a real bug report, please follow Contributing Bug Reports and re-open.

Dec 5 2015, 4:45 PM · Mobile
chad added a comment to T9910: All Columns are as long as the longest column on the board. Due to this the mobile view has a lot of spaces after each column..

Are you testing Phabricator or trying to file a bug?

Dec 5 2015, 5:37 AM · Mobile
rohanmahadar created T9910: All Columns are as long as the longest column on the board. Due to this the mobile view has a lot of spaces after each column..
Dec 5 2015, 5:35 AM · Mobile
angie moved T5031: Draft Comment Persists After Submission from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Dec 5 2015, 5:09 AM · Restricted Project, Support Impact, Mobile

Dec 3 2015

epriestley closed T5031: Draft Comment Persists After Submission as Resolved by committing rPb82863d972a5: Implement versioned drafts in EditEngine comment forms.
Dec 3 2015, 3:07 PM · Restricted Project, Support Impact, Mobile

Dec 2 2015

epriestley added a comment to T5031: Draft Comment Persists After Submission.

I've marked D14637 as fixing this.

Dec 2 2015, 8:59 PM · Restricted Project, Support Impact, Mobile
epriestley added a revision to T5031: Draft Comment Persists After Submission: D14640: Implement versioned drafts in EditEngine comment forms.
Dec 2 2015, 8:58 PM · Restricted Project, Support Impact, Mobile

Dec 1 2015

chad merged T9877: Unable to get back to the build plan view page from the build step view page on mobile into T8175: Mobile always collapses long crumbs.
Dec 1 2015, 5:28 AM · Mobile, PHUI

Nov 5 2015

chad removed a project from T9714: Mobile view of diff changes has dialog that doesn't contain full changes: Differential.
Nov 5 2015, 2:35 PM · Mobile
joshuaspence added a project to T9714: Mobile view of diff changes has dialog that doesn't contain full changes: Differential.
Nov 5 2015, 8:28 AM · Mobile

Nov 4 2015

chad added a comment to T9714: Mobile view of diff changes has dialog that doesn't contain full changes.

To be fair, the "description" of T9065 is just an image, which isn't helpful for searching.

Nov 4 2015, 8:25 PM · Mobile
cspeckmim added a comment to T9714: Mobile view of diff changes has dialog that doesn't contain full changes.

Ach! Just FYI I missed T9065 with a search for "mobile diff"

Nov 4 2015, 8:25 PM · Mobile
chad updated subscribers of T9065: Show Changes dialog isn't mobile friendly.
Nov 4 2015, 8:24 PM · Mobile, Transactions
chad merged task T9714: Mobile view of diff changes has dialog that doesn't contain full changes into T9065: Show Changes dialog isn't mobile friendly.
Nov 4 2015, 8:24 PM · Mobile
cspeckmim created T9714: Mobile view of diff changes has dialog that doesn't contain full changes.
Nov 4 2015, 8:23 PM · Mobile

Nov 3 2015

epriestley closed T5752: Navigation menus should work correctly on mobile by default as Resolved by committing rP300c74c49dc9: Make mobile navigation work properly by default in more cases.
Nov 3 2015, 6:11 PM · ApplicationEditor, Infrastructure, PHUI, Mobile

Nov 2 2015

epriestley added a revision to T5752: Navigation menus should work correctly on mobile by default: D14382: Make mobile navigation work properly by default in more cases.
Nov 2 2015, 8:20 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
chad added a comment to T5752: Navigation menus should work correctly on mobile by default.

Happy to help with conversions if that's how this gets solved.

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

Actually I guess (2) already got pretty much fixed in D11340.

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

There are broadly two issues here:

Nov 2 2015, 6:12 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley claimed T5752: Navigation menus should work correctly on mobile by default.
Nov 2 2015, 6:10 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley moved T5752: Navigation menus should work correctly on mobile by default from Backlog to v1 on the ApplicationEditor board.
Nov 2 2015, 6:06 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley added a project to T5752: Navigation menus should work correctly on mobile by default: ApplicationEditor.
Nov 2 2015, 6:05 PM · ApplicationEditor, Infrastructure, PHUI, Mobile

Oct 27 2015

revi added a comment to T9642: Unable to download images on Android.

Android 4.4.2 (also happens on 4.1 but I know it's too old, and I will replace this 4.1 next month...)
Google Chrome for Android 46.0.2490.76

Oct 27 2015, 2:20 PM · Android, Files
chad added a comment to T9642: Unable to download images on Android.

https://secure.phabricator.com/book/phabcontrib/article/bug_reports/ has a bit more detail, we're trying to get the full steps to reproducing bugs when filed now so we can reduce back and forth time.

Oct 27 2015, 2:18 PM · Android, Files
chad added a comment to T9642: Unable to download images on Android.

I don't know what you're screenshot is supposed to tell me. Can you be more specific about reproduction instructions (OS, Browser, expected results). This behaves as expected on iOS Chrome and Safari.

Oct 27 2015, 2:15 PM · Android, Files
joshuaspence renamed T9642: Unable to download images on Android from Unable to download.image to Unable to download image.
Oct 27 2015, 12:55 PM · Android, Files
joshuaspence created T9642: Unable to download images on Android.
Oct 27 2015, 12:48 PM · Android, Files

Oct 20 2015

KennyWZhang added a member for Mobile: KennyWZhang.
Oct 20 2015, 3:23 PM

Oct 9 2015

honza added a comment to T6027: Way to edit which column a task is in from the task itself.

It would be great to have this in Maniphest - whatever way there is...

Oct 9 2015, 11:17 AM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Sep 18 2015

vintrax added a comment to T6027: Way to edit which column a task is in from the task itself.

I would like to confirm that it would be a great addition to be able to move a task to a different column within the edit mode.
Especially if you're working within the workboard view, it would be efficient to not need to explicitly move the task to another column, right after the task has been edited within the inline popup (or vice versa).

Sep 18 2015, 1:58 PM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Sep 9 2015

epriestley added a comment to T5031: Draft Comment Persists After Submission.

This is fairly straightforward but blocked on T9132, which is prioritized elsewhere. This is likely to be completed adjacently with T9132 anyway.

Sep 9 2015, 11:59 PM · Restricted Project, Support Impact, Mobile

Sep 8 2015

Florian added a comment to T6027: Way to edit which column a task is in from the task itself.

That would make this function at least useable in some way, even if it would be a bit complicated if a task has several (5+) projects assigned (which is, what you said already, very rare).

Sep 8 2015, 4:28 AM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest
isfs added a comment to T6027: Way to edit which column a task is in from the task itself.

Would it perhaps be simpler if instead of developing a "Change columns" UI, actions can be added to move columns, something more along the lines of what I mocked up for T9280, i.e. a simple action with two parameters, rather than a monster action with parameters for all existing boards?:

Sep 8 2015, 2:33 AM · Restricted Project, Prioritized, Restricted Project, Mobile, Workboards, Maniphest

Sep 3 2015

chad closed T9314: Legalpad document title overflows screen as Resolved by committing rP29399849c0ed: Scroll on overflow of property list (mobile).
Sep 3 2015, 6:06 PM · Restricted Project, Differential, Mobile
chad closed T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll as Resolved by committing rP42ed5241204d: Break long strings in all remarkup areas.
Sep 3 2015, 6:03 PM · Mobile, Ponder

Sep 2 2015

chad added a revision to T9314: Legalpad document title overflows screen: D14036: Scroll on overflow of property list (mobile).
Sep 2 2015, 1:31 AM · Restricted Project, Differential, Mobile

Sep 1 2015

joshuaspence added a comment to T9314: Legalpad document title overflows screen.

This might be a more general issue:

Sep 1 2015, 10:07 PM · Restricted Project, Differential, Mobile
joshuaspence created T9314: Legalpad document title overflows screen.
Sep 1 2015, 10:02 PM · Restricted Project, Differential, Mobile
epriestley added a revision to T5752: Navigation menus should work correctly on mobile by default: D14032: Allow Controllers to return a wider range of "response-like" objects.
Sep 1 2015, 5:26 PM · ApplicationEditor, Infrastructure, PHUI, Mobile
epriestley added a parent task for T5752: Navigation menus should work correctly on mobile by default: T9132: Build an ApplicationEditor abstraction.
Sep 1 2015, 5:22 PM · ApplicationEditor, Infrastructure, PHUI, Mobile

Aug 31 2015

chad added a revision to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll: D14018: Break long strings in all remarkup areas.
Aug 31 2015, 7:59 PM · Mobile, Ponder
chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

gotcha, I can repro these and fix easy.

Aug 31 2015, 7:36 PM · Mobile, Ponder
cspeckmim added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

I saw this on Q100 and Q105. On some questions everything did seem to layout properly, such as Q114. If all content is supposed to wrap (to not require horizontal scroll) then it might be the long links on those Questions.

Aug 31 2015, 6:36 PM · Mobile, Ponder
chad reopened T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll as "Open".
Aug 31 2015, 6:32 PM · Mobile, Ponder
chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

What question is it specifically?

Aug 31 2015, 6:32 PM · Mobile, Ponder
chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

Oh, maybe just Safari Mobile bug?

Aug 31 2015, 6:32 PM · Mobile, Ponder
cspeckmim added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

I noticed T9278 was landed and checked here at secure.phabricator.com, so I thought I'd check this out on mobile again. I cleared website history & data from Safari and it looks like this was still happening. Was this put up on the install yet?

Aug 31 2015, 6:30 PM · Mobile, Ponder

Aug 29 2015

chad added a comment to T9016: Weird CSS issue with inline comments.

We'll give you a badge if you complete the task. The problem is easy tasks are easy, so we just usually do them.

Aug 29 2015, 12:58 AM · Badge Awarded, Differential, PHUI, Mobile
joshuaspence added a comment to T9016: Weird CSS issue with inline comments.

Lol

Aug 29 2015, 12:35 AM · Badge Awarded, Differential, PHUI, Mobile

Aug 26 2015

chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

Oh yeah, I'm not sure if this install has that patch. It seems ok on my sandbox, but we can verify.

Aug 26 2015, 2:44 AM · Mobile, Ponder
cspeckmim added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

I noticed this here on secure.phabricator about an hour ago. I haven't tested it on my own install yet (which was only updated to stable on Saturday).

Aug 26 2015, 2:36 AM · Mobile, Ponder
chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

I presumed this was fixed with D13904 but sounds like it didn't get all cases?

Aug 26 2015, 2:34 AM · Mobile, Ponder
cspeckmim updated subscribers of T9239: Cannot scroll horizontally on wide questions.
Aug 26 2015, 2:32 AM · Mobile, Ponder
cspeckmim merged task T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll into T9239: Cannot scroll horizontally on wide questions.
Aug 26 2015, 2:32 AM · Mobile, Ponder
cspeckmim added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

Sorry this looks like dup of T9239. I missed it in the search.

Aug 26 2015, 2:31 AM · Mobile, Ponder
chad added a comment to T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

You're not supposed to scroll, pre should be wrapping.

Aug 26 2015, 2:30 AM · Mobile, Ponder
cspeckmim created T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.
Aug 26 2015, 2:29 AM · Mobile, Ponder

Aug 24 2015

jbeta added a comment to T9016: Weird CSS issue with inline comments.

Well, due to recent shortages of warm expressions of user gratitude, the upstream has had to resort to providing their own positive feedback. They achieve this by self-awarding cool-looking badges for commits.

Aug 24 2015, 10:49 AM · Badge Awarded, Differential, PHUI, Mobile
joshuaspence added a comment to T9016: Weird CSS issue with inline comments.

What does Badge Awarded mean?

Aug 24 2015, 10:18 AM · Badge Awarded, Differential, PHUI, Mobile

Aug 21 2015

epriestley updated subscribers of T5031: Draft Comment Persists After Submission.
Aug 21 2015, 5:34 PM · Restricted Project, Support Impact, Mobile