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

feature/summary-component #210

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feature/summary-component #210

wants to merge 1 commit into from

Conversation

Triki-Ahmed
Copy link
Contributor

@Triki-Ahmed Triki-Ahmed added the agent-slash Component for the agent theme label Mar 15, 2024
@Triki-Ahmed Triki-Ahmed self-assigned this Mar 15, 2024
Copy link
Contributor

@GuillaumeKESTEMAN GuillaumeKESTEMAN left a comment

Choose a reason for hiding this comment

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

Pas de css ?

packages/react/src/Summary/Summary.tsx Outdated Show resolved Hide resolved
@JLou JLou added this to the 1.0.0 milestone Apr 10, 2024
@Triki-Ahmed
Copy link
Contributor Author

Pas de css ?

D'après ce que j'ai vu sur le toolkit non , ce composant utilise le composant Alert.

Comment on lines +33 to +39
{messages.map((message) => {
return (
<li className="af-summary__message-item" key={`message_${message}`}>
{message}
</li>
);
})}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{messages.map((message) => {
return (
<li className="af-summary__message-item" key={`message_${message}`}>
{message}
</li>
);
})}
{messages.map((message) => (
<li className="af-summary__message-item" key={`message_${message}`}>
{message}
</li>
))}

pplancq
pplancq previously approved these changes May 15, 2024
@@ -12,6 +12,7 @@ export {
CheckboxModes,
} from "./Form/Checkbox/Agent";
export { Choice, ChoiceInput } from "./Form/Choice";
export { Summary } from "./Summary/Summary";
Copy link
Contributor

Choose a reason for hiding this comment

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

il faudrait peut-être créer un ./Summary/index.ts afin d'importer en faisant "./Summary" (comme pour les autres composants).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-slash Component for the agent theme
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Component Summary
4 participants