Page MenuHomePhabricator

[S] Feature Request: Create account - Check username validity on text field return
Closed, ResolvedPublic

Assigned To
None
Authored By
Tsevener
Oct 3 2023, 1:56 PM
Referenced Files
F57746110: IMG_9657.PNG
Nov 25 2024, 6:25 PM
F57746106: IMG_9658.PNG
Nov 25 2024, 6:25 PM
F57746101: 2DA395F1-BE9B-4B6C-BF27-7DCFD7EF0C23_1_102_o.jpeg
Nov 25 2024, 6:25 PM
F57721634: image.png
Nov 19 2024, 2:33 PM
F57718563: image.png
Nov 18 2024, 7:14 PM
F57718561: image.png
Nov 18 2024, 7:14 PM

Description

I've recently downloaded and the iOS app and when trying to create a wikipedia account, I've gotten too many "username not available" errors after typing every field (especially Captcha is annoying) and tapping "Create". This frustrated me so I went to the web and tried to create an account: to my surprise, web UI checked username validity on text field return, which didn't require me to input all the fields like password.

So I've inspected the API required and implemented it in the iOS app. I couldn't find any official documentation on this API so I've just tested with 3 username types: taken, not available, and available. I'm opening this PR as a draft since I'm sure it needs some changes before being mergeable. I'm not sure the response struct I've implemented can handle all responses this API could return.

Volunteer PR in draft: https://github.com/wikimedia/wikipedia-ios/pull/4647

Event Timeline

HNordeenWMF subscribed.

Bringing this back up for discussion: if we drive folks to create accounts during Wikipedia Year in Review, this will be an important feature to have.

We should pick this up if we prompt users on Year in Review to login (currently a nice-to-have) - T376350: [S] Login / Account creation prompt

Tsevener renamed this task from Feature Request: Create account - Check username validity on text field return to [S] Feature Request: Create account - Check username validity on text field return.Nov 12 2024, 4:38 PM

@scblr Design review build is Experimental #198.

Go to create an account screen when logged out. Type a username that is already taken. When you leave focus on the username field, an error message will appear beneath the textfield.

@Tsevener thanks for the work! some piece of feedback:

1) Can we increase the spacing so the input fields maintain their position? This is currently happening:

image.png (1,608×1,748 px, 328 KB)

And here’s a rough sketch of the suggestion:

image.png (1,608×1,748 px, 246 KB)

2) Just verifying: Are we using the Destructive + error color for the error and message? Seems to be a brighter red. Thanks!

@Tsevener I realized that one of the images in T347999#10333258 wasn't attached – I hope it makes more sense now. Sorry!

Nice improvements @Tsevener.

1) It still jumps a tiny little bit (focus on the Passwordfield in this video:

https://share.icloud.com/photos/010uzQN1DePCmW_EnbkEuIhuQ

Any chance to eliminate these last 1-2 pixels?

2) While testing again, I noticed that the two screens are inconsistent:

Log inCreate a new account
2DA395F1-BE9B-4B6C-BF27-7DCFD7EF0C23_1_102_o.jpeg (1,179×2,556 px, 239 KB)
IMG_9658.PNG (1,179×2,556 px, 567 KB)

Could you update the Create a new account navigation bar to be like the one from Log in?

3) And I don’t think the separator line is needed for both screens:

IMG_9657.PNG (1,179×2,556 px, 332 KB)

THANKS!

@scblr

It still jumps a tiny little bit (focus on the Passwordfield in this video:

I couldn't reproduce that myself, but perhaps I don't have the eye :)

I made a new Experimental build #202 that might fix this.

For 2 and 3, I would like to tackle that (navigation bar theming, lines, etc) as I work on T363606 this week.

I couldn't reproduce that myself, but perhaps I don't have the eye :)

Yes, you have @Tsevener 👀 It’s fixed – thanks!

For 2 and 3, I would like to tackle that (navigation bar theming, lines, etc) as I work on T363606 this week.

Ok makes sense!

looks good ! I created a new account just to check that nothing is broken in that flow - works as expected.