system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-10-15 12:23:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

HackStudio: Remove adjustment of text range for documentation tooltip

Browse source 
We no longer need to adjust the hovered span's range to get the correct
hovered text. This fixes the "documentation tooltip" feature.
This commit is contained in:
Itamar 2022-10-06 15:24:41 +03:00 committed by Sam Atkins
parent 84290ed7c8
commit 2046a670b3
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/DevTools/HackStudio/Editor.cpp
View file

@ -263,11 +263,8 @@ void Editor::mousemove_event(GUI::MouseEvent& event)
}
if (span.range.contains(text_position)) {
auto adjusted_range = span.range;
auto end_line_length = document().line(span.range.end().line()).length();
adjusted_range.end().set_column(min(end_line_length, adjusted_range.end().column() + 1));
auto hovered_span_text = document().text_in_range(adjusted_range);
dbgln_if(EDITOR_DEBUG, "Hovering: {} \"{}\"", adjusted_range, hovered_span_text);
auto hovered_span_text = document().text_in_range(span.range);
dbgln_if(EDITOR_DEBUG, "Hovering: {} \"{}\"", span.range, hovered_span_text);
if (is_clickable) {
is_over_clickable = true;