Skip to content

Implemented the requested Blade theme for the Select component #349

Open
priyan923 wants to merge 7 commits into
younisdev:mainfrom
priyan923:main
Open

Implemented the requested Blade theme for the Select component #349
priyan923 wants to merge 7 commits into
younisdev:mainfrom
priyan923:main

Conversation

@priyan923

Copy link
Copy Markdown
Contributor

Closes #347

Changes:

  1. Updated src/components/select/dependencies/themes.json to include the Blade theme configuration, following the existing structure.
  2. Added the necessary CSS classes for .dyvix-select-blade in src/components/select/dependencies/style/themes.css, modeling the approach after the Modal component's style implementation.
  3. Registered the new theme in devbench/src/component/select.jsx to allow for verification within the development workbench.

priyan923 added 3 commits July 8, 2026 19:19
Added a new theme 'Blade' with specific classes and animation.
Added styles for blade theme in select component.
Updated theme to 'Blade' and removed background styles.
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

@priyan923 is attempting to deploy a commit to the younis' projects Team on Vercel.

A member of the Team first needs to authorize it.

@younisdev younisdev self-requested a review July 8, 2026 22:13
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dyvix-ui Ready Ready Preview, Comment Jul 9, 2026 2:19pm

Comment thread src/components/select/dependencies/style/themes.css Outdated
"class": "dyvix-select-blade",
"input-class": "dyvix-select-input-blade",
"dropdown-class": "dyvix-select-dropdown-blade",
"default-animation": "drift"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

you need to separate Blade and Ember theme registries.

Comment thread src/components/select/dependencies/style/themes.css Outdated
Comment thread devbench/src/component/select.jsx Outdated
Comment on lines +9 to +13
<DyvixSelect
animation={DYVIX_GLOBAL_ANIMATION.GLITCH}
// theme={DYVIX_GLOBAL_THEME.SINGULARITY}
theme="Blade" // Using your new theme
placeholder={'number'}
background={'Red'}
dropdownBackground={'Red'}
// background and dropdownBackground removed to let the theme handle styles

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Remove the comments.

@younisdev younisdev left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@priyan923 I left a review please revise the pull request. Thanks in advance!

@priyan923

Copy link
Copy Markdown
Contributor Author

Okay @younisdev
, I'll incorporate the changes

priyan923 added 3 commits July 9, 2026 08:10
Removed comments about theme usage and style handling.
Added new theme 'Ember' with associated classes.

@priyan923 priyan923 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@younisdev ,
I have made the required changes , you may check ,
Thanks

@younisdev

younisdev commented Jul 9, 2026

Copy link
Copy Markdown
Owner

@priyan923 Thanks for the revision! The implementation works, but it still doesn't quite match the existing global blade theme. Please take a look at blade them in modal themes.css line 103 for reference and align the colors, borders (top and left white sharp edge), ect. You can also visit https://dyvix-ui.vercel.app/components/modal/modal.html and have a visual example on the theme. Thanks!

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.

[Feature] Add Blade select theme

2 participants