1
0
Fork 0
mirror of https://github.com/alacritty/alacritty.git synced 2024-11-18 13:55:23 -05:00

Powerline glyphs being cut for narrow fonts

Fixes #7470.
This commit is contained in:
Kirill Chibisov 2023-12-31 00:12:41 +04:00 committed by Christian Duerr
parent e297c6bf0c
commit 22abb78c35
2 changed files with 15 additions and 4 deletions

View file

@ -15,6 +15,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
- `chars = "\u000A"` action in bindings inserting `\n` - `chars = "\u000A"` action in bindings inserting `\n`
- Alternate keys not sent for `Shift + <number>` when using kitty protocol - Alternate keys not sent for `Shift + <number>` when using kitty protocol
- Alternative keys being swapped in kitty protocol implementation - Alternative keys being swapped in kitty protocol implementation
- Powerline glyphs being cut for narrow fonts
## 0.13.0 ## 0.13.0

View file

@ -32,7 +32,7 @@ pub fn builtin_glyph(
'\u{2500}'..='\u{259f}' => box_drawing(character, metrics, offset), '\u{2500}'..='\u{259f}' => box_drawing(character, metrics, offset),
// Powerline symbols: '','','','' // Powerline symbols: '','','',''
POWERLINE_TRIANGLE_LTR..=POWERLINE_ARROW_RTL => { POWERLINE_TRIANGLE_LTR..=POWERLINE_ARROW_RTL => {
powerline_drawing(character, metrics, offset) powerline_drawing(character, metrics, offset)?
}, },
_ => return None, _ => return None,
}; };
@ -503,7 +503,11 @@ fn box_drawing(character: char, metrics: &Metrics, offset: &Delta<i8>) -> Raster
} }
} }
fn powerline_drawing(character: char, metrics: &Metrics, offset: &Delta<i8>) -> RasterizedGlyph { fn powerline_drawing(
character: char,
metrics: &Metrics,
offset: &Delta<i8>,
) -> Option<RasterizedGlyph> {
let height = (metrics.line_height as i32 + offset.y as i32) as usize; let height = (metrics.line_height as i32 + offset.y as i32) as usize;
let width = (metrics.average_advance as i32 + offset.x as i32) as usize; let width = (metrics.average_advance as i32 + offset.x as i32) as usize;
let extra_thickness = calculate_stroke_size(width) as i32 - 1; let extra_thickness = calculate_stroke_size(width) as i32 - 1;
@ -519,6 +523,12 @@ fn powerline_drawing(character: char, metrics: &Metrics, offset: &Delta<i8>) ->
// x = (H - 2) / (2 * slope). // x = (H - 2) / (2 * slope).
let x_intersection = (height as i32 + 1) / 2 - 1; let x_intersection = (height as i32 + 1) / 2 - 1;
// Don't use built-in font if we'd cut the tip too much, for example when the font is really
// narrow.
if x_intersection - width as i32 > 1 {
return None;
}
let top_line = (0..x_intersection).map(|x| line_equation(slope, x, top_y)); let top_line = (0..x_intersection).map(|x| line_equation(slope, x, top_y));
let bottom_line = (0..x_intersection).map(|x| line_equation(-slope, x, bottom_y)); let bottom_line = (0..x_intersection).map(|x| line_equation(-slope, x, bottom_y));
@ -555,7 +565,7 @@ fn powerline_drawing(character: char, metrics: &Metrics, offset: &Delta<i8>) ->
let top = height as i32 + metrics.descent as i32; let top = height as i32 + metrics.descent as i32;
let buffer = BitmapBuffer::Rgb(canvas.into_raw()); let buffer = BitmapBuffer::Rgb(canvas.into_raw());
RasterizedGlyph { Some(RasterizedGlyph {
character, character,
top, top,
left: 0, left: 0,
@ -563,7 +573,7 @@ fn powerline_drawing(character: char, metrics: &Metrics, offset: &Delta<i8>) ->
width: width as i32, width: width as i32,
buffer, buffer,
advance: (width as i32, height as i32), advance: (width as i32, height as i32),
} })
} }
#[repr(packed)] #[repr(packed)]