Skip to content

Feature/payment method #2

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

Merged
merged 16 commits into from
Oct 23, 2024
Merged

Feature/payment method #2

merged 16 commits into from
Oct 23, 2024

Conversation

thevuong
Copy link
Contributor

No description provided.

thevuong and others added 11 commits October 21, 2024 18:12
Introduce a new RadioCards component in the payment method form to enhance user experience. This change allows users to visually select between different payment options like card, PayPal, and Apple Pay.
Upgraded turbo and eslint-config-turbo versions from 2.2.0 to 2.2.1. Also added focus and transition styles to radio-cards component, and adjusted code style settings for soft margins to 120 in Project.xml.
Updated the focus-visible styles to improve visual clarity and accessibility. The changes include replacing the ring style with a more detailed outline style, ensuring a better focus indicator for users.
Simplified the payment method selection by replacing the RadioGroup component with the more versatile RadioCards component. This change reduces redundancy and enhances code readability.
Standardize the focus-visible ring color across components and toggle focus-visible ring order in Tailwind plugin. This improves consistency and ensures the focus styles match the new design guidelines.
Changed the primary color from sky-900 to sky-600 and the ring color from sky-500 to sky-400. These updates refine the visual design for better contrast and accessibility.
Removed the unnecessary 'space-x-3' class from the FormItem components in the notifications form. This change simplifies the class definitions while maintaining the same visual layout.
Removed redundant space-x-3 class from several FormItem components to simplify the layout styling. This change ensures consistent spacing and alignment across the forms and stories.
Added focus-visible ring offset to improve visual clarity across multiple components. This enhances accessibility and visual feedback for users focusing on elements such as buttons, inputs, and switches.
Added shadow effect for radio buttons and adjusted focus styles for better visibility. The changes improve the visual feedback and accessibility.
Enhance styling consistency across radio, radio group, and slider components. Introduce refined class names and structure to better align with design requirements and ensure more predictable behavior.
Reverted React versions from 19.0.0-rc to 18.3.1 for stable support. Adjusted TypeScript configurations accordingly to ensure compatibility and stability across applications and packages.
Upgrade react, react-dom, and related packages to version 19.0.0-rc, along with their type definitions. This ensures compatibility with the latest React release and prepares the codebase for future updates.
Enhanced `DayButton` props for better type safety, modified `useRef` initialization in `use-event` hook, converted `MailPage` to async for cookie handling, and improved key assignments in `nav` and `carousel` components to ensure stability.
Refactored styling for Checkbox, Radio, and Slider components to ensure better alignment and consistency. Modified global.css to update the input border color variable.
Updated various dependencies to their latest versions to ensure compatibility and take advantage of the latest fixes and features. Notable updates include `next` to 15.0.1, `turbo` to 2.2.3, and `chromatic` to 11.14.0.
@thevuong thevuong marked this pull request as ready for review October 23, 2024 01:03
@thevuong thevuong merged commit 0b31bcd into main Oct 23, 2024
2 checks passed
@thevuong thevuong deleted the feature/payment-method branch October 23, 2024 01:04
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.

1 participant