Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
616fc57666 | ||
|
|
d150fd1883 | ||
|
|
f9ede537ed | ||
|
|
9571e2121c | ||
|
|
4a7627e461 | ||
|
|
cb11b1f828 | ||
|
|
89005f718e | ||
|
|
b5113cefa2 | ||
|
|
88ea7d2d53 | ||
|
|
7345c00568 | ||
|
|
f76fad92be | ||
|
|
2c60a03460 | ||
|
|
b6580f197d |
Generated
+1
-1
@@ -1048,7 +1048,7 @@ dependencies = [
|
|||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "ink-notes"
|
name = "ink-notes"
|
||||||
version = "0.3.2"
|
version = "0.3.7"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
"bytemuck",
|
"bytemuck",
|
||||||
"egui",
|
"egui",
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
[package]
|
[package]
|
||||||
name = "ink-notes"
|
name = "ink-notes"
|
||||||
version = "0.3.2"
|
version = "0.3.7"
|
||||||
edition = "2021"
|
edition = "2021"
|
||||||
description = "Latenzoptimierte Vektor-Notizbuch- und Inking-Anwendung für Linux"
|
description = "Latenzoptimierte Vektor-Notizbuch- und Inking-Anwendung für Linux"
|
||||||
repository = "https://gitea.pansi.eu/harald/ink-notes"
|
repository = "https://gitea.pansi.eu/harald/ink-notes"
|
||||||
|
|||||||
@@ -3,13 +3,17 @@
|
|||||||
Latenzoptimierte Vektor-Notizbuch- und Inking-Anwendung für Linux (Zorin OS 18 / Ubuntu 24.04).
|
Latenzoptimierte Vektor-Notizbuch- und Inking-Anwendung für Linux (Zorin OS 18 / Ubuntu 24.04).
|
||||||
|
|
||||||
### Highlights:
|
### Highlights:
|
||||||
|
- **Neue Formen**: Ellipsen sowie in einem Zug gezeichnete Pfeile und Doppelpfeile werden erkannt (Dateiformat-Version 3: ältere Versionen öffnen solche Notizbücher nicht)
|
||||||
|
- **Stabiler Start**: Kein Absturz mehr, wenn die Grafikkarte nicht nutzbar ist (z. B. fehlende Rechte auf /dev/dri); Software-Renderer als Ausweichlösung und verständliche Fehlermeldung
|
||||||
|
- **WSL (WSLg)**: Läuft unter WSL automatisch über X11 statt Wayland, da WSLg die Wayland-Verbindung abbricht; übersteuerbar mit `INK_NOTES_DISPLAY=x11|wayland`
|
||||||
|
- **Wichtige Fehlerbehebung**: Nach einem Neustart auf einer späteren Seite konnte Weiterschreiben den bisherigen Inhalt dieser Seite überschreiben (betrifft v0.2.3 – v0.3.2)
|
||||||
- **Neue Werkzeugleiste**: Schmale senkrechte Leiste mit Symbol-Knöpfen, links oder rechts andockbar; selten genutzte Funktionen im ☰-Menü, das Blatt wird daneben zentriert
|
- **Neue Werkzeugleiste**: Schmale senkrechte Leiste mit Symbol-Knöpfen, links oder rechts andockbar; selten genutzte Funktionen im ☰-Menü, das Blatt wird daneben zentriert
|
||||||
- **Einheitliche Dialoge**: Deckende Fenster ohne doppelte Titel, Seitenauswahl als Kachelraster, größere Bedienelemente in allen Popovern
|
- **Einheitliche Dialoge**: Deckende Fenster ohne doppelte Titel, Seitenauswahl mit Vorschaubildern, größere Bedienelemente in allen Popovern
|
||||||
- **UI per Stift und Touch bedienbar**: Werkzeugleiste, Popover und Dialoge reagieren jetzt auch auf Stift und Finger, auch bei Display-Skalierung
|
- **UI per Stift und Touch bedienbar**: Werkzeugleiste, Popover und Dialoge reagieren jetzt auch auf Stift und Finger, auch bei Display-Skalierung
|
||||||
- **Aktive Stylus-Unterstützung (Lazarite / MPP / AES)**: Natives Wayland-Tablet-Protokoll mit präziser Druck- und Neigungserfassung
|
- **Aktive Stylus-Unterstützung (Lazarite / MPP / AES)**: Natives Wayland-Tablet-Protokoll mit präziser Druck- und Neigungserfassung
|
||||||
- **Touchscreen Multi-Touch**: Stufenloses 2-Finger Pinch-to-Zoom und Panning mit umschaltbarem Finger-Zeichenmodus
|
- **Touchscreen Multi-Touch**: Stufenloses 2-Finger Pinch-to-Zoom und Panning mit umschaltbarem Finger-Zeichenmodus
|
||||||
- **Zwei-Stufen-Inking-Modell**: Kausale Echtzeit-Glättung via 1€-Filter & akausale Ecken-Erhaltung (Corner Preservation)
|
- **Zwei-Stufen-Inking-Modell**: Kausale Echtzeit-Glättung via 1€-Filter & akausale Ecken-Erhaltung (Corner Preservation)
|
||||||
- **Geometrie- und Formerkennung**: Geraden, Rechtecke, Kreise, Rauten, Polygone & Koordinatensysteme (Draw-and-Hold)
|
- **Geometrie- und Formerkennung**: Geraden, Pfeile & Doppelpfeile, Rechtecke, Kreise, Ellipsen, Dreiecke, Rauten, Vielecke & Koordinatensysteme mit spitzen Pfeilspitzen (Draw-and-Hold); Freiformen bleiben unverändert
|
||||||
- **Multi-Dokumenten-Manager**: Unbegrenzte Notizbücher mit atomarem Autosave, Papierkorb und fortlaufenden Backups
|
- **Multi-Dokumenten-Manager**: Unbegrenzte Notizbücher mit atomarem Autosave, Papierkorb und fortlaufenden Backups
|
||||||
- **Datensicherheit**: Versioniertes Dateiformat mit Migration, Quarantäne beschädigter Seiten, Schutz vor Doppelöffnung und Crash-getestetem Speichern
|
- **Datensicherheit**: Versioniertes Dateiformat mit Migration, Quarantäne beschädigter Seiten, Schutz vor Doppelöffnung und Crash-getestetem Speichern
|
||||||
- **In-App Updater**: Automatische Versionsprüfung und sichere Atomar-Aktualisierung via Gitea Releases – mit Pflicht-Signaturprüfung (Minisign) und durchgängigem HTTPS
|
- **In-App Updater**: Automatische Versionsprüfung und sichere Atomar-Aktualisierung via Gitea Releases – mit Pflicht-Signaturprüfung (Minisign) und durchgängigem HTTPS
|
||||||
|
|||||||
+6
-1
@@ -312,7 +312,12 @@ impl AutosaveController {
|
|||||||
|
|
||||||
/// Markiert eine Seite als geändert und startet/erneuert das Debounce-Fenster.
|
/// Markiert eine Seite als geändert und startet/erneuert das Debounce-Fenster.
|
||||||
pub fn mark_page_dirty(&mut self, page: &Page, manifest: &NotebookManifest) {
|
pub fn mark_page_dirty(&mut self, page: &Page, manifest: &NotebookManifest) {
|
||||||
if self.read_only || page.persist == crate::document::PersistState::ReadOnlyAfterLoadError {
|
// Nicht geladene Platzhalter nie schreiben: Sie sind leer und würden den Inhalt
|
||||||
|
// der Seitendatei überschreiben
|
||||||
|
if self.read_only
|
||||||
|
|| !page.is_loaded
|
||||||
|
|| page.persist == crate::document::PersistState::ReadOnlyAfterLoadError
|
||||||
|
{
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
let page_data = PageData::from(page);
|
let page_data = PageData::from(page);
|
||||||
|
|||||||
+92
-27
@@ -87,8 +87,35 @@ pub fn spline_vertices_to_ndc(
|
|||||||
}
|
}
|
||||||
|
|
||||||
/// Verwaltet den GPU-Zustand, Surface, Pipelines und Buffer für das Inking.
|
/// Verwaltet den GPU-Zustand, Surface, Pipelines und Buffer für das Inking.
|
||||||
|
/// Meldung, wenn kein Grafikadapter das Fenster darstellen kann.
|
||||||
|
pub const NO_USABLE_GPU_MESSAGE: &str = "Keine nutzbare Grafikschnittstelle gefunden.\n\
|
||||||
|
Häufigste Ursache: Der Benutzer darf nicht auf die Grafikkarte zugreifen (/dev/dri).\n\
|
||||||
|
Prüfen mit `ls -l /dev/dri` und `groups`; ggf. `sudo usermod -aG render,video $USER`\n\
|
||||||
|
und neu anmelden. Alternativ einen Software-Renderer installieren (Paket mesa-vulkan-drivers)\n\
|
||||||
|
oder die Schnittstelle festlegen, z. B. `WGPU_BACKEND=vulkan ink-notes`.";
|
||||||
|
|
||||||
|
/// Wählt den Darstellungsmodus: Mailbox (geringste Latenz), sonst Fifo, sonst den ersten
|
||||||
|
/// angebotenen. `None`, wenn keiner angeboten wird. `INK_FORCE_FIFO` erzwingt Fifo.
|
||||||
|
pub fn choose_present_mode(available: &[wgpu::PresentMode]) -> Option<wgpu::PresentMode> {
|
||||||
|
let force_fifo = std::env::var("INK_FORCE_FIFO").is_ok();
|
||||||
|
let preferred: &[wgpu::PresentMode] = if force_fifo {
|
||||||
|
&[wgpu::PresentMode::Fifo]
|
||||||
|
} else {
|
||||||
|
&[wgpu::PresentMode::Mailbox, wgpu::PresentMode::Fifo]
|
||||||
|
};
|
||||||
|
preferred
|
||||||
|
.iter()
|
||||||
|
.copied()
|
||||||
|
.find(|m| available.contains(m))
|
||||||
|
.or_else(|| available.first().copied())
|
||||||
|
}
|
||||||
|
|
||||||
pub struct CanvasState {
|
pub struct CanvasState {
|
||||||
pub surface: wgpu::Surface<'static>,
|
pub surface: wgpu::Surface<'static>,
|
||||||
|
/// Verwendeter Grafikadapter (für erneute Abfrage der Surface-Fähigkeiten bei Größenänderung)
|
||||||
|
adapter: wgpu::Adapter,
|
||||||
|
/// Meldung „keine Darstellungsmodi“ wurde bereits ausgegeben
|
||||||
|
presentation_warned: bool,
|
||||||
pub device: wgpu::Device,
|
pub device: wgpu::Device,
|
||||||
pub queue: wgpu::Queue,
|
pub queue: wgpu::Queue,
|
||||||
pub config: wgpu::SurfaceConfiguration,
|
pub config: wgpu::SurfaceConfiguration,
|
||||||
@@ -149,23 +176,47 @@ impl CanvasState {
|
|||||||
let viewport =
|
let viewport =
|
||||||
PageViewport::calculate(size.width as f32, size.height as f32, Self::VIEWPORT_MARGIN);
|
PageViewport::calculate(size.width as f32, size.height as f32, Self::VIEWPORT_MARGIN);
|
||||||
|
|
||||||
// 1. WGPU Instance & Surface
|
// 1. WGPU Instance & Surface (Grafikschnittstelle per WGPU_BACKEND wählbar, z. B. "vulkan")
|
||||||
let instance = wgpu::Instance::new(wgpu::InstanceDescriptor {
|
let instance = wgpu::Instance::new(wgpu::InstanceDescriptor {
|
||||||
backends: wgpu::Backends::all(),
|
backends: wgpu::util::backend_bits_from_env().unwrap_or(wgpu::Backends::all()),
|
||||||
..Default::default()
|
..Default::default()
|
||||||
});
|
});
|
||||||
|
|
||||||
let surface = instance.create_surface(window.clone())?;
|
let surface = instance.create_surface(window.clone())?;
|
||||||
|
|
||||||
// 2. Adapter auswählen (hohe Performance bevorzugt)
|
// 2. Adapter auswählen: zuerst die GPU, dann der Software-Renderer (z. B. lavapipe).
|
||||||
let adapter = instance
|
// Ein Adapter taugt nur, wenn er für dieses Fenster Formate und Darstellungsmodi
|
||||||
.request_adapter(&wgpu::RequestAdapterOptions {
|
// anbietet – fehlt z. B. der Zugriff auf /dev/dri, ist die Liste leer.
|
||||||
power_preference: wgpu::PowerPreference::HighPerformance,
|
let mut chosen = None;
|
||||||
compatible_surface: Some(&surface),
|
for force_fallback_adapter in [false, true] {
|
||||||
force_fallback_adapter: false,
|
let Some(adapter) = instance
|
||||||
})
|
.request_adapter(&wgpu::RequestAdapterOptions {
|
||||||
.await
|
power_preference: wgpu::PowerPreference::HighPerformance,
|
||||||
.ok_or("Kein passender GPU-Adapter gefunden")?;
|
compatible_surface: Some(&surface),
|
||||||
|
force_fallback_adapter,
|
||||||
|
})
|
||||||
|
.await
|
||||||
|
else {
|
||||||
|
continue;
|
||||||
|
};
|
||||||
|
let info = adapter.get_info();
|
||||||
|
let caps = surface.get_capabilities(&adapter);
|
||||||
|
match choose_present_mode(&caps.present_modes) {
|
||||||
|
Some(mode) if !caps.formats.is_empty() && !caps.alpha_modes.is_empty() => {
|
||||||
|
eprintln!(
|
||||||
|
"[Grafik] Adapter: {} ({:?}), Darstellungsmodus {:?}",
|
||||||
|
info.name, info.backend, mode
|
||||||
|
);
|
||||||
|
chosen = Some((adapter, caps, mode));
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
_ => eprintln!(
|
||||||
|
"[Grafik] Adapter {} ({:?}) bietet für das Fenster keine Darstellung an, versuche den nächsten.",
|
||||||
|
info.name, info.backend
|
||||||
|
),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
let (adapter, surface_caps, present_mode) = chosen.ok_or(NO_USABLE_GPU_MESSAGE)?;
|
||||||
|
|
||||||
// 3. Device & Queue erstellen
|
// 3. Device & Queue erstellen
|
||||||
let (device, queue) = adapter
|
let (device, queue) = adapter
|
||||||
@@ -179,8 +230,7 @@ impl CanvasState {
|
|||||||
)
|
)
|
||||||
.await?;
|
.await?;
|
||||||
|
|
||||||
// 4. Zero-Latency-Pufferung: Mailbox mit Fallback Fifo & Frame-Latency 1
|
// 4. Zero-Latency-Pufferung (Mailbox, sonst Fifo) & Frame-Latency 1
|
||||||
let surface_caps = surface.get_capabilities(&adapter);
|
|
||||||
let surface_format = surface_caps
|
let surface_format = surface_caps
|
||||||
.formats
|
.formats
|
||||||
.iter()
|
.iter()
|
||||||
@@ -188,20 +238,6 @@ impl CanvasState {
|
|||||||
.find(|f| f.is_srgb())
|
.find(|f| f.is_srgb())
|
||||||
.unwrap_or(surface_caps.formats[0]);
|
.unwrap_or(surface_caps.formats[0]);
|
||||||
|
|
||||||
let present_mode = if std::env::var("INK_FORCE_FIFO").is_ok() {
|
|
||||||
eprintln!("Erzwinge PresentMode::Fifo (via INK_FORCE_FIFO)");
|
|
||||||
wgpu::PresentMode::Fifo
|
|
||||||
} else if surface_caps
|
|
||||||
.present_modes
|
|
||||||
.contains(&wgpu::PresentMode::Mailbox)
|
|
||||||
{
|
|
||||||
eprintln!("Verwende Zero-Latency PresentMode::Mailbox");
|
|
||||||
wgpu::PresentMode::Mailbox
|
|
||||||
} else {
|
|
||||||
eprintln!("Mailbox nicht verfügbar, Fallback auf PresentMode::Fifo");
|
|
||||||
wgpu::PresentMode::Fifo
|
|
||||||
};
|
|
||||||
|
|
||||||
let config = wgpu::SurfaceConfiguration {
|
let config = wgpu::SurfaceConfiguration {
|
||||||
usage: wgpu::TextureUsages::RENDER_ATTACHMENT,
|
usage: wgpu::TextureUsages::RENDER_ATTACHMENT,
|
||||||
format: surface_format,
|
format: surface_format,
|
||||||
@@ -369,6 +405,8 @@ impl CanvasState {
|
|||||||
|
|
||||||
Ok(Self {
|
Ok(Self {
|
||||||
surface,
|
surface,
|
||||||
|
adapter,
|
||||||
|
presentation_warned: false,
|
||||||
device,
|
device,
|
||||||
queue,
|
queue,
|
||||||
config,
|
config,
|
||||||
@@ -452,6 +490,23 @@ impl CanvasState {
|
|||||||
self.size = new_size;
|
self.size = new_size;
|
||||||
self.config.width = new_size.width;
|
self.config.width = new_size.width;
|
||||||
self.config.height = new_size.height;
|
self.config.height = new_size.height;
|
||||||
|
// Fähigkeiten neu abfragen: Ein nicht (mehr) angebotener Modus führt in wgpu zum Absturz
|
||||||
|
let caps = self.surface.get_capabilities(&self.adapter);
|
||||||
|
if !caps.present_modes.contains(&self.config.present_mode) {
|
||||||
|
match choose_present_mode(&caps.present_modes) {
|
||||||
|
Some(mode) => self.config.present_mode = mode,
|
||||||
|
None => {
|
||||||
|
if !self.presentation_warned {
|
||||||
|
eprintln!(
|
||||||
|
"[Grafik] Fenster kann derzeit nicht dargestellt werden (keine Darstellungsmodi). {}",
|
||||||
|
NO_USABLE_GPU_MESSAGE
|
||||||
|
);
|
||||||
|
self.presentation_warned = true;
|
||||||
|
}
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
self.surface.configure(&self.device, &self.config);
|
self.surface.configure(&self.device, &self.config);
|
||||||
self.recalculate_viewport();
|
self.recalculate_viewport();
|
||||||
}
|
}
|
||||||
@@ -858,6 +913,16 @@ impl CanvasState {
|
|||||||
mod tests {
|
mod tests {
|
||||||
use super::*;
|
use super::*;
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_choose_present_mode() {
|
||||||
|
use wgpu::PresentMode::{Fifo, Immediate, Mailbox};
|
||||||
|
assert_eq!(choose_present_mode(&[Fifo, Mailbox]), Some(Mailbox));
|
||||||
|
assert_eq!(choose_present_mode(&[Fifo, Immediate]), Some(Fifo));
|
||||||
|
assert_eq!(choose_present_mode(&[Immediate]), Some(Immediate));
|
||||||
|
// Keine Darstellungsmodi (z. B. kein Zugriff auf /dev/dri): kein Absturz, sondern None
|
||||||
|
assert_eq!(choose_present_mode(&[]), None);
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_pixel_to_ndc_mapping() {
|
fn test_pixel_to_ndc_mapping() {
|
||||||
let width = 800.0;
|
let width = 800.0;
|
||||||
|
|||||||
+121
-65
@@ -329,6 +329,59 @@ fn render_recognized_shape(
|
|||||||
layer.add_polygon(poly);
|
layer.add_polygon(poly);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
RecognizedShape::Ellipse {
|
||||||
|
center,
|
||||||
|
radius_x,
|
||||||
|
radius_y,
|
||||||
|
rotation,
|
||||||
|
} => {
|
||||||
|
let ring = (0..72)
|
||||||
|
.map(|i| {
|
||||||
|
let t = i as f32 / 72.0 * std::f32::consts::TAU;
|
||||||
|
let p =
|
||||||
|
crate::geometry::ellipse_point(*center, *radius_x, *radius_y, *rotation, t);
|
||||||
|
let (x, y) = page_to_pdf_mm(p.x, p.y);
|
||||||
|
(Point::new(Mm(x), Mm(y)), false)
|
||||||
|
})
|
||||||
|
.collect();
|
||||||
|
layer.add_polygon(Polygon {
|
||||||
|
rings: vec![ring],
|
||||||
|
mode: PaintMode::Stroke,
|
||||||
|
winding_order: WindingOrder::NonZero,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
RecognizedShape::Arrow { start, end, double } => {
|
||||||
|
// Gleiche Spitzen wie auf dem Bildschirm; der Schaft endet an ihrer Basis
|
||||||
|
const PDF_SHAPE_HALF_WIDTH: f32 = 2.65;
|
||||||
|
let shaft_len = (*end - *start).length();
|
||||||
|
let dir = (*end - *start).normalize_or_zero();
|
||||||
|
let end_head = crate::geometry::arrow_head(*end, dir, PDF_SHAPE_HALF_WIDTH, shaft_len);
|
||||||
|
let start_head =
|
||||||
|
crate::geometry::arrow_head(*start, -dir, PDF_SHAPE_HALF_WIDTH, shaft_len);
|
||||||
|
let line_start = if *double { start_head.0 } else { *start };
|
||||||
|
let to_pdf = |p: glam::Vec2| {
|
||||||
|
let (x, y) = page_to_pdf_mm(p.x, p.y);
|
||||||
|
(Point::new(Mm(x), Mm(y)), false)
|
||||||
|
};
|
||||||
|
layer.add_line(Line {
|
||||||
|
points: vec![to_pdf(line_start), to_pdf(end_head.0)],
|
||||||
|
is_closed: false,
|
||||||
|
});
|
||||||
|
layer.set_fill_color(stroke_color);
|
||||||
|
let mut heads = vec![(*end, end_head)];
|
||||||
|
if *double {
|
||||||
|
heads.push((*start, start_head));
|
||||||
|
}
|
||||||
|
for (tip, (_, left, right)) in heads {
|
||||||
|
layer.add_polygon(Polygon {
|
||||||
|
rings: vec![vec![to_pdf(tip), to_pdf(left), to_pdf(right)]],
|
||||||
|
mode: PaintMode::Fill,
|
||||||
|
winding_order: WindingOrder::NonZero,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
RecognizedShape::Rectangle { corners } => {
|
RecognizedShape::Rectangle { corners } => {
|
||||||
let mut ring = Vec::with_capacity(4);
|
let mut ring = Vec::with_capacity(4);
|
||||||
for c in corners {
|
for c in corners {
|
||||||
@@ -412,76 +465,58 @@ fn render_recognized_shape(
|
|||||||
y_axis_end,
|
y_axis_end,
|
||||||
with_arrows,
|
with_arrows,
|
||||||
} => {
|
} => {
|
||||||
|
// Gleiche Pfeilspitzen wie auf dem Bildschirm; Achsen enden an der Basis der Spitze
|
||||||
|
// (Linienstärke 1,5 pt ≈ 5,3 Seiteneinheiten, halbe Breite ≈ 2,65)
|
||||||
|
const PDF_SHAPE_HALF_WIDTH: f32 = 2.65;
|
||||||
|
let y_dir = (*y_axis_end - *origin).normalize_or_zero();
|
||||||
|
let x_dir = (*x_axis_end - *origin).normalize_or_zero();
|
||||||
|
let y_arrow = crate::geometry::arrow_head(
|
||||||
|
*y_axis_end,
|
||||||
|
y_dir,
|
||||||
|
PDF_SHAPE_HALF_WIDTH,
|
||||||
|
(*y_axis_end - *origin).length(),
|
||||||
|
);
|
||||||
|
let x_arrow = crate::geometry::arrow_head(
|
||||||
|
*x_axis_end,
|
||||||
|
x_dir,
|
||||||
|
PDF_SHAPE_HALF_WIDTH,
|
||||||
|
(*x_axis_end - *origin).length(),
|
||||||
|
);
|
||||||
|
let (y_line_end, x_line_end) = if *with_arrows {
|
||||||
|
(y_arrow.0, x_arrow.0)
|
||||||
|
} else {
|
||||||
|
(*y_axis_end, *x_axis_end)
|
||||||
|
};
|
||||||
|
|
||||||
let (ox, oy) = page_to_pdf_mm(origin.x, origin.y);
|
let (ox, oy) = page_to_pdf_mm(origin.x, origin.y);
|
||||||
let (xx, xy) = page_to_pdf_mm(x_axis_end.x, x_axis_end.y);
|
for end in [y_line_end, x_line_end] {
|
||||||
let (yx, yy) = page_to_pdf_mm(y_axis_end.x, y_axis_end.y);
|
let (ex, ey) = page_to_pdf_mm(end.x, end.y);
|
||||||
|
layer.add_line(Line {
|
||||||
// Achse 1 (Y)
|
points: vec![
|
||||||
let line_y = Line {
|
(Point::new(Mm(ox), Mm(oy)), false),
|
||||||
points: vec![
|
(Point::new(Mm(ex), Mm(ey)), false),
|
||||||
(Point::new(Mm(ox), Mm(oy)), false),
|
],
|
||||||
(Point::new(Mm(yx), Mm(yy)), false),
|
is_closed: false,
|
||||||
],
|
});
|
||||||
is_closed: false,
|
}
|
||||||
};
|
|
||||||
layer.add_line(line_y);
|
|
||||||
|
|
||||||
// Achse 2 (X)
|
|
||||||
let line_x = Line {
|
|
||||||
points: vec![
|
|
||||||
(Point::new(Mm(ox), Mm(oy)), false),
|
|
||||||
(Point::new(Mm(xx), Mm(xy)), false),
|
|
||||||
],
|
|
||||||
is_closed: false,
|
|
||||||
};
|
|
||||||
layer.add_line(line_x);
|
|
||||||
|
|
||||||
// Pfeilspitzen als gefüllte Dreiecke
|
// Pfeilspitzen als gefüllte Dreiecke
|
||||||
if *with_arrows {
|
if *with_arrows {
|
||||||
layer.set_fill_color(stroke_color);
|
layer.set_fill_color(stroke_color);
|
||||||
|
for (tip, (_, left, right)) in [(*y_axis_end, y_arrow), (*x_axis_end, x_arrow)] {
|
||||||
let y_dir = (*y_axis_end - *origin).normalize_or_zero();
|
let ring = [tip, left, right]
|
||||||
let arrow_len_page = 25.0;
|
.iter()
|
||||||
let arrow_w_page = 16.0;
|
.map(|p| {
|
||||||
|
let (px, py) = page_to_pdf_mm(p.x, p.y);
|
||||||
let base_y = *y_axis_end - y_dir * arrow_len_page;
|
(Point::new(Mm(px), Mm(py)), false)
|
||||||
let left_y = base_y + glam::Vec2::new(-y_dir.y, y_dir.x) * (arrow_w_page * 0.5);
|
})
|
||||||
let right_y = base_y - glam::Vec2::new(-y_dir.y, y_dir.x) * (arrow_w_page * 0.5);
|
.collect();
|
||||||
|
layer.add_polygon(Polygon {
|
||||||
let (tx, ty) = page_to_pdf_mm(y_axis_end.x, y_axis_end.y);
|
rings: vec![ring],
|
||||||
let (lx, ly) = page_to_pdf_mm(left_y.x, left_y.y);
|
mode: PaintMode::Fill,
|
||||||
let (rx, ry) = page_to_pdf_mm(right_y.x, right_y.y);
|
winding_order: WindingOrder::NonZero,
|
||||||
|
});
|
||||||
let arrow_poly_y = Polygon {
|
}
|
||||||
rings: vec![vec![
|
|
||||||
(Point::new(Mm(tx), Mm(ty)), false),
|
|
||||||
(Point::new(Mm(lx), Mm(ly)), false),
|
|
||||||
(Point::new(Mm(rx), Mm(ry)), false),
|
|
||||||
]],
|
|
||||||
mode: PaintMode::Fill,
|
|
||||||
winding_order: WindingOrder::NonZero,
|
|
||||||
};
|
|
||||||
layer.add_polygon(arrow_poly_y);
|
|
||||||
|
|
||||||
let x_dir = (*x_axis_end - *origin).normalize_or_zero();
|
|
||||||
let base_x = *x_axis_end - x_dir * arrow_len_page;
|
|
||||||
let left_x = base_x + glam::Vec2::new(-x_dir.y, x_dir.x) * (arrow_w_page * 0.5);
|
|
||||||
let right_x = base_x - glam::Vec2::new(-x_dir.y, x_dir.x) * (arrow_w_page * 0.5);
|
|
||||||
|
|
||||||
let (tx_x, ty_x) = page_to_pdf_mm(x_axis_end.x, x_axis_end.y);
|
|
||||||
let (lx_x, ly_x) = page_to_pdf_mm(left_x.x, left_x.y);
|
|
||||||
let (rx_x, ry_x) = page_to_pdf_mm(right_x.x, right_x.y);
|
|
||||||
|
|
||||||
let arrow_poly_x = Polygon {
|
|
||||||
rings: vec![vec![
|
|
||||||
(Point::new(Mm(tx_x), Mm(ty_x)), false),
|
|
||||||
(Point::new(Mm(lx_x), Mm(ly_x)), false),
|
|
||||||
(Point::new(Mm(rx_x), Mm(ry_x)), false),
|
|
||||||
]],
|
|
||||||
mode: PaintMode::Fill,
|
|
||||||
winding_order: WindingOrder::NonZero,
|
|
||||||
};
|
|
||||||
layer.add_polygon(arrow_poly_x);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -838,6 +873,27 @@ mod tests {
|
|||||||
);
|
);
|
||||||
page.add_stroke(coords_stroke);
|
page.add_stroke(coords_stroke);
|
||||||
|
|
||||||
|
let ellipse_stroke = Stroke::new(40, vec![], vec![], [0.1, 0.5, 0.5, 1.0]).with_shape(
|
||||||
|
crate::geometry::RecognizedShape::Ellipse {
|
||||||
|
center: glam::Vec2::new(1400.0, 600.0),
|
||||||
|
radius_x: 300.0,
|
||||||
|
radius_y: 150.0,
|
||||||
|
rotation: 0.4,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
page.add_stroke(ellipse_stroke);
|
||||||
|
|
||||||
|
for (id, double) in [(41, false), (42, true)] {
|
||||||
|
let arrow_stroke = Stroke::new(id, vec![], vec![], [0.2, 0.2, 0.2, 1.0]).with_shape(
|
||||||
|
crate::geometry::RecognizedShape::Arrow {
|
||||||
|
start: glam::Vec2::new(1200.0, 1800.0 + id as f32 * 40.0),
|
||||||
|
end: glam::Vec2::new(1800.0, 1800.0 + id as f32 * 40.0),
|
||||||
|
double,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
page.add_stroke(arrow_stroke);
|
||||||
|
}
|
||||||
|
|
||||||
let tri_stroke = Stroke::new(5, vec![], vec![], [0.6, 0.2, 0.7, 1.0]).with_shape(
|
let tri_stroke = Stroke::new(5, vec![], vec![], [0.6, 0.2, 0.7, 1.0]).with_shape(
|
||||||
crate::geometry::RecognizedShape::Triangle {
|
crate::geometry::RecognizedShape::Triangle {
|
||||||
corners: [
|
corners: [
|
||||||
|
|||||||
+1373
-145
File diff suppressed because it is too large
Load Diff
@@ -8,6 +8,7 @@ pub mod export;
|
|||||||
pub mod filter;
|
pub mod filter;
|
||||||
pub mod geometry;
|
pub mod geometry;
|
||||||
pub mod history;
|
pub mod history;
|
||||||
|
pub mod platform;
|
||||||
pub mod selection;
|
pub mod selection;
|
||||||
pub mod settings;
|
pub mod settings;
|
||||||
pub mod spline;
|
pub mod spline;
|
||||||
|
|||||||
+34
-3
@@ -9,8 +9,9 @@ use winit::{
|
|||||||
event::{
|
event::{
|
||||||
ElementState, Event, KeyEvent, MouseButton, MouseScrollDelta, TouchPhase, WindowEvent,
|
ElementState, Event, KeyEvent, MouseButton, MouseScrollDelta, TouchPhase, WindowEvent,
|
||||||
},
|
},
|
||||||
event_loop::{ControlFlow, EventLoop},
|
event_loop::{ControlFlow, EventLoopBuilder},
|
||||||
keyboard::{Key, ModifiersState, NamedKey},
|
keyboard::{Key, ModifiersState, NamedKey},
|
||||||
|
platform::{wayland::EventLoopBuilderExtWayland, x11::EventLoopBuilderExtX11},
|
||||||
window::{Window, WindowBuilder},
|
window::{Window, WindowBuilder},
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -20,6 +21,7 @@ use ink_notes::canvas::CanvasState;
|
|||||||
use ink_notes::document::{Notebook, PageBackground};
|
use ink_notes::document::{Notebook, PageBackground};
|
||||||
use ink_notes::export::{self, export_notebook_to_pdf, ExportStrategy};
|
use ink_notes::export::{self, export_notebook_to_pdf, ExportStrategy};
|
||||||
use ink_notes::geometry::{self, RecognizedShape};
|
use ink_notes::geometry::{self, RecognizedShape};
|
||||||
|
use ink_notes::platform::{self, DisplayBackend};
|
||||||
use ink_notes::selection::{
|
use ink_notes::selection::{
|
||||||
render_selection_overlay, DragMode, SelectionHandleHit, SelectionState,
|
render_selection_overlay, DragMode, SelectionHandleHit, SelectionState,
|
||||||
};
|
};
|
||||||
@@ -137,7 +139,20 @@ fn commit_stroke_to_notebook(
|
|||||||
}
|
}
|
||||||
|
|
||||||
fn main() -> Result<(), Box<dyn std::error::Error>> {
|
fn main() -> Result<(), Box<dyn std::error::Error>> {
|
||||||
let event_loop = EventLoop::new()?;
|
// Anzeigesystem wählen: Unter WSL (WSLg) bricht Wayland ab, X11 (Xwayland) läuft stabil
|
||||||
|
let mut event_loop_builder = EventLoopBuilder::new();
|
||||||
|
match platform::display_backend_from_env() {
|
||||||
|
DisplayBackend::X11 => {
|
||||||
|
eprintln!("[Anzeige] Verwende X11 (unter WSL Standard; übersteuerbar mit INK_NOTES_DISPLAY=wayland)");
|
||||||
|
event_loop_builder.with_x11();
|
||||||
|
}
|
||||||
|
DisplayBackend::Wayland => {
|
||||||
|
eprintln!("[Anzeige] Verwende Wayland (INK_NOTES_DISPLAY=wayland)");
|
||||||
|
event_loop_builder.with_wayland();
|
||||||
|
}
|
||||||
|
DisplayBackend::Automatic => {}
|
||||||
|
}
|
||||||
|
let event_loop = event_loop_builder.build()?;
|
||||||
let window = Arc::new(
|
let window = Arc::new(
|
||||||
WindowBuilder::new()
|
WindowBuilder::new()
|
||||||
.with_title("ink-notes (Latenzoptimiertes Linux-Inking)")
|
.with_title("ink-notes (Latenzoptimiertes Linux-Inking)")
|
||||||
@@ -145,7 +160,13 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
|
|||||||
.build(&event_loop)?,
|
.build(&event_loop)?,
|
||||||
);
|
);
|
||||||
|
|
||||||
let mut canvas = pollster::block_on(CanvasState::new(window.clone()))?;
|
let mut canvas = match pollster::block_on(CanvasState::new(window.clone())) {
|
||||||
|
Ok(canvas) => canvas,
|
||||||
|
Err(err) => {
|
||||||
|
eprintln!("[Grafik-Fehler] {err}");
|
||||||
|
std::process::exit(1);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
let mut tablet_manager = match octotablet::builder::Builder::default().build_shared(&window) {
|
let mut tablet_manager = match octotablet::builder::Builder::default().build_shared(&window) {
|
||||||
Ok(mgr) => {
|
Ok(mgr) => {
|
||||||
@@ -2068,6 +2089,16 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
|
|||||||
update_window_title(&window, ¬ebook, &manifest, ui_state.active_tool, autosave.status());
|
update_window_title(&window, ¬ebook, &manifest, ui_state.active_tool, autosave.status());
|
||||||
window.request_redraw();
|
window.request_redraw();
|
||||||
}
|
}
|
||||||
|
UiAction::LoadAllPages => {
|
||||||
|
// Für die Vorschaubilder der Seitenauswahl; Fehler werden wie beim
|
||||||
|
// Blättern behandelt (Ersatzseite, Warnung beim Öffnen der Seite)
|
||||||
|
for idx in 0..notebook.pages.len() {
|
||||||
|
if !notebook.pages[idx].is_loaded {
|
||||||
|
let _ = storage::ensure_page_loaded(&storage_dir, &mut notebook, idx);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
window.request_redraw();
|
||||||
|
}
|
||||||
UiAction::GoToPage(idx) => {
|
UiAction::GoToPage(idx) => {
|
||||||
if idx < notebook.pages.len() {
|
if idx < notebook.pages.len() {
|
||||||
selection_state.clear();
|
selection_state.clear();
|
||||||
|
|||||||
@@ -0,0 +1,96 @@
|
|||||||
|
//! Erkennung der Laufzeitumgebung und Wahl des Anzeigesystems (X11 oder Wayland).
|
||||||
|
//!
|
||||||
|
//! Unter WSL (WSLg) trennt der Wayland-Compositor die Verbindung, sobald die App ein Bild
|
||||||
|
//! über Vulkan übergibt ("Connection reset by peer"). Über X11 (Xwayland) läuft sie stabil,
|
||||||
|
//! daher wird unter WSL standardmäßig X11 verwendet. `INK_NOTES_DISPLAY=x11|wayland`
|
||||||
|
//! übersteuert die Wahl.
|
||||||
|
|
||||||
|
/// Anzeigesystem, mit dem die Ereignisschleife erzeugt wird.
|
||||||
|
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
|
||||||
|
pub enum DisplayBackend {
|
||||||
|
/// Automatische Wahl von winit (Wayland, falls verfügbar, sonst X11)
|
||||||
|
Automatic,
|
||||||
|
X11,
|
||||||
|
Wayland,
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Wählt das Anzeigesystem.
|
||||||
|
///
|
||||||
|
/// - `override_value`: Inhalt von `INK_NOTES_DISPLAY` (`x11` oder `wayland`, sonst ignoriert)
|
||||||
|
/// - `is_wsl`: läuft unter WSL
|
||||||
|
/// - `has_x11_display`: `DISPLAY` ist gesetzt (X11/Xwayland erreichbar)
|
||||||
|
pub fn choose_display_backend(
|
||||||
|
override_value: Option<&str>,
|
||||||
|
is_wsl: bool,
|
||||||
|
has_x11_display: bool,
|
||||||
|
) -> DisplayBackend {
|
||||||
|
match override_value
|
||||||
|
.map(|v| v.trim().to_ascii_lowercase())
|
||||||
|
.as_deref()
|
||||||
|
{
|
||||||
|
Some("x11") => return DisplayBackend::X11,
|
||||||
|
Some("wayland") => return DisplayBackend::Wayland,
|
||||||
|
_ => {}
|
||||||
|
}
|
||||||
|
if is_wsl && has_x11_display {
|
||||||
|
DisplayBackend::X11
|
||||||
|
} else {
|
||||||
|
DisplayBackend::Automatic
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Läuft das Programm unter WSL (Windows-Subsystem für Linux)?
|
||||||
|
pub fn is_wsl() -> bool {
|
||||||
|
if std::env::var_os("WSL_DISTRO_NAME").is_some() || std::env::var_os("WSL_INTEROP").is_some() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
std::fs::read_to_string("/proc/sys/kernel/osrelease")
|
||||||
|
.map(|release| release.to_ascii_lowercase().contains("microsoft"))
|
||||||
|
.unwrap_or(false)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Wahl anhand der aktuellen Umgebung.
|
||||||
|
pub fn display_backend_from_env() -> DisplayBackend {
|
||||||
|
choose_display_backend(
|
||||||
|
std::env::var("INK_NOTES_DISPLAY").ok().as_deref(),
|
||||||
|
is_wsl(),
|
||||||
|
std::env::var_os("DISPLAY").is_some_and(|d| !d.is_empty()),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_choose_display_backend() {
|
||||||
|
// WSL mit X11-Display: X11 (Wayland unter WSLg bricht ab)
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(None, true, true),
|
||||||
|
DisplayBackend::X11
|
||||||
|
);
|
||||||
|
// WSL ohne X11-Display: winit entscheiden lassen
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(None, true, false),
|
||||||
|
DisplayBackend::Automatic
|
||||||
|
);
|
||||||
|
// Natives Linux: winit entscheiden lassen
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(None, false, true),
|
||||||
|
DisplayBackend::Automatic
|
||||||
|
);
|
||||||
|
// Übersteuerung
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(Some("wayland"), true, true),
|
||||||
|
DisplayBackend::Wayland
|
||||||
|
);
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(Some(" X11 "), false, true),
|
||||||
|
DisplayBackend::X11
|
||||||
|
);
|
||||||
|
assert_eq!(
|
||||||
|
choose_display_backend(Some("quatsch"), true, true),
|
||||||
|
DisplayBackend::X11
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
+117
-2
@@ -124,8 +124,12 @@ impl StrokeData {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Aktuelle Dateiformat-Version von ink-notes (Version 2: Stroke-Treue mit tool, width, finalized).
|
/// Aktuelle Dateiformat-Version von ink-notes.
|
||||||
pub const CURRENT_FORMAT_VERSION: u32 = 2;
|
/// - Version 2: Stroke-Treue mit tool, width, finalized
|
||||||
|
/// - Version 3: neue Formen (Ellipse, Pfeil). Ältere App-Versionen kennen sie nicht und würden
|
||||||
|
/// betroffene Seiten für beschädigt halten; mit Version 3 lehnen sie das Notizbuch stattdessen
|
||||||
|
/// sauber als „neuere Version“ ab.
|
||||||
|
pub const CURRENT_FORMAT_VERSION: u32 = 3;
|
||||||
|
|
||||||
/// Standardwert für ältere Dateien ohne explizite Formatversion (Legacy v0).
|
/// Standardwert für ältere Dateien ohne explizite Formatversion (Legacy v0).
|
||||||
pub fn legacy_format_version() -> u32 {
|
pub fn legacy_format_version() -> u32 {
|
||||||
@@ -697,6 +701,15 @@ pub fn load_notebook(notebook_dir: &Path) -> Result<(Notebook, NotebookManifest)
|
|||||||
);
|
);
|
||||||
notebook.load_warnings = load_warnings;
|
notebook.load_warnings = load_warnings;
|
||||||
|
|
||||||
|
// Die zuletzt geöffnete Seite sofort laden: Sie ist nach dem Start die aktuelle Seite,
|
||||||
|
// und ein leerer Platzhalter würde beim Weiterschreiben ihren Inhalt überschreiben.
|
||||||
|
let current = notebook.current_page_index;
|
||||||
|
if !notebook.pages[current].is_loaded {
|
||||||
|
if let Err(e) = ensure_page_loaded(notebook_dir, &mut notebook, current) {
|
||||||
|
eprintln!("[Storage] Aktuelle Seite {current} konnte nicht geladen werden: {e}");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
Ok((notebook, manifest))
|
Ok((notebook, manifest))
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -2167,4 +2180,106 @@ mod tests {
|
|||||||
assert_eq!(loaded.pages[1].background, PageBackground::Lined);
|
assert_eq!(loaded.pages[1].background, PageBackground::Lined);
|
||||||
assert!(crate::export::generate_pdf_document(&loaded, Default::default()).is_ok());
|
assert!(crate::export::generate_pdf_document(&loaded, Default::default()).is_ok());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_restart_on_later_page_keeps_its_content() {
|
||||||
|
// Regression: Beim Start wurde nur Seite 1 geladen; war zuletzt eine spätere Seite
|
||||||
|
// offen, war sie ein leerer Platzhalter, und Weiterschreiben überschrieb ihren Inhalt.
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let (mut notebook, mut manifest, dir) =
|
||||||
|
create_new_notebook(temp_dir.path(), "Neustart").unwrap();
|
||||||
|
notebook.add_page(PageBackground::Lined);
|
||||||
|
notebook.pages[1] = page_with_one_stroke(notebook.pages[1].id, PageBackground::Lined);
|
||||||
|
notebook.current_page_index = 1;
|
||||||
|
save_notebook(&dir, ¬ebook, &mut manifest).unwrap();
|
||||||
|
|
||||||
|
let (mut loaded, mut loaded_manifest) = load_notebook(&dir).unwrap();
|
||||||
|
assert_eq!(loaded.current_page_index, 1);
|
||||||
|
assert!(
|
||||||
|
loaded.current_page().is_loaded,
|
||||||
|
"aktuelle Seite muss geladen sein"
|
||||||
|
);
|
||||||
|
assert_eq!(loaded.current_page().strokes.len(), 1);
|
||||||
|
|
||||||
|
// Weiterschreiben und speichern darf den bisherigen Inhalt nicht verlieren
|
||||||
|
let extra = page_with_one_stroke(99, PageBackground::Lined).strokes[0].clone();
|
||||||
|
loaded.current_page_mut().strokes.push(extra);
|
||||||
|
let mut autosave = AutosaveController::new(dir.clone(), Duration::from_secs(60));
|
||||||
|
autosave.mark_dirty(&loaded, &mut loaded_manifest);
|
||||||
|
autosave.flush().unwrap();
|
||||||
|
autosave.shutdown().unwrap();
|
||||||
|
|
||||||
|
let (reloaded, _) = load_notebook(&dir).unwrap();
|
||||||
|
assert_eq!(reloaded.current_page().strokes.len(), 2);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_autosave_never_writes_unloaded_placeholder() {
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let (mut notebook, mut manifest, dir) =
|
||||||
|
create_new_notebook(temp_dir.path(), "Platzhalter").unwrap();
|
||||||
|
notebook.add_page(PageBackground::Lined);
|
||||||
|
notebook.pages[1] = page_with_one_stroke(notebook.pages[1].id, PageBackground::Lined);
|
||||||
|
save_notebook(&dir, ¬ebook, &mut manifest).unwrap();
|
||||||
|
let page_file = dir.join(format!("page_{}.json", notebook.pages[1].id));
|
||||||
|
let before = fs::read(&page_file).unwrap();
|
||||||
|
|
||||||
|
// Platzhalter (nicht geladen) als aktuelle Seite markieren und speichern
|
||||||
|
let placeholder = Page::new_unloaded(notebook.pages[1].id, PageBackground::Lined);
|
||||||
|
let mut autosave = AutosaveController::new(dir.clone(), Duration::from_secs(60));
|
||||||
|
autosave.mark_page_dirty(&placeholder, &manifest);
|
||||||
|
autosave.flush().unwrap();
|
||||||
|
autosave.shutdown().unwrap();
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
fs::read(&page_file).unwrap(),
|
||||||
|
before,
|
||||||
|
"Seitendatei wurde überschrieben"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_new_shapes_round_trip_through_page_file() {
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let (mut notebook, mut manifest, dir) =
|
||||||
|
create_new_notebook(temp_dir.path(), "Neue Formen").unwrap();
|
||||||
|
let shapes = [
|
||||||
|
crate::geometry::RecognizedShape::Ellipse {
|
||||||
|
center: glam::Vec2::new(500.0, 600.0),
|
||||||
|
radius_x: 250.0,
|
||||||
|
radius_y: 120.0,
|
||||||
|
rotation: 0.3,
|
||||||
|
},
|
||||||
|
crate::geometry::RecognizedShape::Arrow {
|
||||||
|
start: glam::Vec2::new(100.0, 100.0),
|
||||||
|
end: glam::Vec2::new(700.0, 300.0),
|
||||||
|
double: true,
|
||||||
|
},
|
||||||
|
];
|
||||||
|
for (i, shape) in shapes.iter().enumerate() {
|
||||||
|
notebook
|
||||||
|
.current_page_mut()
|
||||||
|
.strokes
|
||||||
|
.push(crate::geometry::shape_to_stroke(
|
||||||
|
shape.clone(),
|
||||||
|
i as u64 + 1,
|
||||||
|
[0.0, 0.0, 0.0, 1.0],
|
||||||
|
5.0,
|
||||||
|
));
|
||||||
|
}
|
||||||
|
save_notebook(&dir, ¬ebook, &mut manifest).unwrap();
|
||||||
|
assert_eq!(
|
||||||
|
load_manifest(&dir).unwrap().format_version,
|
||||||
|
CURRENT_FORMAT_VERSION
|
||||||
|
);
|
||||||
|
|
||||||
|
let (reloaded, _) = load_notebook(&dir).unwrap();
|
||||||
|
let loaded: Vec<_> = reloaded
|
||||||
|
.current_page()
|
||||||
|
.strokes
|
||||||
|
.iter()
|
||||||
|
.map(|s| s.shape.clone())
|
||||||
|
.collect();
|
||||||
|
assert_eq!(loaded, shapes.iter().cloned().map(Some).collect::<Vec<_>>());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -12,7 +12,7 @@
|
|||||||
use std::time::Instant;
|
use std::time::Instant;
|
||||||
|
|
||||||
use crate::autosave::AutosaveStatus;
|
use crate::autosave::AutosaveStatus;
|
||||||
use crate::document::{Notebook, PageBackground};
|
use crate::document::{Notebook, Page, PageBackground, StrokeTool, PAGE_HEIGHT, PAGE_WIDTH};
|
||||||
use crate::settings::SidebarSide;
|
use crate::settings::SidebarSide;
|
||||||
use crate::storage::NotebookSummary;
|
use crate::storage::NotebookSummary;
|
||||||
|
|
||||||
@@ -96,6 +96,8 @@ pub enum UiAction {
|
|||||||
NextPage,
|
NextPage,
|
||||||
AddPage,
|
AddPage,
|
||||||
GoToPage(usize),
|
GoToPage(usize),
|
||||||
|
/// Alle noch nicht geladenen Seiten laden (für die Vorschaubilder der Seitenauswahl)
|
||||||
|
LoadAllPages,
|
||||||
SetBackground(PageBackground),
|
SetBackground(PageBackground),
|
||||||
ExportPdf,
|
ExportPdf,
|
||||||
ZoomIn,
|
ZoomIn,
|
||||||
@@ -699,8 +701,94 @@ fn color_circle_swatch(
|
|||||||
|
|
||||||
/// Spalten der Seitenauswahl.
|
/// Spalten der Seitenauswahl.
|
||||||
const PAGE_PICKER_COLUMNS: usize = 4;
|
const PAGE_PICKER_COLUMNS: usize = 4;
|
||||||
/// Größe einer Kachel der Seitenauswahl.
|
/// Größe einer Kachel der Seitenauswahl: A4-Vorschau plus Zeile für die Seitenzahl.
|
||||||
const PAGE_TILE: egui::Vec2 = egui::vec2(56.0, 48.0);
|
const PAGE_TILE: egui::Vec2 = egui::vec2(66.0, 66.0 * PAGE_HEIGHT / PAGE_WIDTH + 18.0);
|
||||||
|
/// Höchstzahl gezeichneter Punkte pro Vorschau (dünnt volle Seiten aus).
|
||||||
|
const THUMBNAIL_POINT_BUDGET: usize = 1500;
|
||||||
|
|
||||||
|
/// Zeichenbefehle für die verkleinerte Vorschau einer Seite in `rect` (A4-Format).
|
||||||
|
/// Striche werden als Linien gezeichnet und bei vielen Punkten gleichmäßig ausgedünnt.
|
||||||
|
pub fn page_thumbnail_shapes(page: &Page, rect: egui::Rect) -> Vec<egui::Shape> {
|
||||||
|
let mut shapes = vec![egui::Shape::rect_filled(
|
||||||
|
rect,
|
||||||
|
egui::Rounding::same(3.0),
|
||||||
|
egui::Color32::WHITE,
|
||||||
|
)];
|
||||||
|
let scale = rect.width() / PAGE_WIDTH;
|
||||||
|
let to_screen = |p: glam::Vec2| rect.min + egui::vec2(p.x * scale, p.y * scale);
|
||||||
|
|
||||||
|
let total: usize = page.strokes.iter().map(|s| s.points.len()).sum();
|
||||||
|
let step = total.div_ceil(THUMBNAIL_POINT_BUDGET).max(1);
|
||||||
|
|
||||||
|
for stroke in &page.strokes {
|
||||||
|
let mut color = color32_from(stroke.color);
|
||||||
|
if stroke.tool == StrokeTool::Highlighter {
|
||||||
|
color = color.gamma_multiply(0.45);
|
||||||
|
}
|
||||||
|
let width = (stroke.width * scale).clamp(0.6, 4.0);
|
||||||
|
let last = stroke.points.len().saturating_sub(1);
|
||||||
|
let points: Vec<egui::Pos2> = stroke
|
||||||
|
.points
|
||||||
|
.iter()
|
||||||
|
.enumerate()
|
||||||
|
.filter(|(i, _)| i % step == 0 || *i == last)
|
||||||
|
.map(|(_, s)| to_screen(s.position))
|
||||||
|
.collect();
|
||||||
|
match points.len() {
|
||||||
|
0 => {}
|
||||||
|
1 => shapes.push(egui::Shape::circle_filled(points[0], width / 2.0, color)),
|
||||||
|
_ => shapes.push(egui::Shape::line(points, egui::Stroke::new(width, color))),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
shapes
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Kachel der Seitenauswahl: Vorschau der Seite mit Seitenzahl darunter.
|
||||||
|
fn page_thumbnail(
|
||||||
|
ui: &mut egui::Ui,
|
||||||
|
page: &Page,
|
||||||
|
number: usize,
|
||||||
|
is_current: bool,
|
||||||
|
) -> egui::Response {
|
||||||
|
let (rect, response) = ui.allocate_exact_size(PAGE_TILE, egui::Sense::click());
|
||||||
|
if ui.is_rect_visible(rect) {
|
||||||
|
let thumb = egui::Rect::from_min_size(
|
||||||
|
rect.min,
|
||||||
|
egui::vec2(PAGE_TILE.x, PAGE_TILE.x * PAGE_HEIGHT / PAGE_WIDTH),
|
||||||
|
);
|
||||||
|
let painter = ui.painter();
|
||||||
|
painter.extend(page_thumbnail_shapes(page, thumb));
|
||||||
|
if !page.is_loaded {
|
||||||
|
painter.text(
|
||||||
|
thumb.center(),
|
||||||
|
egui::Align2::CENTER_CENTER,
|
||||||
|
"…",
|
||||||
|
egui::FontId::proportional(16.0),
|
||||||
|
egui::Color32::from_gray(140),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
let border = if is_current {
|
||||||
|
egui::Stroke::new(2.5_f32, egui::Color32::from_rgb(90, 150, 255))
|
||||||
|
} else if response.hovered() {
|
||||||
|
egui::Stroke::new(1.5_f32, egui::Color32::from_white_alpha(200))
|
||||||
|
} else {
|
||||||
|
egui::Stroke::new(1.0_f32, egui::Color32::from_white_alpha(60))
|
||||||
|
};
|
||||||
|
painter.rect_stroke(thumb, egui::Rounding::same(3.0), border);
|
||||||
|
painter.text(
|
||||||
|
egui::pos2(rect.center().x, rect.max.y - 8.0),
|
||||||
|
egui::Align2::CENTER_CENTER,
|
||||||
|
number.to_string(),
|
||||||
|
egui::FontId::proportional(13.0),
|
||||||
|
if is_current {
|
||||||
|
egui::Color32::WHITE
|
||||||
|
} else {
|
||||||
|
egui::Color32::from_gray(180)
|
||||||
|
},
|
||||||
|
);
|
||||||
|
}
|
||||||
|
response.on_hover_text(format!("Seite {number}"))
|
||||||
|
}
|
||||||
|
|
||||||
/// Rahmen der Popover neben der Werkzeugleiste.
|
/// Rahmen der Popover neben der Werkzeugleiste.
|
||||||
fn popover_frame() -> egui::Frame {
|
fn popover_frame() -> egui::Frame {
|
||||||
@@ -1696,24 +1784,19 @@ pub fn render_ui(
|
|||||||
if dialog_header(ui, &title, true) {
|
if dialog_header(ui, &title, true) {
|
||||||
state.show_page_jump_popup = false;
|
state.show_page_jump_popup = false;
|
||||||
}
|
}
|
||||||
|
// Noch nicht geladene Seiten für die Vorschau nachladen lassen
|
||||||
|
if notebook.pages.iter().any(|p| !p.is_loaded) {
|
||||||
|
actions.push(UiAction::LoadAllPages);
|
||||||
|
}
|
||||||
egui::ScrollArea::vertical()
|
egui::ScrollArea::vertical()
|
||||||
.max_height(320.0)
|
.max_height(420.0)
|
||||||
.show(ui, |ui| {
|
.show(ui, |ui| {
|
||||||
egui::Grid::new("page_picker_grid")
|
egui::Grid::new("page_picker_grid")
|
||||||
.spacing([8.0, 8.0])
|
.spacing([8.0, 8.0])
|
||||||
.show(ui, |ui| {
|
.show(ui, |ui| {
|
||||||
for idx in 0..notebook.pages.len() {
|
for (idx, page) in notebook.pages.iter().enumerate() {
|
||||||
let is_current = idx == notebook.current_page_index;
|
let is_current = idx == notebook.current_page_index;
|
||||||
let label =
|
if page_thumbnail(ui, page, idx + 1, is_current)
|
||||||
egui::RichText::new(format!("{}", idx + 1))
|
|
||||||
.size(16.0);
|
|
||||||
if ui
|
|
||||||
.add_sized(
|
|
||||||
[56.0, 48.0],
|
|
||||||
egui::SelectableLabel::new(
|
|
||||||
is_current, label,
|
|
||||||
),
|
|
||||||
)
|
|
||||||
.clicked()
|
.clicked()
|
||||||
{
|
{
|
||||||
actions.push(UiAction::GoToPage(idx));
|
actions.push(UiAction::GoToPage(idx));
|
||||||
@@ -3005,4 +3088,90 @@ mod tests {
|
|||||||
}
|
}
|
||||||
assert_eq!(reached, (0..6).collect(), "alle Seiten erreichbar");
|
assert_eq!(reached, (0..6).collect(), "alle Seiten erreichbar");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Seite mit `strokes` Strichen à `points` Punkten (diagonal über die Seite).
|
||||||
|
fn page_with_strokes(strokes: usize, points: usize) -> Page {
|
||||||
|
let mut page = Page::new(1, PageBackground::Grid5mm);
|
||||||
|
for i in 0..strokes {
|
||||||
|
let samples: Vec<crate::document::StylusSample> = (0..points)
|
||||||
|
.map(|j| crate::document::StylusSample {
|
||||||
|
position: glam::Vec2::new(
|
||||||
|
100.0 + j as f32 * 1800.0 / points as f32,
|
||||||
|
100.0 + i as f32 * 20.0,
|
||||||
|
),
|
||||||
|
pressure: 0.5,
|
||||||
|
timestamp: j as f64 * 0.01,
|
||||||
|
})
|
||||||
|
.collect();
|
||||||
|
page.strokes.push(crate::document::Stroke::new(
|
||||||
|
i as u64 + 1,
|
||||||
|
samples,
|
||||||
|
Vec::new(),
|
||||||
|
[0.1, 0.1, 0.1, 1.0],
|
||||||
|
));
|
||||||
|
}
|
||||||
|
page
|
||||||
|
}
|
||||||
|
|
||||||
|
fn shape_points(shapes: &[egui::Shape]) -> Vec<egui::Pos2> {
|
||||||
|
shapes
|
||||||
|
.iter()
|
||||||
|
.flat_map(|s| match s {
|
||||||
|
egui::Shape::Path(p) => p.points.clone(),
|
||||||
|
egui::Shape::Circle(c) => vec![c.center],
|
||||||
|
_ => Vec::new(),
|
||||||
|
})
|
||||||
|
.collect()
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_page_thumbnail_shapes() {
|
||||||
|
let rect = egui::Rect::from_min_size(egui::pos2(50.0, 80.0), egui::vec2(66.0, 93.0));
|
||||||
|
|
||||||
|
// Leere Seite: nur das weiße Blatt
|
||||||
|
let empty = Page::new(1, PageBackground::Blank);
|
||||||
|
assert_eq!(page_thumbnail_shapes(&empty, rect).len(), 1);
|
||||||
|
|
||||||
|
// Striche liegen innerhalb der Kachel
|
||||||
|
let page = page_with_strokes(3, 20);
|
||||||
|
let shapes = page_thumbnail_shapes(&page, rect);
|
||||||
|
assert_eq!(shapes.len(), 1 + 3);
|
||||||
|
let pts = shape_points(&shapes);
|
||||||
|
assert!(!pts.is_empty());
|
||||||
|
assert!(
|
||||||
|
pts.iter().all(|p| rect.contains(*p)),
|
||||||
|
"Punkte außerhalb der Kachel"
|
||||||
|
);
|
||||||
|
|
||||||
|
// Sehr volle Seite wird auf das Punktbudget ausgedünnt (+ Endpunkte je Strich)
|
||||||
|
let full = page_with_strokes(100, 1000);
|
||||||
|
let pts = shape_points(&page_thumbnail_shapes(&full, rect));
|
||||||
|
assert!(
|
||||||
|
pts.len() <= THUMBNAIL_POINT_BUDGET + 2 * 100,
|
||||||
|
"{} Punkte statt höchstens {}",
|
||||||
|
pts.len(),
|
||||||
|
THUMBNAIL_POINT_BUDGET + 200
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_page_picker_requests_loading_of_unloaded_pages() {
|
||||||
|
let updater = crate::updater::UpdaterController::new();
|
||||||
|
for (unloaded, expected) in [(false, false), (true, true)] {
|
||||||
|
let mut notebook = Notebook::new();
|
||||||
|
notebook.add_page(PageBackground::Grid5mm);
|
||||||
|
if unloaded {
|
||||||
|
notebook.pages[1] =
|
||||||
|
Page::new_unloaded(notebook.pages[1].id, PageBackground::Grid5mm);
|
||||||
|
}
|
||||||
|
let mut state = UiState::new();
|
||||||
|
state.show_page_jump_popup = true;
|
||||||
|
let ctx = render_frames(&mut state, ¬ebook, 3);
|
||||||
|
ctx.begin_frame(egui::RawInput::default());
|
||||||
|
let actions = render_ui(&ctx, &mut state, ¬ebook, AutosaveStatus::Saved, &updater);
|
||||||
|
let _ = ctx.end_frame();
|
||||||
|
let requested = actions.iter().any(|a| matches!(a, UiAction::LoadAllPages));
|
||||||
|
assert_eq!(requested, expected, "ungeladene Seite: {unloaded}");
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user