New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 756237 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Cache view UX issues

Project Member Reported by eostroukhov@chromium.org, Aug 16 2017

Issue description

1. Toolbar should be at the top.
2. JSON preview needs padding.
 
Suggested updated UI:
CacheUX.png
92.5 KB View Download
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/793e8467869415cc6c07a6fde151a1c49e10c600

commit 793e8467869415cc6c07a6fde151a1c49e10c600
Author: Eugene Ostroukhov <eostroukhov@chromium.org>
Date: Thu Aug 17 20:49:09 2017

[DevTools] Improve cache view

1. Response column was removed.
2. Request column was changed to a "path" column and no longer show
useless domain.
3. Toolbar was moved to the top.
4. JSON preview now has padding on the sides.

Bug:  756237 
Change-Id: I1a9461ef65b3c96ff755a8c8998d808ca9e56776
Reviewed-on: https://chromium-review.googlesource.com/618090
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Eugene Ostroukhov <eostroukhov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495311}
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/LayoutTests/http/tests/inspector/cache-storage/cache-data-expected.txt
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/LayoutTests/http/tests/inspector/cache-storage/cache-deletion-expected.txt
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/LayoutTests/http/tests/inspector/cache-storage/cache-entry-deletion-expected.txt
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/BUILD.gn
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/front_end/network/networkPanel.css
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkerCacheViews.js
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/front_end/source_frame/JSONView.js
[add] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/front_end/source_frame/jsonView.css
[modify] https://crrev.com/793e8467869415cc6c07a6fde151a1c49e10c600/third_party/WebKit/Source/devtools/front_end/source_frame/module.json

Status: Fixed (was: Assigned)

Sign in to add a comment