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

improve first page + fix bugs #21

Merged
merged 17 commits into from
Mar 21, 2024
Merged

Conversation

houssembaltii
Copy link
Contributor

@houssembaltii houssembaltii commented Feb 6, 2024

bug cause : getstarted is linked to the sidebar that is auto generated (contains all routes ), it is designed to have active link class whenever an element from the sidebar is active so it's always active

Copy link

vercel bot commented Feb 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-native-ficus-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 20, 2024 11:28pm

},
{
title: 'Theme support',
// Svg: require('@site/static/img/undraw_docusaurus_tree.svg').default,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi c'est commenté ? a retirer ?

title: 'Style props',
// Svg: require('@site/static/img/undraw_docusaurus_mountain.svg').default,
description: (
<>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y'a besoin du fragment ?

title: 'Chakra UI and React Native components',
// Svg: require('@site/static/img/undraw_docusaurus_react.svg').default,
description: (
<>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment on lines 6 to 8
import ExpoLayout from '../ExpoLayout/index.js';
export const SimpleFeature = ({ title, description, code, codetitle }) => {
return (

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
import ExpoLayout from '../ExpoLayout/index.js';
export const SimpleFeature = ({ title, description, code, codetitle }) => {
return (
import ExpoLayout from '../ExpoLayout/index.js';
export const SimpleFeature = ({ title, description, code, codetitle }) => {
return (

Comment on lines 75 to 87
{item != 'Theme' ? (
<img
src={themeimage}
className={styles.img}
alt="Color degradation"
style={{ objectFit: 'cover' }}
/>
) : (
<div>
<ThemesTable />
</div>
)}
</div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ici je prefere la logique inverse
item ==='theme' ? themeTable : img

<div className={clsx('col col--4')} >

<div className="text--center padding-horiz--md padding-vert--md">
<feature.icon size={30} strokeWidth={1} />

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<feature.icon size={30} strokeWidth={1} />
<feature.icon size={30} strokeWidth={1} />

Tu n'as pas l'extension prettier qui formatte ton code ?

@@ -49,10 +63,11 @@ export default function HomepageFeatures() {
return (
<section className={styles.features}>
<div className="container">
<div className="row">
<div className="row ">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<div className="row ">
<div className="row">

idem la il te faut prettier

@@ -23,6 +23,7 @@ function HomepageHeader() {
</Link>
</div>
<p style={{fontSize: 20, fontWeight: 'bold', marginTop: 20}}>🎉 Beta version available ! 🎉</p>
<p style={{fontWeight:'bold'}}>Currently v1.0.0-beta7</p>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

si on affiche la verison a plusieurs endroit ca sera interessant de l'etraire en constante générale pour tout le site comme ca on change qu'a un endroit

Comment on lines 61 to 71
.TryItLiveDescription {
display: flex;
flex-wrap: wrap;
}
.img {
padding-bottom: 10px;
height: min-content;
}
.TryItLiveDescription {
width: 100%;
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on peut pas regrouper ensemble TryItLiveDescription ?

import useBaseUrl from '@docusaurus/useBaseUrl';
import isInternalUrl from '@docusaurus/isInternalUrl';
import IconExternalLink from '@theme/Icon/ExternalLink';
import * as Icons from 'react-icons/fa'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on peut pas faire mieux qu'importer TOUTES les icones ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je n'ai pas trouvé de solution pour effectuer un import dynamique de ces icônes, donc j'ai essayé une autre approche. Merci de vérifier cela. Si vous avez une autre idée pour ce cas, je suis ouvert à la discussion.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faut que la props icon soit un react Element donc en gros tu passes directement l'icone () en props comme ca c'est quand tu utilise "FooterLinkItem" que tu importes l'icone

@houssembaltii houssembaltii changed the title (WIP) improve first page + fix bugs improve first page + fix bugs Feb 26, 2024
@ntorionbearstudio ntorionbearstudio merged commit f87e588 into main Mar 21, 2024
5 checks passed
@ntorionbearstudio ntorionbearstudio deleted the feat-first-page-improvments branch August 23, 2024 21:23
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

Successfully merging this pull request may close these issues.

3 participants