Commit 591d6267 authored by Noel Gordon's avatar Noel Gordon Committed by Commit Bot

[quickview] Add a delete-confirm-dialog styling TODO

The <dialog> element added in CL:2033012 is too dark. We should restyle
its CSS ::backdrop property in quick-view.css to lighten it up

   dialog#delete-confirm-dialog::backdrop {
     ...
   }

Add TODO item for that task.

Tbr: alex
Bug: 803259
Change-Id: Id8f2408f6d9d6cc02f612e1794a52e54dcd2351d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2035493
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737994}
parent b612f753
...@@ -278,6 +278,7 @@ class QuickViewController { ...@@ -278,6 +278,7 @@ class QuickViewController {
// Create a delete confirm dialog if needed. // Create a delete confirm dialog if needed.
if (!this.deleteConfirmDialog_) { if (!this.deleteConfirmDialog_) {
// TODO(crbug.com/803259): style this element to make it lighter.
const dialogElement = document.createElement('dialog'); const dialogElement = document.createElement('dialog');
this.quickView_.shadowRoot.appendChild(dialogElement); this.quickView_.shadowRoot.appendChild(dialogElement);
dialogElement.id = 'delete-confirm-dialog'; dialogElement.id = 'delete-confirm-dialog';
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment