Add TODO comment about screenshots in lib.rs

linebender · Jan 23, 2025 · 240e9c7 · 240e9c7
1 parent f799e0d
commit 240e9c7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/masonry/src/lib.rs b/masonry/src/lib.rs
@@ -96,6 +96,8 @@
 //! [Druid]: https://crates.io/crates/druid
 //! [Xilem]: https://crates.io/crates/xilem
 //! [tracing_tracy]: https://crates.io/crates/tracing-tracy
+// TODO: Add screenshot. This can't use include_screenshot as that doesn't work with cargo-rdme
+// See https://github.com/linebender/xilem/issues/851
 
 // LINEBENDER LINT SET - lib.rs - v1
 // See https://linebender.org/wiki/canonical-lints/

diff --git a/masonry/src/testing/screenshots.rs b/masonry/src/testing/screenshots.rs
@@ -20,11 +20,6 @@ macro_rules! include_screenshot {
     };
 }
 
-// TODO:
-// - Check whether the docs.rs trick actually works.
-// - Check whether this method will work with cargo rdme.
-// - Make better screenshots than the ones we currently use.
-
 #[cfg(not(docsrs))]
 #[doc(hidden)]
 #[macro_export]