Skip to content
This repository was archived by the owner on Feb 20, 2026. It is now read-only.

use type password on api key input#12

Open
zuramai wants to merge 2 commits intotldraw:mainfrom
zuramai:main
Open

use type password on api key input#12
zuramai wants to merge 2 commits intotldraw:mainfrom
zuramai:main

Conversation

@zuramai
Copy link

@zuramai zuramai commented Nov 17, 2023

API Key is a confidential data that should not be shown publicly, especially when we want to record the screen while trying to copy-paste the API Key to the website.

@vercel
Copy link

vercel bot commented Nov 17, 2023

@zuramai is attempting to deploy a commit to the tldraw Team on Vercel.

A member of the Team first needs to authorize it.

@TodePond
Copy link

Hey we tried this, but it was causing some password managers to mess up and think that some text shapes were also passwords (for some reason). I can give it another test

@zuramai
Copy link
Author

zuramai commented Nov 17, 2023

Hi @TodePond, how about blurring the input text like this?

Screenshot 2023-11-17 at 20 02 02

@brunubarbosa
Copy link

it's not a problem when recording the screen since the password is not shown when the input is not focused and when focused it shows just part of the key, maybe just show even less?
Also, something I'd add to it a visual feedback indicating it has a key added (since the same UI is shown in both cases)

@steveruizok
Copy link

Addressed! Sort of. I think we'll do a big CTA and show a modal or something for the api key.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants