Bug 211555 - [GTK] REGRESSION(r260996): Pressing <Tab> in an HTTP authentication dialog makes the dialog lose focus
Summary: [GTK] REGRESSION(r260996): Pressing <Tab> in an HTTP authentication dialog ma...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-07 01:21 PDT by Adrian Perez
Modified: 2020-05-07 07:26 PDT (History)
6 users (show)

See Also:


Attachments
Patch (2.09 KB, patch)
2020-05-07 06:24 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Perez 2020-05-07 01:21:09 PDT
This can be reproduced easily:

 1. Navigate to https://webmail.igalia.com
 2. The HTTP authentication dialog will show up, with the “Username”
    field focused.
 3. Press <Tab>.

Expected outcome: The “Pasword” field gets the focus, pressing <Tab>
  more time focuses the dialog buttons and then the dialog input fields
  again.

Actual outcom: The HTTP authentication dialog loses the focus, if one
  keeps pressing <Tab> the dialog does not seem to ever get the focus
  back. Interacting with the dialog using the mouse is still possible,
  though.
Comment 1 Adrian Perez 2020-05-07 01:22:15 PDT
Possible related to the changes introduced with bug #211283
Comment 2 Carlos Garcia Campos 2020-05-07 06:22:59 PDT
Yes, I made the focus vfunc GTK4 only by mistake
Comment 3 Carlos Garcia Campos 2020-05-07 06:24:30 PDT
Created attachment 398729 [details]
Patch
Comment 4 EWS Watchlist 2020-05-07 06:25:16 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment 5 Adrian Perez 2020-05-07 06:34:16 PDT
(In reply to Carlos Garcia Campos from comment #3)
> Created attachment 398729 [details]
> Patch

Nice 🤩️
Comment 6 EWS 2020-05-07 07:26:17 PDT
Committed r261284: <https://trac.webkit.org/changeset/261284>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 398729 [details].