Skip to content

fix(frontend): clarify row and column insertion actions - #489

Open
Sohail52 wants to merge 2 commits into
c2siorg:mainfrom
Sohail52:fix-row-column-insertion-menu
Open

fix(frontend): clarify row and column insertion actions#489
Sohail52 wants to merge 2 commits into
c2siorg:mainfrom
Sohail52:fix-row-column-insertion-menu

Conversation

@Sohail52

@Sohail52 Sohail52 commented Aug 23, 2026

Copy link
Copy Markdown

Description

Brief summary of the changes. Reference any related issues.

Fixes #468

Type of Change

  • [ x] Bug fix
  • New feature
  • Breaking change
  • Documentation update

How Has This Been Tested?

Describe the tests you ran to verify your changes.

  • [ x] Existing tests pass
  • [ x] New tests added
  • [ x] Manual testing

Screenshots (if applicable)

Screenshot 2026-08-23 155019 Screenshot 2026-08-23 155031

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review
  • I have added/updated documentation as needed
  • My changes generate no new warnings
  • Tests pass locally

@hanzalahwaheed

Copy link
Copy Markdown
Contributor

@Sohail52 thanks for the contribution! I think we can club handleRowsAbove/Below into a single addRows with above/below as function parmas (DRY)

@hanzalahwaheed

Copy link
Copy Markdown
Contributor

@Sohail52 please do the same for handleAddColumns and squash all commits into one. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add Row/Column relatively

2 participants