-
Notifications
You must be signed in to change notification settings - Fork 30
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
feat(slash): add Link Component #598
base: main
Are you sure you want to change the base?
Conversation
f97f57a
to
c80647d
Compare
59a27e8
c80647d
to
59a27e8
Compare
d61e9d2
59a27e8
to
d61e9d2
Compare
d09f3f8
d61e9d2
to
d09f3f8
Compare
d09f3f8
to
cc1cf43
Compare
Hum j'y pense, Link c'est déjà un composant react-router, ca va poser problème et des gymnastiques pour rien ? |
Quality Gate failedFailed conditions See analysis details on SonarQube Cloud Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE |
Alors oui je suis d'accord mais le zeroheight nomme un lien Link import { Link as RouterLink } from "react-router-dom";
export const LinkExample = () => (
<Link component={RouterLink} to="/contact">
Go to Contact Page
</Link>
); |
Description
Implémentation du composant
Link
pour slash, pour ce composant j'ai fait le choix de suffixer la class css paraf-slash
caraf-link
sembler déjà exister (bien que le composant non) et je ne voulais pas occasionner de break ou parasiter les autres class css.Le composant implémente le style décrit ici https://zeroheight.com/4b1e27a45/v/latest/p/657e74-link/b/7443f5
Pour les icones le composant fonctionne avec les
glyphicon
mais aussi lessvg
.Le composant
Link
permet de surcharger automatiquement l'attributrel
de la balisea
parnoopener noreferrer
si latarget
vaut_blank
Le composant
Link
est interopérable facilement avec des lib de routage tel quereact-router-dom