fix(install): instructions iOS plus robustes (icône de partage, pas de position en dur)#65
Merged
Merged
Conversation
…e position en dur)
La friction d'install signalée par des testeurs venait d'instructions qui
décrivaient le bouton de partage par sa POSITION ("••• en bas à droite"). Or
sur iOS la position varie : en bas (iPhone) ou en haut (iPad), et selon le
réglage "barre d'adresse" — invisible côté web. Le split ios-classic/modern
basé sur la version Safari ne pouvait pas capturer cet axe.
- index.html : une seule liste d'étapes, centrée sur l'ICÔNE de partage
(SVG dessiné) plutôt que sur sa position, avec un repli "tu ne la vois
pas ? touche d'abord •••" pour le Safari compact.
- Préambule "ouvre dans Safari" rendu conditionnel : masqué en Safari,
affiché seulement hors Safari (classe ios-other-browser, détection
fail-safe pour Safari). Un visiteur déjà dans Safari ne le voit plus.
- Suppression du split ios-classic/modern (CSS + détection de version dans
l'inline script) et de getIOSMajorVersion() devenu inutile.
- static-landing.css : styles des icônes inline + du préambule conditionnel.
https://claude.ai/code/session_01RaEog9SSJeZJZA63kYoyD9
Contributor
|
Preview supprimée (PR fermée). Les URLs ne sont plus accessibles. |
…upliquée Suite à revue : - static-landing.css : attributs de présentation des SVG (taille, stroke…) factorisés sur `.ios-icon svg` ; le markup inline ne porte plus que le viewBox + le tracé. - Commentaires de renvoi croisé entre l'inline script de index.html et isIOSSafari() (src/lib/install.ts) : la denylist de navigateurs est dupliquée (pré-paint oblige), on évite la dérive silencieuse. https://claude.ai/code/session_01RaEog9SSJeZJZA63kYoyD9
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Friction d'install signalée en test utilisateur (iPhone + iPad) : impossible de retrouver le bouton « Partager » de Safari. Les instructions le décrivaient par sa position (« ••• en bas à droite »), or sur iOS cette position varie :
Le split
ios-classic/ios-modernbasé sur la version de Safari ne pouvait structurellement pas capturer cet axe haut/bas.Changements
ios-other-browserposée par l'inline script. Détection fail-safe pour Safari (on n'ajoute la classe que sur marqueur tiers expliciteCriOS/FxiOS/…).ios-classic/modern: règles CSS mortes, détection de version dans l'inline script, etgetIOSMajorVersion()devenu inutile.Limite assumée (v1)
Les webviews in-app (Messages, Instagram…) ont un UA ambigu et sont traités comme Safari. Cas plus rare, à creuser plus tard si besoin.
Validation
tsc -b✅BASE=/ npm run build✅npm test— 175 tests ✅npm run lint✅À vérifier sur la preview déployée depuis un vrai iPhone et un iPad (rendu des icônes, lisibilité haut/bas).
Pas de changelog
Amélioration d'UX du parcours d'onboarding (instructions), sous le seuil de la page « Nouveautés ».
https://claude.ai/code/session_01RaEog9SSJeZJZA63kYoyD9
Generated by Claude Code