Chat account input validation is too strict - requires http URL but any URI should be allowed
Categories
(Thunderbird :: Address Book, defect, P3)
Tracking
(thunderbird_esr102+ fixed, thunderbird105 wontfix)
People
(Reporter: u695164, Assigned: mkmelin)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
rjl
:
approval-comm-esr102+
|
Details |
(deleted),
text/x-phabricator-request
|
Details |
The current implementation for chat accounts (IMPP) is an input field with the type URL.
This validation is too strict and is confusing. See Thunderbird Developers Matrix chat
The vCard spec demands an URI. See IMPP.
I'm unsure how to move forward from here but my ideas are:
- Using an input field with type text (to support legacy chat protocols)
- Providing a suggestion list (datalist) for popular chat protocols
- matrix:u/
- irc:
- xmpp:
Updated•2 years ago
|
Comment 1•2 years ago
|
||
See also https://phabricator.services.mozilla.com/D146313#4795612 for details on the URIs.
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Comment 2•2 years ago
|
||
(In reply to Nicolai Kasper from comment #0)
This validation is too strict and is confusing.
- Using an input field with type text (to support legacy chat protocols)
- Providing a suggestion list (datalist) for popular chat protocols
Seconded. I think Nicolai's proposal would provide a better UX here.
FTR, Jason's testimony from Matrix:
It took some serious banging my head against a wall to figure out Tbird wants a URL here, and then more sleuthing to figure out how to turn my Matrix username into a URL. Just wanted to point out that from my POV it's not a good experience.
Assignee | ||
Comment 3•2 years ago
|
||
Add title and rolling placeholder displaying example from various protocols.
Adds displaying chat accounts on the display page - they were missing before.
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/0a5f39b827d5
Clarify IMPP entry. r=aleca,freaktechnik
Comment 5•2 years ago
|
||
I think this patch broke some tests: https://treeherder.mozilla.org/jobs?repo=comm-central&selectedTaskRun=AqB_7QUrTRelwNZa4nDzBQ.0
Assignee | ||
Comment 6•2 years ago
|
||
Argh, will land a fix shortly.
Assignee | ||
Comment 8•2 years ago
|
||
Reopening to fix irc protocol pattern and a selection issue Martin noticed.
Assignee | ||
Comment 9•2 years ago
|
||
Comment 10•2 years ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/646e5a688f97
Correct irc: protocol pattern and fix protocol selection. r=freaktechnik
Assignee | ||
Comment 11•2 years ago
|
||
Comment on attachment 9291092 [details]
Bug 1776706 - Clarify IMPP entry. r=aleca,freaktechnik
[Approval Request Comment]
Regression caused by (bug #): new ab
User impact if declined: basically broken functionality. Needed for uplifts (bug 1789990 and others)
Testing completed (on c-c, etc.): c-c beta
Risk to taking this patch (and alternatives if risky): fairly safe. has new strings but we can't do anything about that
Comment 12•2 years ago
|
||
Comment on attachment 9291092 [details]
Bug 1776706 - Clarify IMPP entry. r=aleca,freaktechnik
[Triage Comment]
Approving despite string status. The justification is that Bug 1789990 is serious enough that it needs to be fixed now, and it required the changes in this bug that were not possible without the string changes. It's noted for the record that untranslated strings in a release version make for a poor user experience. The consensus is that having a broken feature with perceived data loss is a worse experience.
As of yesterday, 6/10 tier one locales had translated the new strings with about 33% of locales overall.
Comment 13•2 years ago
|
||
bugherder uplift |
Description
•