lockbox-extension icon indicating copy to clipboard operation
lockbox-extension copied to clipboard

Pre-populate the 'entry' title and the website URL from URI detection

Open sandysage opened this issue 8 years ago • 6 comments
trafficstars

When creating a new online account, I want those credentials to be easily populated into my Lockbox as a new entry.

Acceptance criteria

  1. Entry title field is still editable
  2. Website address is editable, but not easily modifiable

Context

Research findings from Alpha testing indicates there is confusion around what an entry is and how it might be different from the website address. By pre-populating the entry name, as derived from URI, no additional interpretation is needed from the users. And by making the field editable, a user can still control their entry list as desired.

Dependency

  • [ ] #313

Todo

  • [ ] pre-fill title and origin from 'create' in the toolbar UI (the expected primary use case)
  • [ ] no changes to the management tab for 'create'
  • [ ] for the list view, show the title (as we have it today) and fallback to the first origin if no title exists
  • [ ] evaluate the origin UI to make it harder to accidentally modify it (rather than making it not editable at all) - needs-ux

sandysage avatar Nov 10 '17 18:11 sandysage

Rather than doing this, why not just have the items in the list on the left display the first origin (we can only have one now, but eventually, we'll be able to have many) if there's no title? That way, nothing in the data needs to change, but the UI will be a little friendlier.

jimporter avatar Nov 10 '17 19:11 jimporter

If I understand correctly, that means:

  1. user clicks +
  2. the user has blank fields for entry name, website address, username, password, notes
  3. if the user doesn't populate "entry name" then the list view displays the origin as the entry title

If I captured that correctly, then we're still faced with the risk that the user doesn't know what the entry field is for when creating that entry, and we're not working towards helping users to populate their Lockbox entries (alleviating some of the manual burden).

sandysage avatar Nov 10 '17 20:11 sandysage

I think it depends on where the "+" is. In the current management tab, you're essentially creating a new entry from scratch, so I don't think we need to pre-fill anything (or copy data from other fields); in fact, that would be pretty surprising to me as a user. However, if a user is clicking the "+" in the doorhanger, I think it makes sense to prefill the title and origin.

Still, I do think it would be nice to have the entry in the list show the title and fallback to the (first) origin if no title exists. That would cover the case of people creating a "raw" entry from the management tab, as well as explicitly deleting the entry title from a pre-filled entry from the doorhanger.

I also noticed this part:

Website address is set by URI and is not editable

I think it's ok for the origin to be editable (we might have gotten it wrong when we pre-filled it), but it'd be nice if we had a better UI for it that allowed for setting multiple origins and perhaps made it a little harder to accidentally modify the origin.

jimporter avatar Nov 11 '17 07:11 jimporter

Yup. I agree on all those points.

My take-away is that this feature entails:

  • [ ] pre-fill title and origin from 'create' in the toolbar UI (the expected primary use case)
  • [ ] no changes to the management tab for 'create'
  • [ ] for the list view, show the title (as we have it today) and fallback to the first origin if no title exists
  • [ ] evaluate the origin UI to make it harder to accidentally modify it (rather than making it not editable at all) - needs-ux

Does that generally cover it?

Since we're not expecting the management tab to be the primary UI for most users, I think this emphasizes that this work effort is dependent on #315.

sandysage avatar Nov 11 '17 13:11 sandysage

@sandysage Looks good to me!

jimporter avatar Nov 11 '17 22:11 jimporter

May be worth splitting into two issues.

devinreams avatar Nov 13 '17 22:11 devinreams