I'm so stoked about all these badges I'll be getting.

T9132 has a tangential interaction here: Differential saves action state, as well as comment state, and it would be nice to do this generally. After T9132, any generalization here can reasonably also persist action state in a general way.

I can't find any way to reproduce or fix this.

What are you trying to accomplish?

I can't reproduce this on any Android or iOS version of Chrome I have. Page loads instantly.

The same thing happens when trying to select a different encoding.

Just sharing my workaround. I'd like something better too, whatever that is.

Yeah that works but I feel like making the file view a bit more responsive would help as well.

(tap Download)

Was it easier if you turned your phone to landscape?

no worries! wasn't easy for me to find either

Thank you for merging.
(Looked for relation "mobile" "workboard" but did not find anything related.
This is exactly the required solution, thank you and sorry for the duplicate.)

We can't reproduce this on similar hardware/software, manage this stuff primarily with pure CSS (which makes it less likely that this is a bug we can fix), and it sounds like it's easy to avoid (don't do somersaults while using Phabricator), and to work around (reload the page). So we don't really have a path forward here, don't have a real reason to believe it's fixable, and the impact seems very small.

I see the same issue on my nexus 5 on 5.1 chrome beta 43

Tested Nexus 4 with Chrome 40 / Android 5.0 then updated to Chrome 42 / Android 5.1, wasn't able to get a reproducible case. I do see it stick for like a fraction of a second, then it snaps into place each time for me.

Quite possibly.

Honestly, it doesn't look like a Phabricator bug.

My User Agent is Mozilla/5.0 (Linux; Android 5.1; Nexus 5 Build/LMY47I) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.111 Mobile Safari/537.36

Doesn't reproduce for me on Chrome/iOS or Safari/iOS.

I like at least trying the last item as a dropdown. There are a couple of other similar "make UI elements work better on mobile by default" things too (T5752) that I sort of want to clean up, but holding these until after the redesign probably makes sense.

Oh, I know why. We fixed Differential to 1-up view and that was the only example.

Yea... I don't see it anywhere

You can make this one King if you find it first!

But in the other task is there a fancy video? :D

(this is a regression in Chrome)

There is already a task for this somewhere, but I'm having issues finding it.

I'm just going to merge this into T1562, I don't expect to really touch this page before that obsoletes it.

display: inline-table; mwuahahahaha

I hear CSS4 has white-space: nowrap-first-space-but-wrap-the-rest-normally;.

Actually I'm just guessing on that. I really don't know. Obviously the answer is to just make these tables.

I think previously these did used to break at the first word, we used to absolutely position the icon, but moved to a more flexible inline icon.

Particularly after T3670, these might get a lot longer than they are right now in reasonable cases.

Particularly after T3670, these might get a lot longer than they are right now in reasonable cases.

If you add nowrap to the tag, it will get cut off by overflow:hidden when the tag is too long for the bounding box, or break the bounding box. This UI is the lesser of two evils.

Personally, I felt that (at least in the case of my screenshot) it would be better to push the entire tag to the second line.

The options are 1) break the tag or 2) don't break the tag and have overflow hide part of it.