@@ -31,12 +31,18 @@ This is the last release bundling plotly.js 3.x; the upgrade to plotly.js 4.x is
 - `plotly_static` bumped to 0.2.0 (breaking: `ImageFormat::EPS` removed)
 - Bump `darling` to 0.24 and `syn` to 3 in `plotly_derive`, and `dirs` to 7 in `plotly_static`
 - Bump example dependencies: `reqwest` 0.13, `env_logger` 0.11, `itertools` 0.15, `ndarray` 0.17, `rand`/`rand_chacha` 0.10
+- Theme statics in `plotly::layout::themes` (`DEFAULT`, `PLOTLY_WHITE`, `PLOTLY_DARK`, …) are now `std::sync::LazyLock<Template>` instead of `once_cell::sync::Lazy<Template>`; the `once_cell` dependency is dropped. Code that only dereferences them is unaffected
+- Add `# Errors`/`# Panics` sections to public fallible APIs and tidy doc formatting (clippy `missing_errors_doc`, `missing_panics_doc`, `doc_markdown`)
 
 ### Removed
 
 - The deprecated Kaleido static export backend: the `plotly_kaleido` crate is no longer part of the workspace, and the `kaleido` and `kaleido_download` features are gone. Use the `static_export_*` features (`plotly_static`) instead
 - The deprecated `ImageFormat::EPS` variant. Use `ImageFormat::SVG` or `ImageFormat::PDF` for vector output
 
+### Fixed
+
+- `Plot::add_frames` no longer adds every frame twice when the plot has no frames yet
+
 ### Security
 
 - Remove the unused `zip` build-dependency from `plotly_static`; with `plotly_kaleido` gone, no workspace crate depends on `zip` directly
 
@@ -0,0 +1 @@
+doc-valid-idents = ["WebDriver", "MapLibre", "GeoJSON", "MathJax", "ModeBar", ".."]
@@ -8,7 +8,6 @@ authors = [
 ]
 license = "MIT"
 readme = "../README.md"
-homepage = "https://github.com/plotly/plotly.rs"
 documentation = "https://docs.rs/plotly"
 repository = "https://github.com/plotly/plotly.rs"
 edition = "2021"
@@ -57,7 +56,6 @@ image = { version = "0.25", optional = true }
 plotly_derive = { version = "0.15", path = "../plotly_derive" }
 plotly_static = { version = "0.2", path = "../plotly_static", optional = true }
 ndarray = { version = "0.17", optional = true }
-once_cell = "1"
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 serde_repr = "0.1"
 
@@ -22,6 +22,11 @@ extern "C" {
 /// The function signature is slightly constrained in that `id` is a `&str`
 /// which represents the `id` of an existing HTML `div` element, rather than
 /// also allowing an instance of a `div` element, itself.
+///
+/// # Panics
+///
+/// Panics if plotly.js rejects the plot (e.g. no element with the given `id`
+/// exists).
 pub async fn new_plot(id: &str, plot: &Plot) {
     let plot_obj = &plot.to_js_object();
 
@@ -37,6 +42,11 @@ pub async fn new_plot(id: &str, plot: &Plot) {
 /// The function signature is slightly constrained in that `id` is a `&str`
 /// which represents the `id` of an existing HTML `div` element, rather than
 /// also allowing an instance of a `div` element, itself.
+///
+/// # Panics
+///
+/// Panics if plotly.js rejects the plot (e.g. no element with the given `id`
+/// exists).
 pub async fn react(id: &str, plot: &Plot) {
     let plot_obj = &plot.to_js_object();
 
 
@@ -57,6 +57,12 @@ extern "C" {
 ///     web_sys::console::log_1(&format!("{:?}", evt).into());
 /// });
 /// ```
+///
+/// # Panics
+///
+/// Panics if no element with the given `div_id` exists. The registered
+/// listener panics if a click event cannot be deserialized into
+/// `ClickEvent`.
 pub fn bind_click<F>(div_id: &str, mut cb: F)
 where
     F: 'static + FnMut(ClickEvent),
 
@@ -49,9 +49,9 @@ impl Color for i16 {}
 impl Color for i8 {}
 impl Color for usize {}
 
-/// ColorArray is only used internally to provide a helper method for converting
-/// Vec<impl Color> to Vec<Box<dyn Color>>, as we would otherwise fall foul of
-/// the orphan rules.
+/// `ColorArray` is only used internally to provide a helper method for
+/// converting Vec<impl Color> to Vec<Box<dyn Color>>, as we would otherwise
+/// fall foul of the orphan rules.
 pub(crate) struct ColorArray<C: Color>(pub(crate) Vec<C>);
 
 #[allow(clippy::from_over_into)]
 
@@ -211,8 +211,8 @@ impl Configuration {
 
     /// When set it determines base URL for the "Edit in Chart Studio"
     /// `show_edit_in_chart_studio`/`show_send_to_cloud` mode bar button and
-    /// the show_link/send_data on-graph link. To enable sending your data to
-    /// Chart Studio Cloud, you need to set both `plotly_server_url` to <https://chart-studio.plotly.com> and
+    /// the `show_link`/`send_data` on-graph link. To enable sending your data
+    /// to Chart Studio Cloud, you need to set both `plotly_server_url` to <https://chart-studio.plotly.com> and
     /// also set `showSendToCloud` to `true`.
     pub fn plotly_server_url(mut self, plotly_server_url: &str) -> Self {
         self.plotly_server_url = Some(plotly_server_url.to_string());
@@ -228,7 +228,7 @@ impl Configuration {
     }
 
     /// Determines whether the graphs are plotted with respect to
-    /// layout.auto_size: true and infer its container size.
+    /// `layout.auto_size: true` and infer its container size.
     pub fn autosizable(mut self, autosizable: bool) -> Self {
         self.autosizable = Some(autosizable);
         self
 
@@ -7,9 +7,9 @@ pub mod sync {
     /// Extension methods for exporting plots using a synchronous exporter.
     pub trait ExporterSyncExt {
         /// Convert the `Plot` to a static image of the given image format and
-        /// save at the given location using a provided StaticExporter.
+        /// save at the given location using a provided `StaticExporter`.
         ///
-        /// This method allows you to reuse a StaticExporter instance across
+        /// This method allows you to reuse a `StaticExporter` instance across
         /// multiple plots, which is more efficient than creating a new one for
         /// each operation.
         ///
@@ -18,7 +18,7 @@ pub mod sync {
         ///
         /// # Arguments
         ///
-        /// * `exporter` - A mutable reference to a StaticExporter instance
+        /// * `exporter` - A mutable reference to a `StaticExporter` instance
         /// * `filename` - The destination path for the output file
         /// * `format` - The desired output image format
         /// * `width` - The width of the output image in pixels
@@ -45,6 +45,11 @@ pub mod sync {
         ///
         /// exporter.close();
         /// ```
+        ///
+        /// # Errors
+        ///
+        /// Returns an error if the export fails, if the image file cannot be
+        /// written, or if called from within an async (Tokio) context.
         fn write_image<P: AsRef<Path>>(
             &mut self,
             plot: &Plot,
@@ -56,10 +61,10 @@ pub mod sync {
         ) -> Result<(), Box<dyn std::error::Error>>;
 
         /// Convert the `Plot` to a static image and return the image as a
-        /// `base64` string. Supported formats are [ImageFormat::JPEG],
-        /// [ImageFormat::PNG] and [ImageFormat::WEBP].
+        /// `base64` string. Supported formats are [`ImageFormat::JPEG`],
+        /// [`ImageFormat::PNG`] and [`ImageFormat::WEBP`].
         ///
-        /// This method allows you to reuse the same StaticExporter instance
+        /// This method allows you to reuse the same `StaticExporter` instance
         /// across multiple plots, which is more efficient than creating
         /// a new one for each operation.
         ///
@@ -91,6 +96,11 @@ pub mod sync {
         ///
         /// exporter.close();
         /// ```
+        ///
+        /// # Errors
+        ///
+        /// Returns an error if `format` is not JPEG, PNG or WEBP, if the export
+        /// fails, or if called from within an async (Tokio) context.
         fn to_base64(
             &mut self,
             plot: &Plot,
@@ -102,7 +112,7 @@ pub mod sync {
 
         /// Convert the `Plot` to SVG and return it as a String.
         ///
-        /// This method allows you to reuse the same StaticExporter instance
+        /// This method allows you to reuse the same `StaticExporter` instance
         /// across multiple plots, which is more efficient than creating
         /// a new one for each operation.
         ///
@@ -133,6 +143,11 @@ pub mod sync {
         ///
         /// exporter.close();
         /// ```
+        ///
+        /// # Errors
+        ///
+        /// Returns an error if the export fails, or if called from within an
+        /// async (Tokio) context.
         fn to_svg(
             &mut self,
             plot: &Plot,
@@ -218,7 +233,7 @@ pub mod r#async {
         /// the given location using the asynchronous exporter.
         ///
         /// The exporter must have been built with the `build_async` method of
-        /// the StaticExporterBuilder.
+        /// the `StaticExporterBuilder`.
         ///
         /// Functionally signature equivalent to the sync version in
         /// [`crate::export::sync::ExporterSyncExt::write_image`], but meant for
@@ -239,7 +254,7 @@ pub mod r#async {
         /// `base64` string using the asynchronous exporter.
         ///
         /// The exporter must have been built with the `build_async` method of
-        /// the StaticExporterBuilder.
+        /// the `StaticExporterBuilder`.
         ///
         /// Functionally signature equivalent to the sync version in
         /// [`crate::export::sync::ExporterSyncExt::to_base64`], but meant for
 
@@ -11,7 +11,7 @@ use serde::Serialize;
 use crate::{Layout, Traces};
 
 /// A frame represents a single state in an animation sequence.
-/// Based on Plotly.js frame_attributes.js specification
+/// Based on Plotly.js `frame_attributes.js` specification
 #[serde_with::skip_serializing_none]
 #[derive(Serialize, Clone, FieldSetter)]
 pub struct Frame {
@@ -63,7 +63,7 @@ impl Default for Animation {
 
 impl Animation {
     /// Create a new animation args with default values for options and
-    /// FramesMode set to all frames
+    /// `FramesMode` set to all frames
     pub fn new() -> Self {
         Self::default()
     }
@@ -139,7 +139,7 @@ impl Serialize for FrameListMode {
 }
 
 /// Animation configuration options
-/// Based on actual Plotly.js animation API from animation_attributes.js
+/// Based on actual Plotly.js animation API from `animation_attributes.js`
 #[serde_with::skip_serializing_none]
 #[derive(Serialize, Clone, Debug, FieldSetter)]
 pub struct AnimationOptions {
 
@@ -622,7 +622,7 @@ impl Axis {
 
     /// Set the range of the axis. This method accepts various types and
     /// converts them to `AxisRange` while also keeping backward compatibility
-    /// with API prior to AxisRange introduction.
+    /// with API prior to `AxisRange` introduction.
     pub fn range(mut self, range: impl Into<AxisRange>) -> Self {
         self.range = Some(range.into());
         self
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+doc-valid-idents = ["WebDriver", "MapLibre", "GeoJSON", "MathJax", "ModeBar", ".."]`