Update Progress Numerical % Indicator Missing
Reported by
obaykal...@gmail.com,
Jul 8 2016
|
||||||||
Issue descriptionThe numerical percentage process indicator while performing a device update, is missing, and is replaced by a single dot.
,
Jul 13 2016
The C++ logic for communicating progress percentage to the UI has been migrated, see https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/settings/about_handler.cc?l=581,597. So the only thing missing is the UI surface to display it. @tbuckley: Where should the indicator be displayed in the new UI? In the old Options it was under the spinner (see attachement).
,
Jul 14 2016
Either next to the spinning indicator or perhaps inside the spinning indicator.
,
Jul 14 2016
How about just in parentheses next to "Updating your device"? Eg. "Updating your device (25%)"
,
Jul 25 2016
Issue 631239 has been merged into this issue.
,
Aug 8 2016
,
Sep 9 2016
if we have good product/UX requirements around this, I'll implement the suggestion in comment 4
,
Sep 9 2016
Note that the new UI also fails to display the platform version, which is sometimes the only part that an update changes. Old UI shows it, new UI does not (and only shows the browser version, which is not the whole story)
,
Sep 10 2016
bsittler@: what do you mean by platform version? can you upload a screenshot or two showing what you mean? fyi: I'm also adding the progress percentage to the message that shows up when you switch channels (i.e. Updating your device to stable (90%))
,
Sep 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8b9bbf145d2b0051285fdde3a81c5d6fda014baf commit 8b9bbf145d2b0051285fdde3a81c5d6fda014baf Author: dbeam <dbeam@chromium.org> Date: Sat Sep 10 02:11:33 2016 MD Settings/About: Implement update progress percentage R=hcarmona@chromium.org BUG= 626508 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2325193002 Cr-Commit-Position: refs/heads/master@{#417801} [modify] https://crrev.com/8b9bbf145d2b0051285fdde3a81c5d6fda014baf/chrome/app/settings_chromium_strings.grdp [modify] https://crrev.com/8b9bbf145d2b0051285fdde3a81c5d6fda014baf/chrome/app/settings_google_chrome_strings.grdp [modify] https://crrev.com/8b9bbf145d2b0051285fdde3a81c5d6fda014baf/chrome/app/settings_strings.grdp [modify] https://crrev.com/8b9bbf145d2b0051285fdde3a81c5d6fda014baf/chrome/browser/resources/settings/about_page/about_page.js [modify] https://crrev.com/8b9bbf145d2b0051285fdde3a81c5d6fda014baf/chrome/test/data/webui/settings/about_page_tests.js
,
Sep 12 2016
Screenshot attached, platform version is selected/highlighted. Likewise the ARC version and firmware version can change without impacting browser version. Please consider showing all the versions, just as the existing UI does.
,
Sep 12 2016
Also, what is "Detailed build information" supposed to do? Right now it seems to be completely ignored.
,
Sep 12 2016
i'm not really sure what "Platform ##.#.#.#" means on the about page here. seems ChromeOS-y.
,
Sep 12 2016
Yes, it's the OS version for ChromeOS. Sometimes the update applied through this UI changes the OS version but not the browser version. Likewise ARC is the Android container version for ChromeOS, and Firmware is the device firmware version for ChromeOS.
,
Dec 9 2016
@dbeam, do you know if the progress indicator has been finished? We now have all of the Chrome OS-specific platform details discussed in this thread, so the update % is the only piece remaining.
,
Dec 9 2016
bsittler@: this bug was about progress status % while updating. please file a new bug for any other cros-specific issues on the about page
,
Dec 10 2016
dbeam@ - all the info seems to be there now, so no bugs to file :)
,
Dec 15 2016
Verified on ChromeOS 9086.0.0, 57.0.2951.0. Progress of download is shown "Updating your device (x%)" |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by tbuck...@chromium.org
, Jul 13 2016Labels: Hotlist-MD-Settings-About
Status: Available (was: Unconfirmed)