[8.0.0-beta.2-rev13] Selecting a partially-hidden merged cell gives it a wrong color overlay.

[jansiegel]

Description

When selecting a partially-hidden merged cell, it gets a blue overlay, and it shouldn't.

Works fine until rev13.

Steps to reproduce

1. Go to https://jsfiddle.net/js_ziggle/x29rhkgt/
2. Select the merged cell at (1, 1)

Demo

https://jsfiddle.net/js_ziggle/x29rhkgt/

[wszymanski]

The below test hasn't guaranteed proper work of the case as we can see.

handsontable/src/plugins/hiddenColumns/test/hiddenColumns.e2e.js

Lines 5800 to 5844 in 7fbf9c8

	it('should select single merged area properly when it starts with hidden column', () => {
	handsontable({
	data: Handsontable.helper.createSpreadsheetData(5, 5),
	rowHeaders: true,
	colHeaders: true,
	contextMenu: true,
	hiddenColumns: {
	columns: [1],
	indicators: true
	},
	mergeCells: [{ row: 1, col: 1, rowspan: 3, colspan: 3 }]
	});
	const mergedCell = spec().$container.find('tr:eq(2) td:eq(1)');
	simulateClick(mergedCell);
	// Third column is not displayed (CSS - display: none).
	expect(`
	| ║ : - : - : |
	|===:===:===:===:===|
	| ║ : : : |
	| - ║ : # : : |
	| - ║ : : : |
	| - ║ : : : |
	| ║ : : : |
	`).toBeMatchToSelectionPattern();
	expect(getSelected()).toEqual([[1, 1, 3, 3]]);
	expect(getSelectedRangeLast().highlight.row).toBe(1);
	expect(getSelectedRangeLast().highlight.col).toBe(2);
	expect(getSelectedRangeLast().from.row).toBe(1);
	expect(getSelectedRangeLast().from.col).toBe(1);
	expect(getSelectedRangeLast().to.row).toBe(3);
	expect(getSelectedRangeLast().to.col).toBe(3);
	expect($(mergedCell).hasClass('fullySelectedMergedCell')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-multiple')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-0')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-1')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-2')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-3')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-4')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-5')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-6')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-7')).toBeFalse();
	});

I will investigate.

Edit: Situation with rows is even stranger. The table with some configuration (take a look at the below test) doesn't look properly.

handsontable/src/plugins/hiddenRows/test/plugins/mergeCells.e2e.js

Lines 1422 to 1465 in 7fbf9c8

	it('should select single merged area properly when it starts with hidden row', () => {
	handsontable({
	data: Handsontable.helper.createSpreadsheetData(5, 5),
	rowHeaders: true,
	colHeaders: true,
	contextMenu: true,
	hiddenRows: {
	rows: [1],
	indicators: true
	},
	mergeCells: [{ row: 1, col: 1, rowspan: 3, colspan: 3 }]
	});
	const mergedCell = spec().$container.find('tr:eq(2) td:eq(1)');
	simulateClick(mergedCell);
	// Third row is not displayed (CSS - display: none).
	expect(`
	| ║ : - : - : - : |
	|===:===:===:===:===:===|
	| ║ : : : : |
	| - ║ : # : : : |
	| - ║ : : : : |
	| ║ : : : : |
	`).toBeMatchToSelectionPattern();
	expect(getSelected()).toEqual([[1, 1, 3, 3]]);
	expect(getSelectedRangeLast().highlight.row).toBe(2);
	expect(getSelectedRangeLast().highlight.col).toBe(1);
	expect(getSelectedRangeLast().from.row).toBe(1);
	expect(getSelectedRangeLast().from.col).toBe(1);
	expect(getSelectedRangeLast().to.row).toBe(3);
	expect(getSelectedRangeLast().to.col).toBe(3);
	expect($(mergedCell).hasClass('fullySelectedMergedCell')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-multiple')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-0')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-1')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-2')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-3')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-4')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-5')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-6')).toBeFalse();
	expect($(mergedCell).hasClass('fullySelectedMergedCell-7')).toBeFalse();
	});

[aninde]

This issue is solved by 8.0.0-beta2-rev18 https://jsfiddle.net/aninde/xmotLk3u/