bugGNU Octave - Bugs: bug #62984, Option to set 'current line...

 
 

bug #62984: Option to set 'current line highlight color' for Editor has disappeared since 7.x

Submitter:  Philip Nienhuis <philipnienhuis>
Submitted:  Mon 29 Aug 2022 09:55:45 PM UTC
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Feature Request
Status:  Confirmed Assigned to:  None
Originator Name:  Philip Nienhuis Open/Closed:  * Open
Release:  * 7.1.0 Operating System:  * Any
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

( Jump to the original submission )

Mon 05 Sep 2022 04:02:28 PM UTC, comment #9: 

> However, I can reinstate this preference with colors for each mode or (255,0,255) for determining the color automatically.

Thanks, would be very nice. Maybe first a separate patch to try it out & give feedback?

Philip Nienhuis <philipnienhuis>
Project Member
Fri 02 Sep 2022 10:50:42 AM UTC, comment #8: 

I have remove the settings while introducing the second color mode (dark mode) where the current line background would had to be adjusted manually when switching the mode.

However, I can reinstate this preference with colors for each mode or (255,0,255) for determining the color automatically.

Torsten Lilge <ttl>
Project Member
Fri 02 Sep 2022 08:58:05 AM UTC, comment #7: 

The line highlight color is very noticeably darker than the background color of the toolbars on my monitors (to give a reference).

I agree that changing the monitor settings per program is not a solution to anything. I was just suspecting that other programs might benefit as well from monitor settings that allow to distinguish colors that should be well distinguishable (at least in theory for most people)...

Markus Mützel <mmuetzel>
Project Member
Thu 01 Sep 2022 06:53:20 PM UTC, comment #6: 

Contrast and gamma settings are already optimal for most if not all other SW in use and apart from that I don't think it's the way to go to having to adapt them each time for just one program.

Did you try with some fairly dense text sections? It may be just an age thing (I'm almost at retirement age) but especially there it would be nice to have some flexibility.

I'd simply like to have some way to set the highlight background color to some personal preference. Too bad I didn't note earlier on that this option had been removed, and IMO fairly silently at that.
E.g., in the Variable Editor and Command pane this choice still exists, also with alternative color setting.

Philip Nienhuis <philipnienhuis>
Project Member
Thu 01 Sep 2022 06:24:39 AM UTC, comment #5: 

> I had a look in the cset Markus mentioned; I'm wondering how difficult it would be to reinstate the line background setting.


In case it should be reinstated, it should probably move from its old position to an entry in the table further down on that tab. It would probably be necessary to define different colors for the primary and secondary color scheme.

The line highlight is pretty visible on the monitors I tested with. Maybe, you could try to adjust the contrast (or gamma?) settings of your monitor in case it is hardly distinguishable for you.

Markus Mützel <mmuetzel>
Project Member
Wed 31 Aug 2022 08:49:33 PM UTC, comment #4: 

> @Philip: Which colors do you use as background and foreground in the editor?

The default ones: black on white. The background color is light gray, but I usually prefer some light blue as light gray doesn't stand out in sections with long "busy" lines.

I had a look in the cset Markus mentioned; I'm wondering how difficult it would be to reinstate the line background setting.
But currently I just don't have time for that :-(

Philip Nienhuis <philipnienhuis>
Project Member
Wed 31 Aug 2022 07:56:32 PM UTC, comment #3: 

Removing this user preference was a deliberate action as the background color of the current line is now "interpolated" between the background and foreground color in order to give a reasonable color regardless the back- and foreground color the user has selected. Obviously, it does not always lead to a reasonable color ...

The algorithm is in interpolate_color () in gui-utils.cc (comment in gui-utils.h https://hg.savannah.gnu.org/hgweb/octave/file/ad9822dedef8/libgui/src/gui-utils.h).

@Philip: Which colors do you use as background and foreground in the editor?

Torsten Lilge <ttl>
Project Member
Wed 31 Aug 2022 07:51:34 PM UTC, comment #2: 

Thanks, yeah I see it here:

:
* file-editor-tab.cc (file_edito_tab): read lexer settings
  depending on the current color mode, determine color of current
  line depending on background color (no more user pref)
:

IMO a bit unfortunate; the current background color doesn't stand out clearly on my monitor.

Philip Nienhuis <philipnienhuis>
Project Member
Wed 31 Aug 2022 02:04:10 PM UTC, comment #1: 

Afaict, that option was removed as part of this changeset that added the option of a secondary (dark) color scheme (see the changes to libgui/src/m-editor/file-editor-tab.cc):
https://hg.savannah.gnu.org/hgweb/octave/rev/5384bb4efc51

That might have been deliberate. Let's wait for Torsten's take.

Markus Mützel <mmuetzel>
Project Member
Mon 29 Aug 2022 09:55:45 PM UTC, original submission:  

(Hopefully) title says it all.
See attached screenshot of Octave-6.4.0's Preferences pane|Editor tab, where the color for the current line in the Editor could still be user-selected.
As of 7.1.0 this option has disappeared and these days the current line has a light gray background.  Too bad as on some of the monitors I use this background color is hard to distinguish quickly.

Was this a deliberate action? If so, should I set "Item Group" to "Feature request"?

(cc'ing Torsten)

Philip Nienhuis <philipnienhuis>
Project Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ttl (Posted a comment)
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by philipnienhuis (Submitted the item)
  • -email is unavailable- added by philipnienhuis
  •  

    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 project members can vote.

     

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-08-31 mmuetzel Item GroupOther Feature Request
        StatusNone Confirmed
    2022-08-29 philipnienhuis Attached File- Added Pref_Editor_currline_color.png, #53632
        Carbon-Copy- Added ttl

    Back to the top


    Powered by Savane 3.9