disableVisualSelection doesn't work for headers

[mrpiotr-dev]

Description

disableVisualSelection doesn't work for headers. It doesn't matter if we set 'header' or true - headers will be highlighted.

Related issue: #5869

Steps to reproduce

1. Initialize Handsontable with the following configuration:

rowHeaders: true,
colHeaders: true,
disableVisualSelection: true,

2. Select column A or 2nd row by a header.

Demo

https://jsfiddle.net/handsoncode/8ffpsqt6/

Your environment

• Handsontable version: 2.0.0+
• Browser Name and version: any
• Operating System: any

[aninde]

This bug can be replicable with 8.0.0-beta2-rev19 https://jsfiddle.net/aninde/92wa1txc/

[pnowak]

This issue is still replicable using v8.0.0. Here's an updated demo - https://jsfiddle.net/xvq9yw8u/

[AMBudnik]

It works for me (header is not highlighted when we use disableVisualSelection: 'header') on Chrome 87/Firefox 84/Edge (Windows 10) using v7.4.2 and newer https://jsfiddle.net/handsoncode/6m1j5kxd/

EDIT: I've just tested it with macOS Catalina and Chrome. Also works fine.

[wszymanski]

It works for me (header is not highlighted when we use disableVisualSelection: 'header') on Chrome 87/Firefox 84/Edge (Windows 10) using v7.4.2 and newer jsfiddle.net/handsoncode/6m1j5kxd

EDIT: I've just tested it with macOS Catalina and Chrome. Also works fine.

The issue is addressing problem only for clicks on headers :) The disableVisualSelection option set to value 'header' works properly for clicks on cells now.

[aninde]

This bug has been fixed on v8.3.2. https://jsfiddle.net/e19odr0g/.
Setting disableVisualSelection on true or on header causes the inability to select headers.

More information examples of working disableVisualSelection can be found here.

[AMBudnik]

I'm closing this issue as solved and released under Handsontable v8.3.2