-
Notifications
You must be signed in to change notification settings - Fork 14
Dropdown option hover state #688
Dropdown option hover state #688
Conversation
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.
hey @daniel-martic-sociomantic could you double check also font color, distances of the icon information list item and distance of the list category title. According to the issue there are some changes also there, Thanks!!
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.
It looks good to me so far, but still missing some updates, the navigation role sub needs to be updated according to the new specs, also the distance between icon an option text not matching the design in some cases, also the current alignment of the option icon and icon text is different than the specs
Added missing styles and fixed sizings
…mantic-tsunami/nessie-ui into Issue-685-DropdownOptionHoverState
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.
LGTM
For #685