Skip to content

Range Slider Styles#524

Merged
cwolf10 merged 2 commits into
masterfrom
IAE-62420
Feb 11, 2022
Merged

Range Slider Styles#524
cwolf10 merged 2 commits into
masterfrom
IAE-62420

Conversation

@cwolf10
Copy link
Copy Markdown
Contributor

@cwolf10 cwolf10 commented Feb 3, 2022

No description provided.

}
}
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's use the variable theme tokens instead of hard coding hex values

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.

Updated.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@cwolf10 Might have not been pushed

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.

doh. should be pushed now

background-color: #81aefc;
}
}
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do all browsers support webkit css?

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.

Good catch, got the other browser's selectors added

Copy link
Copy Markdown
Contributor

@Bibek-Regmi Bibek-Regmi left a comment

Choose a reason for hiding this comment

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

Update on hard coded hex values and investigation into defining css through webkit only selectors

@cwolf10 cwolf10 merged commit a8db6be into master Feb 11, 2022
@cwolf10 cwolf10 deleted the IAE-62420 branch July 28, 2022 12:34
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.

2 participants