Skip to content

feat(client,prospect): add warning variant to InputDate (#1804)#1805

Open
Debaerdm wants to merge 4 commits intomainfrom
feat/input-date-warning-variant-1804
Open

feat(client,prospect): add warning variant to InputDate (#1804)#1805
Debaerdm wants to merge 4 commits intomainfrom
feat/input-date-warning-variant-1804

Conversation

@Debaerdm
Copy link
Copy Markdown
Contributor

Adds the warning and hover-warning state to the InputDate component (Apollo + LF), aligned with the InputText warning pattern (#1693).

Behavior: when messageType="warning" is set with a non-empty message and no error, the input now renders an orange (Orange-1050 / #BF4A13) border — 2px at rest, 3px on hover/focus/active.

Closes #1804
Figma: https://www.figma.com/design/vwprvN2ELfI50pjU6MK1Ea/branch/b1G0wMGz53MvQCqkAQCU1X/%E2%9C%A8-B2C-%E2%80%A2-Design-System-Canop%C3%A9e?node-id=17278-55799


Made with Claude

Comment thread apps/look-and-feel-stories/src/components/InputDate/InputDate.stories.tsx Outdated
Comment thread packages/canopee-css/src/prospect-client/Form/InputDate/InputDateLF.css Outdated
Comment thread packages/canopee-react/src/prospect-client/Form/InputDate/InputDateCommon.tsx Outdated
JLou
JLou previously approved these changes Apr 22, 2026
Debaerdm added a commit that referenced this pull request Apr 23, 2026
…pr skill

Add section "Conventions apprises" with 11 rules extracted from PR reviews:
CSS Common over Apollo/LF duplication, classModifier for variants,
getClassName mandatory, MDX in English, no invented token values,
focus-visible vs focus, one story per variant, argTypes/decorators in meta.

Sources: PRs #1793 #1801 #1807 #1805 #1783 #1792 #1765 #1766 #1775 #1771
@Debaerdm
Copy link
Copy Markdown
Contributor Author

Retours appliques :

  • CSS warning mutualise dans InputDateCommon.css, bloc supprime de Apollo et LF (commit 7bfda45)
  • Modifiers simplifie : hasWarning && "warning" dans InputDateCommon.tsx (commit 7bfda45)
  • Story InputDateWarningEmptyStory retiree sur Apollo et LF, il ne reste que Date on warning (commit b0cf1ce)

Made with Claude

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Input Date • Evolution • Ajout State warning

3 participants