Skip to content

Add documentation comments to key sections of app.rs #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
quantdale merged 1 commit into from
Jul 23, 2026
Merged

Add documentation comments to key sections of app.rs #9

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/app.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -526,6 +526,7 @@ impl eframe::App for CopyIt {
// ---- Main grid ----
egui::CentralPanel::default().show(ctx, |ui| {
let q = self.search.to_lowercase();
// Filter snippets by category (if not "All") and search query (case-insensitive across title/body/category)
let filtered: Vec<usize> = self
.snippets
.iter()
Expand All @@ -552,12 +553,14 @@ impl eframe::App for CopyIt {
let mut drag_start: Option<(u64, usize, egui::Pos2)> = None;
let mut hover_cursor: Option<egui::CursorIcon> = None;

// Card layout: inner content (300x168) + frame padding (20px total) = visible card size
let card_inner_w = 300.0_f32;
let card_inner_h = 168.0_f32;
// The group frame around each card has 10 px inner margin on each side.
let card_frame_margin = 20.0_f32;
let card_w = card_inner_w + card_frame_margin;
let card_h = card_inner_h + card_frame_margin;
// Spacing between cards and rows; margin for horizontal scroll area padding
let spacing = 12.0_f32;
let top_space = 4.0_f32;
let margin_x = 18.0_f32;
Expand Down Expand Up @@ -606,6 +609,7 @@ impl eframe::App for CopyIt {
let pointer_over_buttons = widgets.copy.hovered()
|| widgets.edit.hovered();

// Initiate drag only if: pointer is not over buttons, no active drag, and drag sensor triggered
if drag_resp.drag_started()
&& !pointer_over_buttons
&& self.drag.is_none()
Expand Down Expand Up @@ -749,6 +753,7 @@ impl eframe::App for CopyIt {
});

// ---- Editor window (new / edit / delete) ----
// Modal editor for creating or modifying snippets; supports inline category creation via the dropdown
if self.editor.is_some() {
let mut ed = self.editor.take().unwrap();
let categories = self.categories.clone();
Expand Down Expand Up @@ -886,6 +891,7 @@ impl eframe::App for CopyIt {

match result {
EditorResult::Save => {
// Normalize category: ensure it's canonical, fall back to "Uncategorized" if empty
let category = {
let canonical = self.add_category(&ed.category);
if canonical.is_empty() {
Expand Down Expand Up @@ -1197,6 +1203,9 @@ fn draw_dashed_line(
}
}

/// Unit tests for grid layout and drag-and-drop logic.
/// Validates that card positioning, gap detection, and insertion line rendering work correctly
/// across different grid configurations (single and multi-card layouts).
#[cfg(test)]
mod layout_tests {
use super::*;
Expand Down
Loading