-
Notifications
You must be signed in to change notification settings - Fork 3
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
Getting the (first) changes from Ranjith #108
Conversation
site/demo-styles.css
Outdated
margin: 0; | ||
|
||
/* Keep input filter's text from running over the dropdown indicator. */ | ||
padding: 0 0.8em 0 0; | ||
padding: 0 0.8em 0 10px; | ||
border-right: 2px dotted #D0D4CA; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This color should be defined by a variable.
site/demo-styles.css
Outdated
} | ||
|
||
much-select::part(dropdown-option) { | ||
background-color: var(--much-select-dropdown-option-background-color); | ||
background-color: #f1f1f1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This color should be defined by a variable.
border: 3px solid black; | ||
min-width: 10px; | ||
|
||
border: 2px solid black; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(I know you didn't do this Ranjith but this color, in fact probably this whole border definition should be defined in a variable.
src/much-select.js
Outdated
@@ -1829,6 +1829,27 @@ class MuchSelect extends HTMLElement { | |||
0% { transform: rotate(0); } | |||
100% { transform: rotate(360deg); } | |||
} | |||
#wrapper .error-message ul li{ | |||
color:red; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This color should be in a variable.
src/much-select.js
Outdated
padding-left: 5px; | ||
margin-left:6px; | ||
padding-bottom:3px; | ||
border-left: 1px dotted #718093; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This border definition should be in a variable.
I refactored how the "parts" attribute is figured out for each dropdown item.
The selected-value part should be different from the dropdown-items.
Not so much the styling but putting in place the CSS hooks to hand the styling on.
Lots of work trying to make the styling better.
There is still a loooong way to go.