Skip to content

Bug: fix highlighting on dead view mode#694

Merged
GuzekAlan merged 4 commits intomainfrom
681-bug-highlighting-in-deadviewmode-crashes
Aug 25, 2025
Merged

Bug: fix highlighting on dead view mode#694
GuzekAlan merged 4 commits intomainfrom
681-bug-highlighting-in-deadviewmode-crashes

Conversation

@GuzekAlan
Copy link
Contributor

No description provided.

@GuzekAlan GuzekAlan linked an issue Aug 21, 2025 that may be closed by this pull request
@GuzekAlan GuzekAlan requested review from hhubert6 and kraleppa August 21, 2025 10:37
Copy link
Contributor

@hhubert6 hhubert6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐬

Copy link
Member

@kraleppa kraleppa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a chance to create test for this?

@GuzekAlan GuzekAlan force-pushed the 681-bug-highlighting-in-deadviewmode-crashes branch from c9aa184 to caf8918 Compare August 24, 2025 14:48
@GuzekAlan GuzekAlan requested a review from kraleppa August 24, 2025 15:53
@GuzekAlan
Copy link
Contributor Author

Not sure if we want to do it but I can also disable inspecting when in DeadViewMode. AFAIC currently it will just redirect you to the successor when you try to inspect while in DeadViewMode

cc @kraleppa

@kraleppa
Copy link
Member

@GuzekAlan please do it - I think it is a bit misleading when you can inspect dead LiveView

@GuzekAlan GuzekAlan force-pushed the 681-bug-highlighting-in-deadviewmode-crashes branch from d69ab42 to 1845869 Compare August 25, 2025 08:19
@GuzekAlan GuzekAlan merged commit b5d612d into main Aug 25, 2025
2 checks passed
@GuzekAlan GuzekAlan deleted the 681-bug-highlighting-in-deadviewmode-crashes branch August 25, 2025 09:01
GuzekAlan added a commit that referenced this pull request Aug 25, 2025
* disable highlight on entering dead view mode

* add handle_info fallback

* add dead_view_mode e2e test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: highlighting in DeadViewMode crashes

3 participants