Apply suggestions from code review

Co-authored-by: Daniel McNab <[email protected]>

masonry/src/text/selection.rs

@@ -662,7 +662,7 @@ impl<'a> StringCursor<'a> {
     }
 
     /// Move cursor to previous codepoint boundary, if it exists.
-    /// Returns previous codepoint as usize offset.
+    /// Returns previous codepoint as usize offset, or `None` if this cursor was already at the first boundary.
     pub(crate) fn prev(&mut self) -> Option<usize> {
         let current_pos = self.pos();
 

masonry/src/widget/label.rs

@@ -204,10 +204,6 @@ impl Widget for Label {
             }
             LifeCycle::BuildFocusChain =>
             {
-                #[cfg(FALSE)]
-                if !self.text_layout.text().links().is_empty() {
-                    tracing::warn!("Links present in text, but not yet integrated");
-                }
             }
             _ => {}
         }

masonry/src/widget/prose.rs

@@ -223,11 +223,7 @@ impl Widget for Prose {
                 ctx.request_layout();
             }
             LifeCycle::BuildFocusChain => {
-                // TODO: This is *definitely* empty
-                #[cfg(FALSE)]
-                if !self.text_layout.text().links().is_empty() {
-                    tracing::warn!("Links present in text, but not yet integrated");
-                }
+                // When we add links to `Prose`, they will probably need to be handled here.
             }
             _ => {}
         }