Page MenuHomePhabricator

Display client hint data in Special:CheckUser's 'Get edits' mode result lines
Closed, ResolvedPublic

Description

Client hint data should be displayed in Special:CheckUser's 'Get edits' mode. An example of the results from this mode is shown below.

image.png (281×882 px, 137 KB)

Following T345764: Design UI for displaying client hint data, the design decisions were made to use the following for the 'Get edits' lines:

Log-line.png (1×1 px, 261 KB)

Acceptance criteria
  • Have Client Hints data be displayed in Special:CheckUser 'Get edits' mode for each result line

QA Results - Local

Event Timeline

To display client hints data appropriately, the new query builder needs to be used. This is so that the client hints data can be selected for log events. Without the use of the query builder, the client hint data will only be displayable for edits.

Change 956457 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] clienthints: Display data in Special:CheckUser 'get edits'

https://gerrit.wikimedia.org/r/956457

Change 956457 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] clienthints: Display data in Special:CheckUser 'get edits'

https://gerrit.wikimedia.org/r/956457

Suggested QA steps:

  1. Install CheckUser
  2. Login to an account with the checkuser right
  3. Make an edit using Google Chrome
  4. Load Special:CheckUser, put the username of the account you are logged in to in the "Username" box and then select 'Get edits' as the check type.
  5. Run the check

Expected behaviour to verify the acceptance criteria: Ensure that the edit you made using Google Chrome, which should be shown at the top of the results list, has Client Hints data displayed. The displayed Client Hints data should be below the "User-Agent string" and IP address, in a similar way to the first design in F37657179.

@Dreamy_Jazz Client Hints data are displayed in results and are similar to the first design mentioned in the screenshots below. The only difference is the prefix names that changed "Browsers" to "Brand" which has been discussed and is fine since it aligns with how it's described in the client hint spec. Also, the order is fine. I will move this to Done. Thanks for all your work!

Status: ✅PASS
Environment: Local: 1.41.0-alpha (62bb717) 18:16, 27 September 2023. Checkuser: 2.5 (d121f43) 16:23, 27 September 2023
OS: macOS Ventura 13.5.2
Browser: Chrome 117
Device: MBA M2
Emulated Device:: N/A
Test Links:
http://localhost:8080/wiki/Beer
http://localhost:8080/wiki/Special:CheckUser

✅AC1: https://phabricator.wikimedia.org/T340993#9197595

Design CompareLatest Patch
Log-line.png (1×1 px, 261 KB)
2023-09-27_12-24-25.png (1×3 px, 291 KB)