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

Issue 660806 link

Starred by 0 users
Status: Fixed
Owner:
█ kochi@chromium.org
Last visit > 30 days ago
Closed: Sep 2017
Cc:
█ elkurin@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Fix focus navigation for nested fallback slots case

Project Member Reported by kochi@chromium.org, Oct 31 2016 
While focus navigation for nested slots is fixed in  issue 618587 ,
nested fallback slots case is not fixed yet, as it uses different
code path.

As the fallback slot case is usually rare (not useful), the priority
is low and issue is separated from  issue 618587 .

Comment 1 by kochi@chromium.org, Aug 30 2017

Cc: elkurin@google.com
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 4 2017 

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

commit 0cdb56c9b89f57abc8f42f753f35aab929a6f471
Author: Eriko Kurimoto <elkurin@google.com>
Date: Mon Sep 04 10:13:06 2017

Add focus navigation test with nested slots and fallback contents

Bug:  660806 
Change-Id: Ie8a56b9376f2dfdda1e85efc1b18cf28baf1469a
Reviewed-on: https://chromium-review.googlesource.com/644671
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Eriko Kurimoto <elkurin@google.com>
Cr-Commit-Position: refs/heads/master@{#499470}
[add] https://crrev.com/0cdb56c9b89f57abc8f42f753f35aab929a6f471/third_party/WebKit/LayoutTests/shadow-dom/focus-navigation-slot-nested-fallback.html

Comment 3 by kochi@chromium.org, Sep 5 2017

Status: Fixed (was: Assigned)
Looks like we don't have the issue in the current implementation. Closing.

Comment 4 by benhenry@chromium.org, Sep 29 2017

Components: Blink>HTML>Focus

Comment 5 by benhenry@chromium.org, Sep 29 2017

Components: -Blink>Focus
►

Sign in to add a comment