Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Token en spoiler #7

Open
Proryon opened this issue Dec 19, 2021 · 6 comments
Open

Token en spoiler #7

Proryon opened this issue Dec 19, 2021 · 6 comments

Comments

@Proryon
Copy link

Proryon commented Dec 19, 2021

Définir le token de blague api en tant que spoiler pour éviter de le montrer accidentellement, sur cette page

@swiizyy swiizyy assigned swiizyy and unassigned swiizyy Jun 25, 2022
@swiizyy
Copy link
Member

swiizyy commented Jul 2, 2022

@DraftProducts Tu en pense quoi ?

@DraftProducts
Copy link
Member

Pourquoi pas, tu peux t'en occuper si tu le souhaites 👍
Tente peut être déjà un design sur Figma si tu veux 🚀

@swiizyy
Copy link
Member

swiizyy commented Jul 5, 2022

@DraftProducts j'avais une idée pourquoi ne pas refaire la page complètement et ne pas affiché le Token ou juste la fin car de toute façon on peut que le copier en appuyant dessus ou sur le bouton 🤔

@DraftProducts
Copy link
Member

Le fait qu'on ne puisse pas le sélectionner n'est pas une bonne chose d'un point de vue ergonomie.

@swiizyy
Copy link
Member

swiizyy commented Jul 5, 2022

C'est vrai mais c'est déjà un peut le cas ? car on ne peut pas le sélectionné nous même... 🤔
Aussi j'aurais bien vu un truc ou on peut crée plusieurs applications avec un token par application ca permettrais de ne pas juste avoir le token en plein milieu je trouve sa assez bizarre en plus le token est vachement trop grand il vaudriez mieux le réduire je pense 😕
Et le fait de pouvoir scroll sur cette page est pas ouf car il n'y a rien en dessous... On peut pas bloquer le scroll sur cette page ?

Tu en pense quoi ? @DraftProducts

@DraftProducts
Copy link
Member

Oui justement, d'où le fait d'améliorer ça ✌️
Le fait d'avoir plusieurs applications nécessitera une grosse PR de migration sur le bot, mais c'est ok pour moi 👍
Pour ce qui est du scroll si tu refais la page avec le design dont tu parles on aura l'occasion d'en reparler.

PS : Le ping ne sert à rien sur GitHub si la personne est déjà dans les participants de la PR, on reçoit de toute manière la notif.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants