search

anmcgrath / BlazorDatasheet

Simple excel-like datasheet Blazor component
MIT License
131 stars 31 forks source link

Merged cells impovement #49

Closed for7raid closed 1 year ago

for7raid commented 1 year ago

Hello @anmcgrath !

In this PR:

  1. Improve key navigation in DropDown based editors
  2. Updates merged cells after insert\remove rows\columns. After we insert or remove rows or columns we have to shift\expand\contact all merged cells located below, after or inside affected rows or columns.
anmcgrath commented 1 year ago

Thanks for the great PR. I found an issue when inserting inside column/row merges and added a fix for that. Please see if you agree.

for7raid commented 1 year ago

You are quite right. Only we need to skip it depend on inserted axis. updated code and tests. please take a look