aleleba/figcraft
1
0
Fork 0
mirror of https://github.com/storytold/figcraft.git synced 2026-10-26 19:36:46 -06:00
Code Issues Packages Projects Releases Wiki Activity

#352: name the font list's height once

Browse Source 
The list's max height and the arrow keys' scroll offset now share one
LIST_HEIGHT const.
This commit is contained in:
bflatastic
2026-10-15 19:07:26 +07:00
parent cf06f2ca30
commit 62ed3452d6
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/ui-egui/src/typography.rs
+4 -2
View File
@@ -1065,6 +1065,8 @@ pub(crate) fn parse_letter_spacing(s: &str, cur: Option<LetterSpacing>) -> Optio
const RECENT_KEY: &str = "figcraft_recent_fonts";
const ROW: f32 = 26.0;
/// The font list's tallest (it scrolls past that).
const LIST_HEIGHT: f32 = 380.0;
fn recent_families(ctx: &egui::Context) -> Vec<String> {
ctx.data_mut(|d| d.get_persisted::<Vec<String>>(Id::new(RECENT_KEY))).unwrap_or_default()
@@ -1162,7 +1164,7 @@ pub(crate) fn font_picker(ui: &mut Ui, current: Option<&str>, just_opened: bool)
// Scrolled just enough to show it.
let seen: f32 = ui.data(|d| d.get_temp(hi_id.with("offset"))).unwrap_or(0.0);
let top = i as f32 * stride;
offset = Some(seen.max(top + stride - 380.0).min(top));
offset = Some(seen.max(top + stride - LIST_HEIGHT).min(top));
}
}
match hi {
@@ -1180,7 +1182,7 @@ pub(crate) fn font_picker(ui: &mut Ui, current: Option<&str>, just_opened: bool)
}
let mut budget = 4;
let mut picked = None;
let mut area = egui::ScrollArea::vertical().max_height(380.0).auto_shrink([false, true]);
let mut area = egui::ScrollArea::vertical().max_height(LIST_HEIGHT).auto_shrink([false, true]);
if let Some(y) = offset {
area = area.vertical_scroll_offset(y);
}