Widget: `data-welcome-bubble-delay="3s"` becomes a 3 ms delay
Les mainteneurs répondent en général sous 1 jour
Évaluation
Cette issue n'a pas encore été évaluée.
Description
Which part?
@chatcops/widget
Package version
dev
What happened?
data-welcome-bubble-delay is read with parseInt(value, 10) in packages/widget/src/script-config.ts (line 44), and the loader uses it as bubble.delay ?? DEFAULT_BUBBLE_DELAY (packages/widget/src/loader/loader.ts, line 226). So a value that isn't plain milliseconds doesn't fall back to the 3000 ms default:
data-welcome-bubble-delay |
delay used |
|---|---|
"3000" |
3000 ms |
"3s" |
3 ms |
"1.5s" |
1 ms |
"soon" |
NaN, and ?? keeps it, so the bubble shows at once |
The other numeric attributes in the same file (data-auto-open, data-history-ttl-days) only accept /^\d+$/ and give undefined otherwise, so they fall back to their defaults.
What did you expect?
The same check as the other attributes: digits only become the delay, anything else is undefined and the 3000 ms default applies. The docs say the value is in ms.
How can we reproduce it?
<script src=".../chatcops.min.js" data-api-url="/api/chat" data-welcome-bubble="Need help?" data-welcome-bubble-delay="3s"></script>
The bubble shows almost immediately instead of after 3 seconds. parseInt("3s", 10) is 3.
Environment
Read from the code on dev and checked the parsing in Node 22
- Langage dominant
- TypeScript
- Étoiles
- 13
- Forks
- 7
- Merge moyen
- 3 h 39 min
- PR mergées (30 j)
- 275
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de codercops/chatcops
-
Widget: fetch signed-in visitor tokens when a session is requestedPeut-être pris @piyusshhjangid l’a pris il y a 1 jour. Ouverteenhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 63/100
codercops/chatcops#461 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Widget: support CSP nonces for injected stylesPeut-être pris @piyusshhjangid l’a pris il y a 1 jour. Ouverteenhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
codercops/chatcops#460 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Server: validate `rateLimit.windowMs` before creating the limiterPeut-être pris @ashutoshpalhare l’a pris il y a 1 jour. Ouverte
codercops/chatcops#452 · 1 commentaire · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Widget: a "New reply" button when the visitor has scrolled upPeut-être pris @srujankaleru2007 l’a pris il y a 2 jours. Ouverteenhancement hacktoberfest widget
codercops/chatcops#420 · 3 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
chatcops test: --url to test a deployed serverPeut-être pris @srujankaleru2007 l’a pris il y a 2 jours. Ouverteenhancement hacktoberfest testing
codercops/chatcops#419 · 3 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codercops/chatcops
Issues similaires
-
bug ready-for-agent
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
PhyberApex/kuroshiro#1687 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
microsoft/monaco-editor#5563 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 86/100
awslabs/visual-asset-management-system#937 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100