generated from nl-design-system/example
-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Remy
committed
Mar 20, 2024
1 parent
c84487f
commit 6abff17
Showing
1 changed file
with
0 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,13 +1,3 @@ | ||
# Login Method Button | ||
|
||
De Login Method Button is een knop die specifiek bedoeld is voor de Inlogstraat. | ||
|
||
## Hernoemen | ||
|
||
Aline maakt de terechte opmerking dat "method" niet passend is voor dit component. We hebben het hier over een _middel_ (Digid, e-Herkenning, Google Accounts, Microsoft, etc.) en geen _manier/methode_ (gebruikersnaam/wachtwoord, One Time Password, login link, etc.). Hier wat suggesties voor betere naamgeving: | ||
|
||
- `auth-agent` ("Auth" applies to both "authentication" and "authorization") | ||
- `auth-button`/`button-auth` | ||
- `login-button`/`button-login` | ||
|
||
Persoonlijk ben ik (Remy Parzinski) voor de `button-*` vorm, daar het een button betreft en deze vorm overeen komt met de API naming convention. Ik wil er ook voor pleiten om het `variant` attribuut te hernoemen naar `agent`. |