New issue
Advanced search Search tips

Issue 723925 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Evaluate mobile viewer UI and viewing experience.

Project Member Reported by thakis@chromium.org, May 18 2017

Issue description

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.linux%2Flinux_chromium_chromeos_dbg_ng%2F1139%2F%2B%2Frecipes%2Fsteps%2Fcompile__with_patch_%2F0%2Fstdout doesn't load more than 232 lines of output for me. On phone, in a train. Connectivity isn't awesome but also not horrible.
 

Comment 1 by d...@chromium.org, May 18 2017

So I think that you're experiencing a thing that I put in to try and not be a jerk to mobile users. On mobile devices, the log viewer loads the first set of logs, but defaults "automatic loading" to OFF so that huge logs don't tear through mobile data. If you click the "play" button in the upper-left, this should load the remaining logs.

Comment 2 by thakis@chromium.org, May 18 2017

It seems to load now, thanks. I'd be surprised if the number of people who click a logdog link but then don't want to see the contents is greater than 0 though -- did anyone ask for that?

Comment 3 by d...@chromium.org, May 18 2017

Nope, just something I thought would be a good idea. "See the contents" isn't really prevented here - just burst loading of hundreds of megabytes by default. In this mode, you can scroll around, iteratively load logs, load the tail log and work from the bottom, or load the full log. The mobile mode just asks you to state your intent rather than presuming it.

Comment 4 by thakis@chromium.org, May 18 2017

In a way that I didn't understand and that slowed .e down enough that I bothered to file a bug :-)

I don't know of any other webapp that stops loading in some unfinished state on mobile. I'd suggest removing this again.

Comment 5 by d...@chromium.org, May 18 2017

Yeah, the UI definitely needs some improvement, and could present the situation better.

I appreciate the feedback, and I'll keep that in mind for future design iterations. For the time being, though, I'm planning on keeping this behavior.

Comment 6 by d...@chromium.org, May 18 2017

Labels: Pri-3
Summary: Evaluate mobile viewer UI and viewing experience. (was: Fails to load more than 232 lines of output)

Comment 7 by estaab@chromium.org, May 18 2017

Is there a way I can see this on desktop? I tried using dev tools in phone mode but I get the full log still.

I was going to suggest showing the button at the bottom of the loaded text where they'll be looking when it stops loading, similar to where a "next" or "load all" button may be.

Agree on this being pri-3 though.

Comment 8 by estaab@chromium.org, Jun 22 2017

Status: Available (was: Untriaged)
Project Member

Comment 9 by sheriffbot@chromium.org, Jun 22 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Recharge-Cold Type-Task
Status: Available (was: Untriaged)
Owner: hinoka@chromium.org
Status: Fixed (was: Available)
Viewer should work as well as if you were loading a page of text now (which isn't particularly great on mobile anyways, but we're talking about logs here, so that can't be helped)

Sign in to add a comment