bugGNU Octave - Bugs: bug #65071, Brace Matching in Dark Mode is...

 
 

bug #65071: Brace Matching in Dark Mode is problematic

Submitter:  Bill Eaton <wpeaton>
Submitted:  Thu 21 Dec 2023 11:57:30 PM UTC
   
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Patch Submitted Assigned to:  None
Originator Name:  Open/Closed:  * Open
Release:  * 9.0.90 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 28 Jan 2024 08:52:51 PM UTC, comment #3: 

I tested the patch and, even though the matching brace highlighting color changed (now green), the cursor is still hardly visible for my aging eyes. Having a lighter grey background is still a better workaround for me.

Pantxo Diribarne <pantxo>
Group Member
Fri 26 Jan 2024 09:28:10 PM UTC, comment #2: 

I tried some other colors for the brace matching style. Please find attached a patch (for the default branch) that should work for light and dark themes.

(file #55619)

Torsten Lilge <ttl>
Group Member
Fri 12 Jan 2024 10:10:32 PM UTC, comment #1: 

I can confirm the issue with the current 9.0.90 which is to become Octave 9: the cursor is hardly visible when it is next to an highlighted matching parenthesis (fat white rectangle in dark mode). Changing the background color of the current line (in the editor color preferences) from the default dark gray to light gray helps and shows that the cursor is actually still blinking.


Pantxo Diribarne <pantxo>
Group Member
Thu 21 Dec 2023 11:57:30 PM UTC, original submission:  

I really like the new dark mode option in the editor. However, I find that it's impossible to see the cursor in the vicinity of braces, i.e. ()[]{}.

In dark mode, the cursor is usually a flashing vertical white bar. But it stops flashing in the vicinity of a brace character. And I find myself very often getting confused about where the cursor is.

This doesn't happen in light mode. The cursor is always flashing and it's a vertical black bar.

Bill Eaton <wpeaton>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #55619:  bug65071_v1.patch added by ttl (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ttl (Updated the item)
  • -email is unavailable- added by pantxo (Posted a comment)
  • -email is unavailable- added by wpeaton (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2024-01-26 ttl Attached File- Added bug65071_v1.patch, #55619
        StatusConfirmed Patch Submitted
    2024-01-12 pantxo StatusNone Confirmed
        Release8.2.0 9.0.90

    Back to the top

    Powered by Savane 3.12.
    Corresponding source code