New issue
Advanced search Search tips

Issue 828930 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

<text> hit-testing doesn't work if fill == none but descendants have fill !=none

Project Member Reported by f...@opera.com, Apr 4 2018

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Apr 9 2018

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

commit 90a5f0bd2096695800bcd2ec0bb6f1eed4d8a7eb
Author: Fredrik Söderquist <fs@opera.com>
Date: Mon Apr 09 11:45:21 2018

[PE] Don't early-out in LayoutSVGText::NodeAtFloatPoint

Since <text> can have descendants that may both differ in visibility as
well as fill/stroke, we can't evaluate pointer-events in LayoutSVGText,
but should rather let LayoutBlock (and SVGInlineTextBox et al) handle
the hit query. This makes LayoutSVGText somewhat similar to the same
method on LayoutSVGContainer.

Bug:  828930 
Change-Id: I2767ed149705289e770d5ae165efd910c2c7ff15
Reviewed-on: https://chromium-review.googlesource.com/998352
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549154}
[add] https://crrev.com/90a5f0bd2096695800bcd2ec0bb6f1eed4d8a7eb/third_party/WebKit/LayoutTests/svg/hittest/text-fill-none-with-nested-tspan.html
[add] https://crrev.com/90a5f0bd2096695800bcd2ec0bb6f1eed4d8a7eb/third_party/WebKit/LayoutTests/svg/hittest/text-visibility-hidden-with-nested-tspan.html
[modify] https://crrev.com/90a5f0bd2096695800bcd2ec0bb6f1eed4d8a7eb/third_party/blink/renderer/core/layout/svg/layout_svg_text.cc

Comment 2 by f...@opera.com, Apr 9 2018

Status: Fixed (was: Assigned)

Sign in to add a comment