Merge pull request #54 from hecrj/feature/external-fonts

Custom font support

3 files changed, 32 insertions, 95 deletions

diff --git a/wgpu/src/renderer.rs b/wgpu/src/renderer.rs
index e9df3550..d3bdc878 100644
--- a/wgpu/src/renderer.rs
+++ b/wgpu/src/renderer.rs
@@ -1,4 +1,4 @@
-use crate::{font, quad, Image, Primitive, Quad, Transformation};
+use crate::{quad, text, Image, Primitive, Quad, Transformation};
 use iced_native::{
     renderer::Debugger, renderer::Windowed, Background, Color, Layout,
     MouseCursor, Point, Rectangle, Vector, Widget,
@@ -7,11 +7,8 @@ use iced_native::{
 use wgpu::{
     Adapter, BackendBit, CommandEncoderDescriptor, Device, DeviceDescriptor,
     Extensions, Limits, PowerPreference, Queue, RequestAdapterOptions,
-    TextureFormat,
 };
 
-use std::cell::RefCell;
-
 mod target;
 mod widget;
 
@@ -22,8 +19,7 @@ pub struct Renderer {
     queue: Queue,
     quad_pipeline: quad::Pipeline,
     image_pipeline: crate::image::Pipeline,
-    text_pipeline: wgpu_glyph::GlyphBrush<'static, ()>,
-    text_measurements: RefCell<glyph_brush::GlyphBrush<'static, ()>>,
+    text_pipeline: text::Pipeline,
 }
 
 pub struct Layer<'a> {
@@ -61,28 +57,7 @@ impl Renderer {
             limits: Limits { max_bind_groups: 2 },
         });
 
-        // TODO: Font customization
-        let font_source = font::Source::new();
-        let default_font = font_source
-            .load(&[font::Family::SansSerif, font::Family::Serif])
-            .expect("Find sans-serif or serif font");
-
-        let mono_font = font_source
-            .load(&[font::Family::Monospace])
-            .expect("Find monospace font");
-
-        let text_pipeline =
-            wgpu_glyph::GlyphBrushBuilder::using_fonts_bytes(vec![
-                default_font.clone(),
-                mono_font,
-            ])
-            .initial_cache_size((2048, 2048))
-            .build(&mut device, TextureFormat::Bgra8UnormSrgb);
-
-        let text_measurements =
-            glyph_brush::GlyphBrushBuilder::using_font_bytes(default_font)
-                .build();
-
+        let text_pipeline = text::Pipeline::new(&mut device);
         let quad_pipeline = quad::Pipeline::new(&mut device);
         let image_pipeline = crate::image::Pipeline::new(&mut device);
 
@@ -92,7 +67,6 @@ impl Renderer {
             quad_pipeline,
             image_pipeline,
             text_pipeline,
-            text_measurements: RefCell::new(text_measurements),
         }
     }
 
@@ -173,6 +147,7 @@ impl Renderer {
                 bounds,
                 size,
                 color,
+                font,
                 horizontal_alignment,
                 vertical_alignment,
             } => {
@@ -205,6 +180,7 @@ impl Renderer {
                     bounds: (bounds.width, bounds.height),
                     scale: wgpu_glyph::Scale { x: *size, y: *size },
                     color: color.into_linear(),
+                    font_id: self.text_pipeline.find_font(*font),
                     layout: wgpu_glyph::Layout::default()
                         .h_align(match horizontal_alignment {
                             iced_native::text::HorizontalAlignment::Left => {
@@ -298,7 +274,7 @@ impl Renderer {
         let first = layers.first().unwrap();
         let mut overlay = Layer::new(first.bounds, Vector::new(0, 0));
 
-        let font_id = wgpu_glyph::FontId(self.text_pipeline.fonts().len() - 1);
+        let font_id = self.text_pipeline.overlay_font();
         let scale = wgpu_glyph::Scale { x: 20.0, y: 20.0 };
 
         for (i, line) in lines.iter().enumerate() {
@@ -383,20 +359,18 @@ impl Renderer {
                 self.text_pipeline.queue(text);
             }
 
-            self.text_pipeline
-                .draw_queued_with_transform_and_scissoring(
-                    &mut self.device,
-                    encoder,
-                    target,
-                    transformation.into(),
-                    wgpu_glyph::Region {
-                        x: bounds.x,
-                        y: bounds.y,
-                        width: bounds.width,
-                        height: bounds.height,
-                    },
-                )
-                .expect("Draw text");
+            self.text_pipeline.draw_queued(
+                &mut self.device,
+                encoder,
+                target,
+                transformation,
+                wgpu_glyph::Region {
+                    x: bounds.x,
+                    y: bounds.y,
+                    width: bounds.width,
+                    height: bounds.height,
+                },
+            );
         }
     }
 }
@@ -410,15 +384,7 @@ impl iced_native::Renderer for Renderer {
     ) -> iced_native::layout::Node {
         let node = element.layout(self, &iced_native::layout::Limits::NONE);
 
-        // Trim measurements cache
-        // TODO: We should probably use a `GlyphCalculator` for this. However,
-        // it uses a lifetimed `GlyphCalculatorGuard` with side-effects on drop.
-        // This makes stuff quite inconvenient. A manual method for trimming the
-        // cache would make our lives easier.
-        self.text_measurements
-            .borrow_mut()
-            .process_queued(|_, _| {}, |_| {})
-            .expect("Trim text measurements");
+        self.text_pipeline.clear_measurement_cache();
 
         node
     }
diff --git a/wgpu/src/renderer/widget/text.rs b/wgpu/src/renderer/widget/text.rs
index b9ccd787..a8ead70b 100644
--- a/wgpu/src/renderer/widget/text.rs
+++ b/wgpu/src/renderer/widget/text.rs
@@ -1,8 +1,6 @@
 use crate::{Primitive, Renderer};
 use iced_native::{layout, text, Color, Layout, MouseCursor, Size, Text};
 
-use wgpu_glyph::{GlyphCruncher, Section};
-
 use std::f32;
 
 // TODO: Obtain from renderer configuration
@@ -14,20 +12,9 @@ impl text::Renderer for Renderer {
         let size = text.size.map(f32::from).unwrap_or(DEFAULT_TEXT_SIZE);
         let bounds = limits.max();
 
-        let section = Section {
-            text: &text.content,
-            scale: wgpu_glyph::Scale { x: size, y: size },
-            bounds: (bounds.width, bounds.height),
-            ..Default::default()
-        };
-
-        let (width, height) = if let Some(bounds) =
-            self.text_measurements.borrow_mut().glyph_bounds(&section)
-        {
-            (bounds.width().ceil(), bounds.height().ceil())
-        } else {
-            (0.0, 0.0)
-        };
+        let (width, height) =
+            self.text_pipeline
+                .measure(&text.content, size, text.font, bounds);
 
         let size = limits.resolve(Size::new(width, height));
 
@@ -41,6 +28,7 @@ impl text::Renderer for Renderer {
                 size: text.size.map(f32::from).unwrap_or(DEFAULT_TEXT_SIZE),
                 bounds: layout.bounds(),
                 color: text.color.unwrap_or(Color::BLACK),
+                font: text.font,
                 horizontal_alignment: text.horizontal_alignment,
                 vertical_alignment: text.vertical_alignment,
             },
diff --git a/wgpu/src/renderer/widget/text_input.rs b/wgpu/src/renderer/widget/text_input.rs
index 75eb20f7..855e945c 100644
--- a/wgpu/src/renderer/widget/text_input.rs
+++ b/wgpu/src/renderer/widget/text_input.rs
@@ -2,7 +2,7 @@ use crate::{Primitive, Renderer};
 
 use iced_native::{
     text::HorizontalAlignment, text::VerticalAlignment, text_input, Background,
-    Color, MouseCursor, Point, Rectangle, TextInput, Vector,
+    Color, Font, MouseCursor, Point, Rectangle, Size, TextInput, Vector,
 };
 use std::f32;
 
@@ -60,6 +60,7 @@ impl text_input::Renderer for Renderer {
                 [0.3, 0.3, 0.3]
             }
             .into(),
+            font: Font::Default,
             bounds: Rectangle {
                 width: f32::INFINITY,
                 ..text_bounds
@@ -70,41 +71,23 @@ impl text_input::Renderer for Renderer {
         };
 
         let (contents_primitive, offset) = if text_input.state.is_focused {
-            use wgpu_glyph::{GlyphCruncher, Scale, Section};
-
             let text_before_cursor = &text_input
                 .value
                 .until(text_input.state.cursor_position(&text_input.value))
                 .to_string();
 
-            let mut text_value_width = self
-                .text_measurements
-                .borrow_mut()
-                .glyph_bounds(Section {
-                    text: text_before_cursor,
-                    bounds: (f32::INFINITY, text_bounds.height),
-                    scale: Scale { x: size, y: size },
-                    ..Default::default()
-                })
-                .map(|bounds| bounds.width().round())
-                .unwrap_or(0.0);
+            let (mut text_value_width, _) = self.text_pipeline.measure(
+                text_before_cursor,
+                size,
+                Font::Default,
+                Size::new(f32::INFINITY, text_bounds.height),
+            );
 
             let spaces_at_the_end =
                 text_before_cursor.len() - text_before_cursor.trim_end().len();
 
             if spaces_at_the_end > 0 {
-                let space_width = {
-                    let glyph_brush = self.text_measurements.borrow();
-
-                    // TODO: Select appropriate font
-                    let font = &glyph_brush.fonts()[0];
-
-                    font.glyph(' ')
-                        .scaled(Scale { x: size, y: size })
-                        .h_metrics()
-                        .advance_width
-                };
-
+                let space_width = self.text_pipeline.space_width(size);
                 text_value_width += spaces_at_the_end as f32 * space_width;
             }