Mirrors
/
helix
mirror of https://github.com/helix-editor/helix
1
1
Fork 1
Code Issues Packages Projects Releases Wiki Activity

Revert "Fix precedence of ui.virtual.whitespace (#8750)"

Browse Source 
This reverts commit 41b307b673.
pull/8879/head
Blaž Hrastnik 1 year ago
parent 41b307b673
commit 811d62d3b3
No known key found for this signature in database
GPG Key ID: 1238B9C4AD889640
3 changed files with 65 additions and 125 deletions
Show Stats Download Patch File Download Diff File

84
helix-term/src/ui/document.rs
Unescape Escape View File

@ -97,8 +97,7 @@ pub fn render_document(
doc: &Document, doc: &Document,
offset: ViewPosition, offset: ViewPosition,
doc_annotations: &TextAnnotations, doc_annotations: &TextAnnotations,
syntax_highlight_iter: impl Iterator<Item = HighlightEvent>, highlight_iter: impl Iterator<Item = HighlightEvent>,
overlay_highlight_iter: impl Iterator<Item = HighlightEvent>,
theme: &Theme, theme: &Theme,
line_decoration: &mut [Box<dyn LineDecoration + '_>], line_decoration: &mut [Box<dyn LineDecoration + '_>],
translated_positions: &mut [TranslatedPosition], translated_positions: &mut [TranslatedPosition],
@ -110,8 +109,7 @@ pub fn render_document(
offset, offset,
&doc.text_format(viewport.width, Some(theme)), &doc.text_format(viewport.width, Some(theme)),
doc_annotations, doc_annotations,
syntax_highlight_iter, highlight_iter,
overlay_highlight_iter,
theme, theme,
line_decoration, line_decoration,
translated_positions, translated_positions,
@ -159,8 +157,7 @@ pub fn render_text<'t>(
offset: ViewPosition, offset: ViewPosition,
text_fmt: &TextFormat, text_fmt: &TextFormat,
text_annotations: &TextAnnotations, text_annotations: &TextAnnotations,
syntax_highlight_iter: impl Iterator<Item = HighlightEvent>, highlight_iter: impl Iterator<Item = HighlightEvent>,
overlay_highlight_iter: impl Iterator<Item = HighlightEvent>,
theme: &Theme, theme: &Theme,
line_decorations: &mut [Box<dyn LineDecoration + '_>], line_decorations: &mut [Box<dyn LineDecoration + '_>],
translated_positions: &mut [TranslatedPosition], translated_positions: &mut [TranslatedPosition],
@ -181,16 +178,10 @@ pub fn render_text<'t>(
let (mut formatter, mut first_visible_char_idx) = let (mut formatter, mut first_visible_char_idx) =
DocumentFormatter::new_at_prev_checkpoint(text, text_fmt, text_annotations, offset.anchor); DocumentFormatter::new_at_prev_checkpoint(text, text_fmt, text_annotations, offset.anchor);
let mut syntax_styles = StyleIter { let mut styles = StyleIter {
text_style: renderer.text_style, text_style: renderer.text_style,
active_highlights: Vec::with_capacity(64), active_highlights: Vec::with_capacity(64),
highlight_iter: syntax_highlight_iter, highlight_iter,
theme,
};
let mut overlay_styles = StyleIter {
text_style: renderer.text_style,
active_highlights: Vec::with_capacity(64),
highlight_iter: overlay_highlight_iter,
theme, theme,
}; };
@ -202,10 +193,7 @@ pub fn render_text<'t>(
}; };
let mut is_in_indent_area = true; let mut is_in_indent_area = true;
let mut last_line_indent_level = 0; let mut last_line_indent_level = 0;
let mut syntax_style_span = syntax_styles let mut style_span = styles
.next()
.unwrap_or_else(|| (Style::default(), usize::MAX));
let mut overlay_style_span = overlay_styles
.next() .next()
.unwrap_or_else(|| (Style::default(), usize::MAX)); .unwrap_or_else(|| (Style::default(), usize::MAX));
@ -233,16 +221,9 @@ pub fn render_text<'t>(
// skip any graphemes on visual lines before the block start // skip any graphemes on visual lines before the block start
if pos.row < row_off { if pos.row < row_off {
if char_pos >= syntax_style_span.1 { if char_pos >= style_span.1 {
syntax_style_span = if let Some(syntax_style_span) = syntax_styles.next() { style_span = if let Some(style_span) = styles.next() {
syntax_style_span style_span
} else {
break;
}
}
if char_pos >= overlay_style_span.1 {
overlay_style_span = if let Some(overlay_style_span) = overlay_styles.next() {
overlay_style_span
} else { } else {
break; break;
} }
@ -279,15 +260,8 @@ pub fn render_text<'t>(
} }
// acquire the correct grapheme style // acquire the correct grapheme style
if char_pos >= syntax_style_span.1 { if char_pos >= style_span.1 {
syntax_style_span = syntax_styles style_span = styles.next().unwrap_or((Style::default(), usize::MAX));
.next()
.unwrap_or((Style::default(), usize::MAX));
}
if char_pos >= overlay_style_span.1 {
overlay_style_span = overlay_styles
.next()
.unwrap_or((Style::default(), usize::MAX));
} }
char_pos += grapheme.doc_chars(); char_pos += grapheme.doc_chars();
@ -301,25 +275,22 @@ pub fn render_text<'t>(
pos, pos,
); );
let (syntax_style, overlay_style) = let grapheme_style = if let GraphemeSource::VirtualText { highlight } = grapheme.source {
if let GraphemeSource::VirtualText { highlight } = grapheme.source { let style = renderer.text_style;
let mut style = renderer.text_style; if let Some(highlight) = highlight {
if let Some(highlight) = highlight { style.patch(theme.highlight(highlight.0))
style = style.patch(theme.highlight(highlight.0))
}
(style, Style::default())
} else { } else {
(syntax_style_span.0, overlay_style_span.0) style
}; }
} else {
style_span.0
};
let is_virtual = grapheme.is_virtual(); let virt = grapheme.is_virtual();
renderer.draw_grapheme( renderer.draw_grapheme(
grapheme.grapheme, grapheme.grapheme,
GraphemeStyle { grapheme_style,
syntax_style, virt,
overlay_style,
},
is_virtual,
&mut last_line_indent_level, &mut last_line_indent_level,
&mut is_in_indent_area, &mut is_in_indent_area,
pos, pos,
@ -351,11 +322,6 @@ pub struct TextRenderer<'a> {
pub viewport: Rect, pub viewport: Rect,
} }
pub struct GraphemeStyle {
syntax_style: Style,
overlay_style: Style,
}
impl<'a> TextRenderer<'a> { impl<'a> TextRenderer<'a> {
pub fn new( pub fn new(
surface: &'a mut Surface, surface: &'a mut Surface,
@ -429,7 +395,7 @@ impl<'a> TextRenderer<'a> {
pub fn draw_grapheme( pub fn draw_grapheme(
&mut self, &mut self,
grapheme: Grapheme, grapheme: Grapheme,
grapheme_style: GraphemeStyle, mut style: Style,
is_virtual: bool, is_virtual: bool,
last_indent_level: &mut usize, last_indent_level: &mut usize,
is_in_indent_area: &mut bool, is_in_indent_area: &mut bool,
@ -439,11 +405,9 @@ impl<'a> TextRenderer<'a> {
let is_whitespace = grapheme.is_whitespace(); let is_whitespace = grapheme.is_whitespace();
// TODO is it correct to apply the whitespace style to all unicode white spaces? // TODO is it correct to apply the whitespace style to all unicode white spaces?
let mut style = grapheme_style.syntax_style;
if is_whitespace { if is_whitespace {
style = style.patch(self.whitespace_style); style = style.patch(self.whitespace_style);
} }
style = style.patch(grapheme_style.overlay_style);
let width = grapheme.width(); let width = grapheme.width();
let space = if is_virtual { " " } else { &self.space }; let space = if is_virtual { " " } else { &self.space };

96
helix-term/src/ui/editor.rs
Unescape Escape View File

@ -124,20 +124,16 @@ impl EditorView {
line_decorations.push(Box::new(line_decoration)); line_decorations.push(Box::new(line_decoration));
} }
let syntax_highlights = let mut highlights =
Self::doc_syntax_highlights(doc, view.offset.anchor, inner.height, theme); Self::doc_syntax_highlights(doc, view.offset.anchor, inner.height, theme);
let overlay_highlights = Self::overlay_syntax_highlights(
let mut overlay_highlights =
Self::empty_highlight_iter(doc, view.offset.anchor, inner.height);
let overlay_syntax_highlights = Self::overlay_syntax_highlights(
doc, doc,
view.offset.anchor, view.offset.anchor,
inner.height, inner.height,
&text_annotations, &text_annotations,
); );
if !overlay_syntax_highlights.is_empty() { if !overlay_highlights.is_empty() {
overlay_highlights = highlights = Box::new(syntax::merge(highlights, overlay_highlights));
Box::new(syntax::merge(overlay_highlights, overlay_syntax_highlights));
} }
for diagnostic in Self::doc_diagnostics_highlights(doc, theme) { for diagnostic in Self::doc_diagnostics_highlights(doc, theme) {
@ -146,12 +142,12 @@ impl EditorView {
if diagnostic.is_empty() { if diagnostic.is_empty() {
continue; continue;
} }
overlay_highlights = Box::new(syntax::merge(overlay_highlights, diagnostic)); highlights = Box::new(syntax::merge(highlights, diagnostic));
} }
if is_focused { let highlights: Box<dyn Iterator<Item = HighlightEvent>> = if is_focused {
let highlights = syntax::merge( let highlights = syntax::merge(
overlay_highlights, highlights,
Self::doc_selection_highlights( Self::doc_selection_highlights(
editor.mode(), editor.mode(),
doc, doc,
@ -162,11 +158,13 @@ impl EditorView {
); );
let focused_view_elements = Self::highlight_focused_view_elements(view, doc, theme); let focused_view_elements = Self::highlight_focused_view_elements(view, doc, theme);
if focused_view_elements.is_empty() { if focused_view_elements.is_empty() {
overlay_highlights = Box::new(highlights) Box::new(highlights)
} else { } else {
overlay_highlights = Box::new(syntax::merge(highlights, focused_view_elements)) Box::new(syntax::merge(highlights, focused_view_elements))
} }
} } else {
Box::new(highlights)
};
let gutter_overflow = view.gutter_offset(doc) == 0; let gutter_overflow = view.gutter_offset(doc) == 0;
if !gutter_overflow { if !gutter_overflow {
@ -199,8 +197,7 @@ impl EditorView {
doc, doc,
view.offset, view.offset,
&text_annotations, &text_annotations,
syntax_highlights, highlights,
overlay_highlights,
theme, theme,
&mut line_decorations, &mut line_decorations,
&mut translated_positions, &mut translated_positions,
@ -260,39 +257,27 @@ impl EditorView {
.for_each(|area| surface.set_style(area, ruler_theme)) .for_each(|area| surface.set_style(area, ruler_theme))
} }
fn viewport_byte_range( pub fn overlay_syntax_highlights(
text: helix_core::RopeSlice,
row: usize,
height: u16,
) -> std::ops::Range<usize> {
// Calculate viewport byte ranges:
// Saturating subs to make it inclusive zero indexing.
let last_line = text.len_lines().saturating_sub(1);
let last_visible_line = (row + height as usize).saturating_sub(1).min(last_line);
let start = text.line_to_byte(row.min(last_line));
let end = text.line_to_byte(last_visible_line + 1);
start..end
}
pub fn empty_highlight_iter(
doc: &Document, doc: &Document,
anchor: usize, anchor: usize,
height: u16, height: u16,
) -> Box<dyn Iterator<Item = HighlightEvent>> { text_annotations: &TextAnnotations,
) -> Vec<(usize, std::ops::Range<usize>)> {
let text = doc.text().slice(..); let text = doc.text().slice(..);
let row = text.char_to_line(anchor.min(text.len_chars())); let row = text.char_to_line(anchor.min(text.len_chars()));
// Calculate viewport byte ranges: let range = {
// Saturating subs to make it inclusive zero indexing. // Calculate viewport byte ranges:
let range = Self::viewport_byte_range(text, row, height); // Saturating subs to make it inclusive zero indexing.
Box::new( let last_line = text.len_lines().saturating_sub(1);
[HighlightEvent::Source { let last_visible_line = (row + height as usize).saturating_sub(1).min(last_line);
start: text.byte_to_char(range.start), let start = text.line_to_byte(row.min(last_line));
end: text.byte_to_char(range.end), let end = text.line_to_byte(last_visible_line + 1);
}]
.into_iter(), start..end
) };
text_annotations.collect_overlay_highlights(range)
} }
/// Get syntax highlights for a document in a view represented by the first line /// Get syntax highlights for a document in a view represented by the first line
@ -307,7 +292,16 @@ impl EditorView {
let text = doc.text().slice(..); let text = doc.text().slice(..);
let row = text.char_to_line(anchor.min(text.len_chars())); let row = text.char_to_line(anchor.min(text.len_chars()));
let range = Self::viewport_byte_range(text, row, height); let range = {
// Calculate viewport byte ranges:
// Saturating subs to make it inclusive zero indexing.
let last_line = text.len_lines().saturating_sub(1);
let last_visible_line = (row + height as usize).saturating_sub(1).min(last_line);
let start = text.line_to_byte(row.min(last_line));
let end = text.line_to_byte(last_visible_line + 1);
start..end
};
match doc.syntax() { match doc.syntax() {
Some(syntax) => { Some(syntax) => {
@ -340,20 +334,6 @@ impl EditorView {
} }
} }
pub fn overlay_syntax_highlights(
doc: &Document,
anchor: usize,
height: u16,
text_annotations: &TextAnnotations,
) -> Vec<(usize, std::ops::Range<usize>)> {
let text = doc.text().slice(..);
let row = text.char_to_line(anchor.min(text.len_chars()));
let range = Self::viewport_byte_range(text, row, height);
text_annotations.collect_overlay_highlights(range)
}
/// Get highlight spans for document diagnostics /// Get highlight spans for document diagnostics
pub fn doc_diagnostics_highlights( pub fn doc_diagnostics_highlights(
doc: &Document, doc: &Document,

10
helix-term/src/ui/picker.rs
Unescape Escape View File

@ -736,20 +736,17 @@ impl<T: Item + 'static> Picker<T> {
} }
} }
let syntax_highlights = EditorView::doc_syntax_highlights( let mut highlights = EditorView::doc_syntax_highlights(
doc, doc,
offset.anchor, offset.anchor,
area.height, area.height,
&cx.editor.theme, &cx.editor.theme,
); );
let mut overlay_highlights =
EditorView::empty_highlight_iter(doc, offset.anchor, area.height);
for spans in EditorView::doc_diagnostics_highlights(doc, &cx.editor.theme) { for spans in EditorView::doc_diagnostics_highlights(doc, &cx.editor.theme) {
if spans.is_empty() { if spans.is_empty() {
continue; continue;
} }
overlay_highlights = Box::new(helix_core::syntax::merge(overlay_highlights, spans)); highlights = Box::new(helix_core::syntax::merge(highlights, spans));
} }
let mut decorations: Vec<Box<dyn LineDecoration>> = Vec::new(); let mut decorations: Vec<Box<dyn LineDecoration>> = Vec::new();
@ -780,8 +777,7 @@ impl<T: Item + 'static> Picker<T> {
offset, offset,
// TODO: compute text annotations asynchronously here (like inlay hints) // TODO: compute text annotations asynchronously here (like inlay hints)
&TextAnnotations::default(), &TextAnnotations::default(),
syntax_highlights, highlights,
overlay_highlights,
&cx.editor.theme, &cx.editor.theme,
&mut decorations, &mut decorations,
&mut [], &mut [],

Write Preview
Loading…
Cancel
Save