Issue metadata
Sign in to add a comment
|
Regression : Unable to close Help Overlay in EULA screen |
||||||||||||||||||||||
Issue descriptionChrome Version: 59.0.3062.0/9430.1.0 dev-channel Daisy,Peppy and Minnie OS: Chrome What steps will reproduce the problem? (1)Recover build -> In EULA screen Click on "Learn more" link (2)Now try to close Help overlay and observe (Please refer video) Expected: should be able to close Help overlay Actual: Instead Unable to close Help Overlay This is Regression issue as same is working fine in 59.0.2055.0/9429.0.0 dev-channel Daisy @alemate : please confirm the Issue
,
Apr 7 2017
Issue 708818 has been merged into this issue.
,
Apr 11 2017
This is help app, not just overlay. Passing to morsed@. David, do you have any ideas on what could be wrong here?
,
Apr 11 2017
The HelpApp provides the "Diagnostics and Usage Data" header text through the "Need More Help" footer box. The wrapping window with the "Help" title bar and close "x" are controlled by something else. I've only looked at this element briefly and have not yet determined where the code for the wrapper and title bar resides. I look at it because there is a related bug (https://bugs.chromium.org/p/chromium/issues/detail?id=665753) requesting the "Help" text be made larger, but the tickets is on hold while the final design issues are worked out. I cannot duplicate this error (708455) with official beta or dev builds. I powerwashed and reviewed the following. Both closed as expected when clicking the "x". I opened and closed the "Learn More" overlay twice for each. Google Chrome 58.0.3029.51 (Official Build) beta (32-bit) Platform 9334.33.0 (Official Build) beta-channel veyron_minnie Google Chrome 59.0.3054.0 (Official Build) dev (32-bit) Platform 9413.0.0 (Official Build) dev-channel veyron_minnie Note that M59 version I tested is "3054," so perhaps something was changed between that and "3062" reported in this bug.
,
Apr 11 2017
CCing weifangsun@, since she's handling the other ticket related to this same element that I mentioned above in Comment #4.
,
Apr 18 2017
Any updates? Is this stil seen on latest tot builds? M59 goes beta in ~1 week
,
Apr 18 2017
This issue is no longer seen on latest tot builds. I think we can close it.
,
Apr 18 2017
,
Apr 18 2017
Marking this as fixed. Please reopen if it occurs again.
,
Apr 18 2017
,
Apr 18 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-59; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-59 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 18 2017
,
Apr 19 2017
,
Apr 24 2017
,
Apr 24 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-59; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-59 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 24 2017
,
May 8 2017
9532.0.0, 60.0.3092.0 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by r...@chromium.org
, Apr 7 2017