Hide cursor when menu opened

[mc-butler]

Important

This issue was migrated from Trac:

Origin	https://midnight-commander.org/ticket/1771
Reporter	sergey-feo (sergey-feo@….ru)

1. Please press F9 to open menu.
2. Open "Left" item.
3. Item "Listing mode" will be selected. Move cursor to "FTP link" item with down arrow key.

Selected item "FTP link" is colored in 2 ways:
1) All string "FTP link" is colored;
2) First symbol "F" is colored.

I suggest disable coloring of first symbol. It make view more clear IMHO.

Please see attached screenshots also.

Note

Original attachments:

• with_cursor.png (raw) by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 15:47 UTC
• without_cursor.png (raw) by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 15:48 UTC

[mc-butler]

Changed by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 15:47 UTC

• Attachment with_cursor.png (raw) added

[mc-butler]

Changed by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 15:48 UTC

• Attachment without_cursor.png (raw) added

[mc-butler]

Changed by andrew_b (@aborodin) on Oct 31, 2009 at 16:53 UTC (comment 1)

There is an alternative proposal: place cursor on the menu item hotkey (as in button).

[mc-butler]

Changed by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 17:27 UTC (comment 2)

This alternative will save unneeded additional coloring. Such coloring makes menu's view inaccurate IMHO.

[mc-butler]

Changed by sergey-feo (sergey-feo@….ru) on Oct 31, 2009 at 17:28 UTC (comment 3)

By the way: a suggest to disable cursor on buttons too.

[mc-butler]

Changed by andrew_b (@aborodin) on May 10, 2010 at 6:21 UTC (comment 4)

• Type changed from task to enhancement
• Severity changed from no branch to on review
• Milestone changed from Future Releases to 4.7.3
• Owner set to andrew_b
• Status changed from new to accepted

Created 1771_menu_hide_cursor branch. Parent branch is master.
[9139a1c9e4a35d8c2c3c7d7a65d2b5139f7b05c9]

[mc-butler]

Changed by angel_il (@ilia-maslakov) on May 11, 2010 at 5:48 UTC (comment 5)

• Votes set to angel_il

[mc-butler]

Changed by slavazanko (@slavaz) on May 21, 2010 at 20:45 UTC (comment 6)

• Votes angel_il deleted

Branch rebased into current master and added some changes:

• [7c018bb00fdc0f14cffa3857865943e50c979f01]: don't show cursor in menu.
• [f5dd9b7afb990890f9a0040cbcae67049024aad2]: Code cleanup for compile with -Werror option
• [206c6460f01033c97c3a7607c0f26826d7780b43]: don't show cursor in listboxes

Review again, please.

[mc-butler]

Changed by andrew_b (@aborodin) on May 22, 2010 at 7:31 UTC (comment 7)

[f5dd9b7afb990890f9a0040cbcae67049024aad2]: Code cleanup for compile with -Werror option

362	 	    int i = sizeof (s_split_direction) / sizeof (char *); 
 	362	    off_t i = sizeof (s_split_direction) / sizeof (char *); 

511	 	    int i; 
 	511	    off_t i; 

Why off_t but not size_t?

[mc-butler]

Changed by slavazanko (@slavaz) on May 24, 2010 at 8:57 UTC (comment 8)

Why off_t but not size_t?

See [a601e0598742a12c5aae74f7571a719c409ec158]

[mc-butler]

Changed by andrew_b (@aborodin) on May 24, 2010 at 9:22 UTC (comment 9)

• Votes set to andrew_b

[mc-butler]

Changed by angel_il (@ilia-maslakov) on May 25, 2010 at 18:21 UTC (comment 10)

• Votes changed from andrew_b to andrew_b angel_il

[mc-butler]

Changed by angel_il (@ilia-maslakov) on May 25, 2010 at 18:21 UTC (comment 11)

• Severity changed from on review to approved

[mc-butler]

Changed by andrew_b (@aborodin) on May 26, 2010 at 6:12 UTC (comment 12)

• Status changed from accepted to testing
• Votes changed from andrew_b angel_il to committed-master
• Resolution set to fixed
• Severity changed from approved to merged

Merged to master.
[6117c5b]

git log --pretty=oneline f513925..6117c5b

[mc-butler]

Changed by andrew_b (@aborodin) on May 26, 2010 at 6:12 UTC (comment 13)

• Status changed from testing to closed