Add Rgb::from_hex and implement parsing for u16, u32, f32, f64

palette/src/color_difference.rs

@@ -310,7 +310,7 @@ pub trait EuclideanDistance: Sized {
 ///
 /// // the rustdoc "DARK" theme background and text colors
 /// let background: Srgb<f32> = Srgb::from(0x353535).into_format();
-/// let foreground = Srgb::from_str("#ddd")?.into_format();
+/// let foreground = Srgb::from_str("#ddd")?;
 ///
 /// assert!(background.has_enhanced_contrast_text(foreground));
 /// # Ok(())
@@ -515,7 +515,7 @@ mod test {
             black.relative_contrast(white)
         );
 
-        let c1 = Srgb::from_str("#600").unwrap().into_format();
+        let c1 = Srgb::from_str("#600").unwrap();
 
         assert_relative_eq!(c1.relative_contrast(white), 13.41, epsilon = 0.01);
         assert_relative_eq!(c1.relative_contrast(black), 1.56, epsilon = 0.01);
@@ -532,12 +532,12 @@ mod test {
         assert!(!c1.has_enhanced_contrast_large_text(black));
         assert!(!c1.has_min_contrast_graphics(black));
 
-        let c1 = Srgb::from_str("#066").unwrap().into_format();
+        let c1 = Srgb::from_str("#066").unwrap();
 
         assert_relative_eq!(c1.relative_contrast(white), 6.79, epsilon = 0.01);
         assert_relative_eq!(c1.relative_contrast(black), 3.09, epsilon = 0.01);
 
-        let c1 = Srgb::from_str("#9f9").unwrap().into_format();
+        let c1 = Srgb::from_str("#9f9").unwrap();
 
         assert_relative_eq!(c1.relative_contrast(white), 1.22, epsilon = 0.01);
         assert_relative_eq!(c1.relative_contrast(black), 17.11, epsilon = 0.01);

palette/src/okhsl.rs

@@ -368,7 +368,7 @@ mod tests {
         #[test]
         fn test_srgb_to_okhsl() {
             let red_hex = "#834941";
-            let rgb: Srgb<f64> = Srgb::from_str(red_hex).unwrap().into_format();
+            let rgb: Srgb<f64> = Srgb::from_str(red_hex).unwrap();
             let lin_rgb = LinSrgb::<f64>::from_color_unclamped(rgb);
             let oklab = Oklab::from_color_unclamped(lin_rgb);
             println!(

palette/src/okhsv.rs

@@ -292,8 +292,6 @@ mod tests {
 
     #[cfg(feature = "approx")]
     mod conversion {
-        use core::str::FromStr;
-
         use crate::{
             convert::FromColorUnclamped, encoding, rgb::Rgb, visual::VisuallyEqual, LinSrgb, Okhsv,
             Oklab, OklabHue, Srgb,
@@ -403,9 +401,7 @@ mod tests {
         #[test]
         fn test_srgb_to_okhsv() {
             let red_hex = "#ff0004";
-            let rgb: Srgb = Rgb::<encoding::Srgb, _>::from_str(red_hex)
-                .unwrap()
-                .into_format();
+            let rgb: Srgb = red_hex.parse().unwrap();
             let okhsv = Okhsv::from_color_unclamped(rgb);
             assert_relative_eq!(okhsv.saturation, 1.0, epsilon = 1e-3);
             assert_relative_eq!(okhsv.value, 1.0, epsilon = 1e-3);

palette/src/oklab.rs

@@ -545,7 +545,7 @@ mod test {
         /// Asserts that, for any color space, the lightness of pure white is converted to `l == 1.0`
         #[test]
         fn lightness_of_white_is_one() {
-            let rgb: Srgb<f64> = Rgb::from_str("#ffffff").unwrap().into_format();
+            let rgb: Srgb<f64> = Rgb::from_str("#ffffff").unwrap();
             let lin_rgb = LinSrgb::from_color_unclamped(rgb);
             let oklab = Oklab::from_color_unclamped(lin_rgb);
             println!("white {rgb:?} == {oklab:?}");
@@ -565,7 +565,7 @@ mod test {
         #[test]
         fn blue_srgb() {
             // use f64 to be comparable to javascript
-            let rgb: Srgb<f64> = Rgb::from_str("#0000ff").unwrap().into_format();
+            let rgb: Srgb<f64> = Rgb::from_str("#0000ff").unwrap();
             let lin_rgb = LinSrgb::from_color_unclamped(rgb);
             let oklab = Oklab::from_color_unclamped(lin_rgb);
 

palette/src/relative_contrast.rs

@@ -26,7 +26,7 @@ use crate::{
 ///
 /// // the rustdoc "DARK" theme background and text colors
 /// let background: Srgb<f32> = Srgb::from(0x353535).into_format();
-/// let foreground = Srgb::from_str("#ddd")?.into_format();
+/// let foreground = Srgb::from_str("#ddd")?;
 ///
 /// assert!(background.has_enhanced_contrast_text(foreground));
 /// # Ok(())
@@ -145,7 +145,7 @@ mod test {
             black.get_contrast_ratio(white)
         );
 
-        let c1 = Srgb::from_str("#600").unwrap().into_format();
+        let c1 = Srgb::from_str("#600").unwrap();
 
         assert_relative_eq!(c1.get_contrast_ratio(white), 13.41, epsilon = 0.01);
         assert_relative_eq!(c1.get_contrast_ratio(black), 1.56, epsilon = 0.01);
@@ -162,12 +162,12 @@ mod test {
         assert!(!c1.has_enhanced_contrast_large_text(black));
         assert!(!c1.has_min_contrast_graphics(black));
 
-        let c1 = Srgb::from_str("#066").unwrap().into_format();
+        let c1 = Srgb::from_str("#066").unwrap();
 
         assert_relative_eq!(c1.get_contrast_ratio(white), 6.79, epsilon = 0.01);
         assert_relative_eq!(c1.get_contrast_ratio(black), 3.09, epsilon = 0.01);
 
-        let c1 = Srgb::from_str("#9f9").unwrap().into_format();
+        let c1 = Srgb::from_str("#9f9").unwrap();
 
         assert_relative_eq!(c1.get_contrast_ratio(white), 1.22, epsilon = 0.01);
         assert_relative_eq!(c1.get_contrast_ratio(black), 17.11, epsilon = 0.01);