MD Extensions: detail_view page missing warning/corruption message and repair/reload button |
||||
Issue descriptionSee title.
,
Sep 7 2017
,
Oct 2 2017
,
Oct 4 2017
Attached is screenshot of the implemented UI for patch 3 (https://chromium-review.googlesource.com/c/chromium/src/+/695641/2..3) changes.
,
Oct 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5078a2973e873020ea8b35ce4e54cc4c10696207 commit 5078a2973e873020ea8b35ce4e54cc4c10696207 Author: Catherine Mullings <catmullings@chromium.org> Date: Tue Oct 10 22:12:03 2017 Extensions: Add style changes for warnings in md-extensions detail page Style for corrupted, blacklisted, and suspicious warnings are missing from the md-extensions details view page. This CL adds the necessary style changes to support warnings in the details view page, excluding style for the "Learn more" link associated with suspicious warnings. Bug: 760322 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I486ca6e3810e4ad7655e2f659e6d5514ca65ca3c Reviewed-on: https://chromium-review.googlesource.com/695641 Commit-Queue: catmullings <catmullings@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#507788} [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/app/md_extensions_strings.grdp [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/browser/resources/md_extensions/detail_view.html [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/browser/resources/md_extensions/detail_view.js [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/browser/ui/webui/extensions/extensions_ui.cc [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/test/data/webui/extensions/cr_extensions_browsertest.js [modify] https://crrev.com/5078a2973e873020ea8b35ce4e54cc4c10696207/chrome/test/data/webui/extensions/extension_detail_view_test.js
,
Oct 10 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by scottchen@chromium.org
, Sep 7 2017